From ef87e2e54f0f1af54c541f505f9cad0a7e81f654 Mon Sep 17 00:00:00 2001 From: Robin Lee Date: Tue, 21 Apr 2020 23:02:00 +0200 Subject: [PATCH] Change BLACK to TRANSPARENT to fix test * BLACK = 0xFF000000 * TRANSPARENT = 0x000000 These are not the same thing. We want lights that are off to show up as transparent, and they do. Test: atest LightsServiceTest Bug: 154638739 Change-Id: I6db2bef8c543688ad76577e5aae363f26ef0b1d2 --- .../src/com/android/server/lights/LightsServiceTest.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services/tests/servicestests/src/com/android/server/lights/LightsServiceTest.java b/services/tests/servicestests/src/com/android/server/lights/LightsServiceTest.java index d58937ea9cab9..3e9709d552686 100644 --- a/services/tests/servicestests/src/com/android/server/lights/LightsServiceTest.java +++ b/services/tests/servicestests/src/com/android/server/lights/LightsServiceTest.java @@ -16,9 +16,9 @@ package com.android.server.lights; -import static android.graphics.Color.BLACK; import static android.graphics.Color.BLUE; import static android.graphics.Color.GREEN; +import static android.graphics.Color.TRANSPARENT; import static android.graphics.Color.WHITE; import static android.hardware.lights.LightsRequest.Builder; @@ -128,7 +128,7 @@ public class LightsServiceTest { Light micLight = manager.getLights().get(0); // The light should begin by being off. - assertThat(manager.getLightState(micLight).getColor()).isEqualTo(BLACK); + assertThat(manager.getLightState(micLight).getColor()).isEqualTo(TRANSPARENT); // When a session commits changes: LightsManager.LightsSession session = manager.openSession(); @@ -139,7 +139,7 @@ public class LightsServiceTest { // When the session goes away: session.close(); // Then the light should turn off. - assertThat(manager.getLightState(micLight).getColor()).isEqualTo(BLACK); + assertThat(manager.getLightState(micLight).getColor()).isEqualTo(TRANSPARENT); } @Test