Revert "Prevent some leaks from leaking to the next test"
This reverts commit d17a270b31.
Reason for revert: Perhaps, caused failures like: Launcher internal state didn't switch to Home, https://android-build.corp.google.com/test_investigate/?invocationId=I32700010190010056&testResultId=TR70828881289196802
Change-Id: I1c7afb75785aac6b02f451b9d5b74de7833abfed
This commit is contained in:
committed by
Android (Google) Code Review
parent
d17a270b31
commit
84f51ec03e
@@ -68,7 +68,6 @@ import com.android.launcher3.util.rule.FailureWatcher;
|
||||
import com.android.launcher3.util.rule.SamplerRule;
|
||||
import com.android.launcher3.util.rule.ScreenRecordRule;
|
||||
import com.android.launcher3.util.rule.ShellCommandRule;
|
||||
import com.android.launcher3.util.rule.TestIsolationRule;
|
||||
import com.android.launcher3.util.rule.TestStabilityRule;
|
||||
import com.android.launcher3.util.rule.ViewCaptureRule;
|
||||
|
||||
@@ -124,10 +123,6 @@ public abstract class AbstractLauncherUiTest {
|
||||
}, DEFAULT_UI_TIMEOUT, launcher);
|
||||
}
|
||||
|
||||
public void checkDetectedLeaks() {
|
||||
checkDetectedLeaks(mLauncher);
|
||||
}
|
||||
|
||||
private static String getActivityLeakErrorMessage(LauncherInstrumentation launcher) {
|
||||
sActivityLeakReported = true;
|
||||
return "Activity leak detector has found leaked activities, "
|
||||
@@ -220,8 +215,7 @@ public abstract class AbstractLauncherUiTest {
|
||||
|
||||
@Rule
|
||||
public TestRule mOrderSensitiveRules = RuleChain
|
||||
.outerRule(new TestIsolationRule(this))
|
||||
.around(new SamplerRule())
|
||||
.outerRule(new SamplerRule())
|
||||
.around(new TestStabilityRule())
|
||||
.around(getRulesInsideActivityMonitor());
|
||||
|
||||
@@ -494,12 +488,6 @@ public abstract class AbstractLauncherUiTest {
|
||||
}
|
||||
|
||||
protected void closeLauncherActivity() {
|
||||
finishLauncherActivity();
|
||||
waitForLauncherCondition(
|
||||
"Launcher still active", launcher -> launcher == null, DEFAULT_UI_TIMEOUT);
|
||||
}
|
||||
|
||||
public void finishLauncherActivity() {
|
||||
// Destroy Launcher activity.
|
||||
executeOnLauncher(launcher -> {
|
||||
if (launcher != null) {
|
||||
@@ -507,6 +495,8 @@ public abstract class AbstractLauncherUiTest {
|
||||
launcher.finish();
|
||||
}
|
||||
});
|
||||
waitForLauncherCondition(
|
||||
"Launcher still active", launcher -> launcher == null, DEFAULT_UI_TIMEOUT);
|
||||
}
|
||||
|
||||
protected boolean isInLaunchedApp(Launcher launcher) {
|
||||
|
||||
Reference in New Issue
Block a user