From e6d86e20aa27084a2fd88ea5b244bb242bb2b1cf Mon Sep 17 00:00:00 2001 From: Vinit Nayak Date: Tue, 28 Mar 2023 22:04:46 -0700 Subject: [PATCH] Consolidate WorkProfileTest "@After" methods * There were two separate methods, which can run in any order. Consolidate to reduce variability Bug: 243688989 Change-Id: Ic924aa8a5d75505aecf6c830c63a85a0d2028d83 --- tests/src/com/android/launcher3/ui/WorkProfileTest.java | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/tests/src/com/android/launcher3/ui/WorkProfileTest.java b/tests/src/com/android/launcher3/ui/WorkProfileTest.java index c22f93dca3..199653b613 100644 --- a/tests/src/com/android/launcher3/ui/WorkProfileTest.java +++ b/tests/src/com/android/launcher3/ui/WorkProfileTest.java @@ -36,6 +36,7 @@ import com.android.launcher3.allapps.WorkEduCard; import com.android.launcher3.allapps.WorkPausedCard; import com.android.launcher3.allapps.WorkProfileManager; import com.android.launcher3.tapl.LauncherInstrumentation; +import com.android.launcher3.testing.shared.TestProtocol; import org.junit.After; import org.junit.Before; @@ -90,17 +91,14 @@ public class WorkProfileTest extends AbstractLauncherUiTest { @After public void removeWorkProfile() throws Exception { - mDevice.executeShellCommand("pm remove-user " + mProfileUserId); - } - - @After - public void resumeAppStoreUpdate() { + Log.d(TestProtocol.WORK_TAB_MISSING, "WorkProfileTest teardown"); executeOnLauncher(launcher -> { if (launcher == null || launcher.getAppsView() == null) { return; } launcher.getAppsView().getAppsStore().disableDeferUpdates(DEFER_UPDATES_TEST); }); + mDevice.executeShellCommand("pm remove-user " + mProfileUserId); } private void waitForWorkTabSetup() {