Skip to content

Commit 1c9d86c

Browse files
author
Aurélien Pomini
committed
Improve test invokeOnColorsChangedListenerTest_clearLock
Currently, the test sets a shared bitmap on home & lock screen, then clears the wallpaper on either home, lock, or home+lock, then checks that we receive color events for the clear operation. This is dubious for the clear(lock) case: clear(lock) means "set the lock screen wallpaper to be the same as the home screen wallpaper". Since we set a shared bitmap before clearing, the lock screen wallpaper is already the same and this clear(lock) is a no-op. It is absolutely fine not to receive a color event in that case. This CLs sets a bitmap only to the destination that is cleared, before running the checks on the clear method. Flag: NONE Test: atest WallpaperManagerTest --iterations 20 Bug: 308757171 Change-Id: I8ff8ce28f6d3b18d8326b111938fbedc35d3e0bb
1 parent 82bb023 commit 1c9d86c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tests/app/WallpaperTest/src/android/app/cts/wallpapers/WallpaperManagerTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1612,7 +1612,7 @@ private void verifyColorListenerInvoked(int which, int whichExpected) {
16121612
* @param which FLAG_LOCK, FLAG_SYSTEM or a combination of both.
16131613
*/
16141614
private void verifyColorListenerInvokedClearing(int which) {
1615-
ensureCleanState();
1615+
ensureCleanState(which);
16161616

16171617
final CountDownLatch latch = new CountDownLatch(1);
16181618

0 commit comments

Comments
 (0)