Merge pull request #44 from morckx/actually-check-initial-playback-state

romanofranz · web-flow · commit 3e50bf59c36d · 2022-10-20T10:07:37.000+02:00
Actually check initial playback state in initialPlaybackStateTest
diff --git a/mediacontroller/src/main/java/com/example/android/mediacontroller/testing/MediaAppTestSuite.kt b/mediacontroller/src/main/java/com/example/android/mediacontroller/testing/MediaAppTestSuite.kt
@@ -113,7 +113,7 @@ Array<TestOptionDetails>) {
             Looper.prepare()
             try {
                 for(i in 0 until numIter) {
-                    for (test in testList) {
+                    for (test in testList.sortedBy { it.id }) {
                         resetSingleResults()
                         onStartTest()
 
diff --git a/mediacontroller/src/main/java/com/example/android/mediacontroller/testing/TestDescriptor.kt b/mediacontroller/src/main/java/com/example/android/mediacontroller/testing/TestDescriptor.kt
@@ -296,7 +296,7 @@ class TestDescriptor {
         }
 
         val initialPlaybackStateTest = TestOptionDetails(
-            15,
+            -1,  // this test must be run first
             context.getString(R.string.playback_state_test_title),
             context.getString(R.string.playback_state_test_desc),
             TestResult.NONE,

Original file line number	Diff line number	Diff line change
`@@ -296,7 +296,7 @@ class TestDescriptor {`
`296`	`296`	`}`
`297`	`297`
`298`	`298`	`val initialPlaybackStateTest = TestOptionDetails(`
`299`		`- 15,`
	`299`	`+ -1, // this test must be run first`
`300`	`300`	`context.getString(R.string.playback_state_test_title),`
`301`	`301`	`context.getString(R.string.playback_state_test_desc),`
`302`	`302`	`TestResult.NONE,`