diff --git a/tests/unit/java/com/android/server/ConnectivityServiceTest.java b/tests/unit/java/com/android/server/ConnectivityServiceTest.java index 1608241c11..3507050bed 100644 --- a/tests/unit/java/com/android/server/ConnectivityServiceTest.java +++ b/tests/unit/java/com/android/server/ConnectivityServiceTest.java @@ -10490,6 +10490,12 @@ public class ConnectivityServiceTest { mCellNetworkAgent.connect(true); callback.expectAvailableThenValidatedCallbacks(mCellNetworkAgent); callback.assertNoCallback(); + + // Make sure a report is sent and that the caps are suitably redacted. + verify(mConnectivityDiagnosticsCallback, timeout(TIMEOUT_MS)) + .onConnectivityReportAvailable(argThat(report -> + areConnDiagCapsRedacted(report.getNetworkCapabilities()))); + reset(mConnectivityDiagnosticsCallback); } private boolean areConnDiagCapsRedacted(NetworkCapabilities nc) { @@ -10502,17 +10508,6 @@ public class ConnectivityServiceTest { && ti.settingsRedacted; } - @Test - public void testConnectivityDiagnosticsCallbackOnConnectivityReportAvailable() - throws Exception { - setUpConnectivityDiagnosticsCallback(); - - // Verify onConnectivityReport fired - verify(mConnectivityDiagnosticsCallback, timeout(TIMEOUT_MS)) - .onConnectivityReportAvailable(argThat(report -> - areConnDiagCapsRedacted(report.getNetworkCapabilities()))); - } - @Test public void testConnectivityDiagnosticsCallbackOnDataStallSuspected() throws Exception { setUpConnectivityDiagnosticsCallback(); @@ -10530,9 +10525,6 @@ public class ConnectivityServiceTest { public void testConnectivityDiagnosticsCallbackOnConnectivityReported() throws Exception { setUpConnectivityDiagnosticsCallback(); - // reset to ignore callbacks from setup - reset(mConnectivityDiagnosticsCallback); - final Network n = mCellNetworkAgent.getNetwork(); final boolean hasConnectivity = true; mService.reportNetworkConnectivity(n, hasConnectivity); @@ -10564,9 +10556,6 @@ public class ConnectivityServiceTest { throws Exception { setUpConnectivityDiagnosticsCallback(); - // reset to ignore callbacks from setup - reset(mConnectivityDiagnosticsCallback); - // report known Connectivity from a different uid. Verify that network is not re-validated // and this callback is not notified. final Network n = mCellNetworkAgent.getNetwork();