Skip to content
Commit 36fc266f authored by Gustav Sennton's avatar Gustav Sennton
Browse files

Mark smart replies as sent on notification update/removal.

To keep notifications for which the user has sent replies we have some
logic using 'lifetime-extenders'. The SmartReply lifetime extender
ensures that we keep notifications around at least while smart replies
are in the process of being sent.
However, before this CL we never marked smart replies as sent - after
sending them we would mark them as 'sending' but not 'sent'. With this
CL we mark smart replies as 'sent' on notification update and removal to
allow notifications with smart replies to be removed.

Test note: I'm not adding any unit tests in this CL - the existing tests
only use mocks (not fakes), and there's thus no easy way to ensure we
call stopSending() from one of the NotificationEntryListener callbacks.

Bug: 122140949
Test: manual
1. Receive message
2. Tap on smart reply -> smart reply sent
3. receive new message (so 'mark as read' button appears)
4. tap 'mark as read'
5. ensure notification removed.
Test: manual
1. Receive message
2. Respond using reply button (remote-input) -> reply sent
3. receive new message (so 'mark as read' button appears)
4. tap 'mark as read'
5. ensure notification removed.
Change-Id: I2f0edba65f81d5b6fb577e9f83374c404a99e3fb
parent 2a4e6f6e
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment