Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c95bc4d6 authored Feb 09, 2022 by Nataniel Borges

Run flicker tests 3x

We run the flicker tests multiple times to detect CLs that intoruce flakiness early. Currently we run 5 executions of each CUJ. However, due to lab capacity and impact in presubmit times, this CL reduces the repetitions to 3

Bug: 209843622
Test: atest FlickerTests WMShellFlickerTests
Change-Id: I165937fb82953d62c028ffe96a948d0ca00dbbe2

parent 154a3c55

libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/bubble/BaseBubbleScreen.kt

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -95,7 +95,7 @@ abstract class BaseBubbleScreen(protected val testSpec: FlickerTestParameter) {
	fun getParams(): List<FlickerTestParameter> {		fun getParams(): List<FlickerTestParameter> {
	return FlickerTestParameterFactory.getInstance()		return FlickerTestParameterFactory.getInstance()
	.getConfigNonRotationTests(supportedRotations = listOf(Surface.ROTATION_0),		.getConfigNonRotationTests(supportedRotations = listOf(Surface.ROTATION_0),
	repetitions = 5)		repetitions = 3)
	}		}

	const val FIND_OBJECT_TIMEOUT = 2000L		const val FIND_OBJECT_TIMEOUT = 2000L

libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/EnterPipTest.kt

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -205,7 +205,7 @@ class EnterPipTest(testSpec: FlickerTestParameter) : PipTransition(testSpec) {
	fun getParams(): List<FlickerTestParameter> {		fun getParams(): List<FlickerTestParameter> {
	return FlickerTestParameterFactory.getInstance()		return FlickerTestParameterFactory.getInstance()
	.getConfigNonRotationTests(supportedRotations = listOf(Surface.ROTATION_0),		.getConfigNonRotationTests(supportedRotations = listOf(Surface.ROTATION_0),
	repetitions = 5)		repetitions = 3)
	}		}
	}		}
	}		}

libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/EnterPipToOtherOrientationTest.kt

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -224,7 +224,7 @@ class EnterPipToOtherOrientationTest(
	fun getParams(): Collection<FlickerTestParameter> {		fun getParams(): Collection<FlickerTestParameter> {
	return FlickerTestParameterFactory.getInstance()		return FlickerTestParameterFactory.getInstance()
	.getConfigNonRotationTests(supportedRotations = listOf(Surface.ROTATION_0),		.getConfigNonRotationTests(supportedRotations = listOf(Surface.ROTATION_0),
	repetitions = 5)		repetitions = 3)
	}		}
	}		}
	}		}

libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/ExitPipViaExpandButtonClickTest.kt

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -98,7 +98,7 @@ class ExitPipViaExpandButtonClickTest(
	@JvmStatic		@JvmStatic
	fun getParams(): List<FlickerTestParameter> {		fun getParams(): List<FlickerTestParameter> {
	return FlickerTestParameterFactory.getInstance().getConfigNonRotationTests(		return FlickerTestParameterFactory.getInstance().getConfigNonRotationTests(
	supportedRotations = listOf(Surface.ROTATION_0), repetitions = 5)		supportedRotations = listOf(Surface.ROTATION_0), repetitions = 3)
	}		}
	}		}
	}		}
			No newline at end of file

libs/WindowManager/Shell/tests/flicker/src/com/android/wm/shell/flicker/pip/ExitPipViaIntentTest.kt

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -117,7 +117,7 @@ class ExitPipViaIntentTest(testSpec: FlickerTestParameter) : ExitPipToAppTransit
	@JvmStatic		@JvmStatic
	fun getParams(): List<FlickerTestParameter> {		fun getParams(): List<FlickerTestParameter> {
	return FlickerTestParameterFactory.getInstance().getConfigNonRotationTests(		return FlickerTestParameterFactory.getInstance().getConfigNonRotationTests(
	supportedRotations = listOf(Surface.ROTATION_0), repetitions = 5)		supportedRotations = listOf(Surface.ROTATION_0), repetitions = 3)
	}		}
	}		}
	}		}