commit 6cd149f5057cb1a04a07042a7dc929bbd5d7d469
author Thomas Stuart <tjstuart@google.com> Fri Jan 19 16:45:50 2024 -0800
committer Thomas Stuart <tjstuart@google.com> Sat Jan 20 01:06:12 2024 +0000
tree 68b04c4b9ac6a27fa2930fb2e605ae0535b5a9ed
parent d225bde8bac12a3cd2c4a076042b68866d341aff

adjust CallControl tests

API changes:
- change setMuteState to requestMuteState
- change @Nullable CallControl to @NonNull

Impl changes:
- remove ClientTransactionalWrapper from the hidden contructor
- remove PhoneAccountHandle from the hidden constructor
- remove the if serviceInterface != null clause

Fixes: 320507724
Test: adjusted CallControlTest class
Change-Id: I900abce261fa636cffdf52b7c2edf5e8cbf04637

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

diff --git a/tests/src/com/android/server/telecom/tests/CallControlTest.java b/tests/src/com/android/server/telecom/tests/CallControlTest.java
index 2613206..c69521a 100644
--- a/tests/src/com/android/server/telecom/tests/CallControlTest.java
+++ b/tests/src/com/android/server/telecom/tests/CallControlTest.java
@@ -39,14 +39,7 @@
 import java.util.UUID;
 
 public class CallControlTest extends TelecomTestCase {
-
-    private static final PhoneAccountHandle mHandle = new PhoneAccountHandle(
-            new ComponentName("foo", "bar"), "1");
-
-    @Mock
-    private ICallControl mICallControl;
-    @Mock
-    private ClientTransactionalServiceRepository mRepository;
+    @Mock private ICallControl mICallControl;
     private static final String CALL_ID_1 = UUID.randomUUID().toString();
 
     @Override
@@ -64,15 +57,7 @@
 
     @Test
     public void testGetCallId() {
-        CallControl control = new CallControl(CALL_ID_1, mICallControl, mRepository, mHandle);
+        CallControl control = new CallControl(CALL_ID_1, mICallControl);
         assertEquals(CALL_ID_1, control.getCallId().toString());
     }
-
-    @Test
-    public void testCallControlHitsIllegalStateException() {
-        CallControl control = new CallControl(CALL_ID_1, null, mRepository, mHandle);
-        assertThrows(IllegalStateException.class, () ->
-                control.setInactive(Runnable::run, result -> {
-                }));
-    }
 }