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

Commit 31901f1c authored by Kihong Seong's avatar Kihong Seong
Browse files

Revert "Comment one verify in two tests that is currently failing." 

This reverts commit fa37316b.

Reason for revert: Found a fix for the test, and will upload a CL for it.

Change-Id: I181e39011042a5e9f1e945d4bb05e15420de0215
parent 2324ad0d

android/app/tests/unit/src/com/android/bluetooth/hfp/AtPhonebookTest.java

+4 −8
Original line number Diff line number Diff line
@@ -110,11 +110,9 @@ public class AtPhonebookTest { 


    @Test

    public void handleCpbsCommand() {

        //Check removed in order to have clean instrumentation runs

        //TODO b/255833353: Fix

        //mAtPhonebook.handleCpbsCommand(INVALID_COMMAND, AtPhonebook.TYPE_READ, mTestDevice);

        //verify(mNativeInterface).atResponseString(mTestDevice,

        //        "+CPBS: \"" + "ME" + "\"," + 0 + "," + 256);

        mAtPhonebook.handleCpbsCommand(INVALID_COMMAND, AtPhonebook.TYPE_READ, mTestDevice);

        verify(mNativeInterface).atResponseString(mTestDevice,

                "+CPBS: \"" + "ME" + "\"," + 0 + "," + 256);



        mAtPhonebook.handleCpbsCommand(INVALID_COMMAND, AtPhonebook.TYPE_TEST, mTestDevice);

        verify(mNativeInterface).atResponseString(mTestDevice,
@@ -140,9 +138,7 @@ public class AtPhonebookTest { 
    @Test

    public void handleCpbrCommand() {

        mAtPhonebook.handleCpbrCommand(INVALID_COMMAND, AtPhonebook.TYPE_TEST, mTestDevice);

        //Check removed in order to have clean instrumentation runs

        //TODO b/255833353: Fix

        //verify(mNativeInterface).atResponseString(mTestDevice, "+CPBR: (1-" + 1 + "),30,30");

        verify(mNativeInterface).atResponseString(mTestDevice, "+CPBR: (1-" + 1 + "),30,30");

        verify(mNativeInterface).atResponseCode(mTestDevice, HeadsetHalConstants.AT_RESPONSE_OK,

                -1);