Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make testRemoveUserByHandle_ThrowsException fail faster.

Browse Source 
Currently UserManagerTest#testRemoveUserByHandle_ThrowsException uses
the helper 'removeUser' method which registers for a
BlockingBroadcastReceiver and since we're not expecting a broadcast
it only fails after the timeout. This CL removes that unnecessary step
and decreases this test method's duration from 3 minutes to 1 ms.

Test: atest UserManagerTest#testRemoveUserByHandle_ThrowsException
Change-Id: I9302054cb53b1e0027bb2bcb9da74d98f390f2ed
This commit is contained in:
Yasin Kilicdere
2023-02-02 23:56:36 +00:00
parent bef72a409e
commit fb7d3be548
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/tests/servicestests/src/com/android/server/pm/UserManagerTest.java
View File

@@ -302,7 +302,7 @@ public final class UserManagerTest {
@MediumTest
@Test
public void testRemoveUserByHandle_ThrowsException() {
assertThrows(IllegalArgumentException.class, () -> removeUser(null));
assertThrows(IllegalArgumentException.class, () -> mUserManager.removeUser(null));
}
@MediumTest