diff options
| author | markchien <markchien@google.com> | 2022-03-02 18:07:35 +0800 |
|---|---|---|
| committer | markchien <markchien@google.com> | 2022-03-09 11:17:27 +0800 |
| commit | e46042b833c96ae18045fe9b96e6584db9447b4e (patch) | |
| tree | af64ba157d30b7a2fee2762780d8f0781161cecd /tests/unit/java/com/android/server/NetworkManagementServiceTest.java | |
| parent | 3b45bca306c77cdc0fcb45d168bf24dec987c618 (diff) | |
Split updateMeteredNetwork{Allow, Deny}List into add and remove
Split the updateXXX methods into an addXXX and removeXXX instead of using
a boolean parameter to indicate whether the uid should be added or removed
Bug: 218494748
Test: atest FrameworksNetTests
Change-Id: I868cf35c8f51b25d8719e618c6c48a5cd642da7f
Diffstat (limited to 'tests/unit/java/com/android/server/NetworkManagementServiceTest.java')
| -rw-r--r-- | tests/unit/java/com/android/server/NetworkManagementServiceTest.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/unit/java/com/android/server/NetworkManagementServiceTest.java b/tests/unit/java/com/android/server/NetworkManagementServiceTest.java index a3b0e7c8ab..7688a6ba5c 100644 --- a/tests/unit/java/com/android/server/NetworkManagementServiceTest.java +++ b/tests/unit/java/com/android/server/NetworkManagementServiceTest.java @@ -250,7 +250,7 @@ public class NetworkManagementServiceTest { mNMService.setUidOnMeteredNetworkDenylist(TEST_UID, true); assertTrue("Should be true since mobile data usage is restricted", mNMService.isNetworkRestricted(TEST_UID)); - verify(mCm).updateMeteredNetworkDenyList(TEST_UID, true /* enabled */); + verify(mCm).addUidToMeteredNetworkDenyList(TEST_UID); mNMService.setDataSaverModeEnabled(true); verify(mNetdService).bandwidthEnableDataSaver(true); @@ -258,16 +258,16 @@ public class NetworkManagementServiceTest { mNMService.setUidOnMeteredNetworkDenylist(TEST_UID, false); assertTrue("Should be true since data saver is on and the uid is not allowlisted", mNMService.isNetworkRestricted(TEST_UID)); - verify(mCm).updateMeteredNetworkDenyList(TEST_UID, true /* false */); + verify(mCm).removeUidFromMeteredNetworkDenyList(TEST_UID); mNMService.setUidOnMeteredNetworkAllowlist(TEST_UID, true); assertFalse("Should be false since data saver is on and the uid is allowlisted", mNMService.isNetworkRestricted(TEST_UID)); - verify(mCm).updateMeteredNetworkAllowList(TEST_UID, true /* enabled */); + verify(mCm).addUidToMeteredNetworkAllowList(TEST_UID); // remove uid from allowlist and turn datasaver off again mNMService.setUidOnMeteredNetworkAllowlist(TEST_UID, false); - verify(mCm).updateMeteredNetworkAllowList(TEST_UID, false /* enabled */); + verify(mCm).removeUidFromMeteredNetworkAllowList(TEST_UID); mNMService.setDataSaverModeEnabled(false); verify(mNetdService).bandwidthEnableDataSaver(false); assertFalse("Network should not be restricted when data saver is off", |