commit 96d3d20f4372337f121899ad1c10e1194da57fc0
author Etan Cohen <etancohen@google.com> Thu May 23 16:01:51 2019 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Thu May 23 16:01:51 2019 +0000
tree f2663c308ee8dc8135dc9145fe49c2bb14ca0d03
parent f9a84cd5307fe8a578a602ffb03db732995ad677
parent bcf0006ff73f8e059ddeb905ef4d7bafaeb78cc1

Merge "[CM] Fix NPE due to unvalidated callback value"

tests/net/java/com/android/server/ConnectivityServiceTest.java

diff --git a/tests/net/java/com/android/server/ConnectivityServiceTest.java b/tests/net/java/com/android/server/ConnectivityServiceTest.java
index 1f32bd4..5934fcb 100644
--- a/tests/net/java/com/android/server/ConnectivityServiceTest.java
+++ b/tests/net/java/com/android/server/ConnectivityServiceTest.java
@@ -3864,6 +3864,7 @@
             }
         }
 
+        testFactory.expectRemoveRequests(1);
         if (preUnregister) {
             mCm.unregisterNetworkCallback(networkCallback);
 
@@ -3873,7 +3874,6 @@
             testFactory.triggerUnfulfillable(requests.get(newRequestId));
         } else {
             // Simulate the factory releasing the request as unfulfillable and expect onUnavailable!
-            testFactory.expectRemoveRequests(1);
             testFactory.triggerUnfulfillable(requests.get(newRequestId));
 
             networkCallback.expectCallback(CallbackState.UNAVAILABLE, null);