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

Commit f37635b3 authored by Qiong Liu's avatar Qiong Liu Committed by Tyler Gunn
Browse files

Fix test failure of CallLogManagerTest#testLogCorrectPhoneNumber 

In CallLogManagerTest#testLogCorrectPhoneNumber, the expected number
is formatted to "US" convention, while the logged number is formatted
with current country iso. So the logged number doesn't match the
expected if running test when current country iso is not "US", for
example, when the system language is not English(United State).

The change is to format the expected number with current country iso.

Test: atest CallLogManagerTest
Bug: 384837722
Flag: NONE test fix.
Change-Id: Icfced42c646ff850a112d581a6ad08699b1ae73a
parent b3a412e4

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

+2 −1
Original line number Diff line number Diff line
@@ -588,7 +588,8 @@ public class CallLogManagerTest extends TelecomTestCase { 
        assertEquals(insertedValues.getAsString(CallLog.Calls.NUMBER),

                TEL_PHONEHANDLE.getSchemeSpecificPart());

        assertEquals(insertedValues.getAsString(CallLog.Calls.POST_DIAL_DIGITS), POST_DIAL_STRING);

        String expectedNumber = PhoneNumberUtils.formatNumber(VIA_NUMBER_STRING, "US");

        String expectedNumber = PhoneNumberUtils.formatNumber(VIA_NUMBER_STRING,

                mCallLogManager.getCountryIso());

        assertEquals(insertedValues.getAsString(Calls.VIA_NUMBER), expectedNumber);

    }