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

Commit fb7a391b authored by Andreas Gampe's avatar Andreas Gampe
Browse files

Telecomm: Fix PhoneAccountRegistrarTest 

Fix state comparison in PhoneAccountRegistrarTest to actually do
something (and not compare nulls).

Found by Errorprone.

Bug: PhoneAccountRegistrarTest
Test: m javac-check RUN_ERROR_PRONE=true
Test: atest PhoneAccountRegistrarTest
Change-Id: Ied3b3b3e38e3b73a37479dd2bdbac8a3e3fc3228
parent 1839ce18

tests/src/com/android/server/telecom/tests/PhoneAccountRegistrarTest.java

+5 −3
Original line number Diff line number Diff line
@@ -59,6 +59,7 @@ import java.io.ByteArrayOutputStream; 
import java.io.File;

import java.util.Arrays;

import java.util.List;

import java.util.Map;

import java.util.Set;



import static org.junit.Assert.assertEquals;
@@ -1036,9 +1037,10 @@ public class PhoneAccountRegistrarTest extends TelecomTestCase { 
            PhoneAccountRegistrar.State a, PhoneAccountRegistrar.State b) {

        assertEquals(a.defaultOutgoingAccountHandles.size(),

                b.defaultOutgoingAccountHandles.size());

        for (int i = 0; i < a.defaultOutgoingAccountHandles.size(); i++) {

            assertDefaultPhoneAccountHandleEquals(a.defaultOutgoingAccountHandles.get(i),

                    b.defaultOutgoingAccountHandles.get(i));

        for (Map.Entry<UserHandle, DefaultPhoneAccountHandle> e :

                a.defaultOutgoingAccountHandles.entrySet()) {

            assertDefaultPhoneAccountHandleEquals(e.getValue(),

                    b.defaultOutgoingAccountHandles.get(e.getKey()));

        }

        assertEquals(a.accounts.size(), b.accounts.size());

        for (int i = 0; i < a.accounts.size(); i++) {