From 5e53ad62aec70f904af3dd39c3ffda4454a80177 Mon Sep 17 00:00:00 2001 From: Chandru S Date: Thu, 13 Apr 2023 12:20:44 -0700 Subject: [PATCH] Switch the mocking session strictness to WARN from STRICT KeyguardUpdateMonitorTest The stubbing is not great in this test, not all flows are stubbed out properly, if a device is not provisioned when the test runs, mockito complains quite a bit about either unmatched stubs or unnecessary stubbing. Bug: 277873197 Test: atest KeyguardUpdateMonitorTest Test: force mDeviceProvisioned flag to false in KeyguardUpdateMonitor run the test, it should not fail Change-Id: I5adf4e50e1fa93807dc2e6bab53ddc78963605f6 --- .../src/com/android/keyguard/KeyguardUpdateMonitorTest.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java b/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java index db86ac6fbb0da..ed40eed549783 100644 --- a/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java +++ b/packages/SystemUI/tests/src/com/android/keyguard/KeyguardUpdateMonitorTest.java @@ -155,6 +155,7 @@ import org.mockito.Mock; import org.mockito.MockitoAnnotations; import org.mockito.MockitoSession; import org.mockito.internal.util.reflection.FieldSetter; +import org.mockito.quality.Strictness; import java.util.ArrayList; import java.util.Arrays; @@ -303,6 +304,7 @@ public class KeyguardUpdateMonitorTest extends SysuiTestCase { mMockitoSession = ExtendedMockito.mockitoSession() .spyStatic(SubscriptionManager.class) + .strictness(Strictness.WARN) .startMocking(); ExtendedMockito.doReturn(SubscriptionManager.INVALID_SUBSCRIPTION_ID) .when(SubscriptionManager::getDefaultSubscriptionId);