Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ef87e2e5 authored by Robin Lee's avatar Robin Lee
Browse files

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
parent 9e118130

services/tests/servicestests/src/com/android/server/lights/LightsServiceTest.java

+3 −3
Original line number Diff line number Diff line
@@ -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