| 6dceace0eddd08156e6b71c17e3de4ed5f4f2f41 |
|
16-May-2018 |
Jason Monk <jmonk@google.com> |
Reduce usages of main looper in sysui tests
Push over to a standard testable looper, and a testable looper +
setAsMain when needed.
Also make tests more synchronous and single threaded as possible.
This will make them more deterministic and speeds them up noticeably.
Test: runtest systemui
Bug: 79550837
Change-Id: Iab0eb794329d7b1de95aef904ec08ecae7dadc98
/frameworks/base/packages/SystemUI/tests/src/com/android/systemui/statusbar/NotificationBlockingHelperManagerTest.java
|
| ae5fdf099326a194d263096c90179206a9a768a8 |
|
10-May-2018 |
Rohan Shah <shahrk@google.com> |
Move BH test to flaky and change it slightly
Marked the test as flaky. Additionally, changed how mocks are created.
This may or may not affect the test's passing rate. Also updated the
double spy horribleness.
Bug: 79444334
Test: Ran atest, no functional changes
Change-Id: Ieb901a1702ab15bdf98f37cb4b37d5068882162d
/frameworks/base/packages/SystemUI/tests/src/com/android/systemui/statusbar/NotificationBlockingHelperManagerTest.java
|
| 4ed1b2ad49f8da7cd6995d41cfde29cc25a40121 |
|
30-Mar-2018 |
Rohan Shah <shahrk@google.com> |
[Notif] Cache bool instead of manager/info
Follow up to ag/3815403
Updated to cache bool instead of manager/info in order to reduce IPC
calls made thru packagemanager.
Also collapsed test setup for shade.
Change-Id: I8fa116b3f24eaa887d2139c0aab1996e3c95fbdc
Fixes: 74115090
Test: Visually & atest for ExpandableNR
/frameworks/base/packages/SystemUI/tests/src/com/android/systemui/statusbar/NotificationBlockingHelperManagerTest.java
|
| 63411fc9f3fe35ccf4256f123f4f4ec915c44c44 |
|
29-Mar-2018 |
Rohan Shah <shahrk@google.com> |
[Notif] Dont show blocking helper for non-blockable notifs
If the notification is considered non-blockable, we don't want to show
it. Moved necessary logic around into the appropriate classes (minor
refactor).
Cached system info (IPC) in ExpandableNotifRow instead of calling so
late in the game.
Updated tests & added nonblocking checks.
Change-Id: Ic010a0b32cf6df42e9d0d2c600cbbe95a68ffde2
Bug: 74115090
Test: atest {notif tests}
Test: Visually via debug flag on sys notif
/frameworks/base/packages/SystemUI/tests/src/com/android/systemui/statusbar/NotificationBlockingHelperManagerTest.java
|
| 524cf7b68df47b39509cee59523458efb7611210 |
|
15-Mar-2018 |
Rohan Shah <shahrk@google.com> |
[Notif] Add Blocking helper to swipe
Implemented blocking helper UI as part of guts - the
ExpandableNotificationRow that's swiped away is effectively detached
from the notification logic but kept in as a view in the layout (it'll
still show up when looping through the normal view hierarchy). We
purposefully open up NotificationInfo with the intent of showing it as
the blocking helper.
It's animated away/closed when the user interacts with any other content
or swipes away, as with guts (Can be changed in the future if we want it
to stick around for longer).
KIs, that will be tracked in bugs once CL is in:
- Swiping a notification if it's partially hidden by the shelf does not
bring it up into focus
- If only one notification is showing, the shade auto-closes as if it
didn't have any notifications (which it technically doesn't)
- Animation on multiple blocking helper shows cuts too early and doesn't
allow the notification to transition out
- Long press on notification dismisses (it should probably ignore it)
- Metrics are still missing
- Accessibility needs a runthrough
Bug: 63095540,73783854
Test: Visually & Ran all tests
Change-Id: Iaa5e3d5537bbb72c946fdd7ee35b155e22ca2b05
/frameworks/base/packages/SystemUI/tests/src/com/android/systemui/statusbar/NotificationBlockingHelperManagerTest.java
|