Bug 1815339 - Flip the assertion for the GeckoView translation test; r=geckoview-reviewers,ohall

gregtatum · gregtatum · commit 86aef9268791 · 2023-11-04T21:57:47.000Z
Differential Revision: https://phabricator.services.mozilla.com/D192594
diff --git a/mobile/android/geckoview/src/androidTest/java/org/mozilla/geckoview/test/TranslationsTest.kt b/mobile/android/geckoview/src/androidTest/java/org/mozilla/geckoview/test/TranslationsTest.kt
@@ -109,10 +109,9 @@ class TranslationsTest : BaseSessionTest() {
         val translate = mainSession.sessionTranslation!!.translate("es", "en", null)
         try {
             sessionRule.waitForResult(translate)
-            // ToDo: bug 1853469 models not available in automation
-            assertTrue("Should not be able to translate.", false)
+            assertTrue("Should be able to translate.", true)
         } catch (e: Exception) {
-            assertTrue("Should have an exception.", true)
+            assertTrue("Should not have an exception.", false)
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -109,10 +109,9 @@ class TranslationsTest : BaseSessionTest() {`
`109`	`109`	`val translate = mainSession.sessionTranslation!!.translate("es", "en", null)`
`110`	`110`	`try {`
`111`	`111`	`sessionRule.waitForResult(translate)`
`112`		`- // ToDo: bug 1853469 models not available in automation`
`113`		`- assertTrue("Should not be able to translate.", false)`
	`112`	`+ assertTrue("Should be able to translate.", true)`
`114`	`113`	`} catch (e: Exception) {`
`115`		`- assertTrue("Should have an exception.", true)`
	`114`	`+ assertTrue("Should not have an exception.", false)`
`116`	`115`	`}`
`117`	`116`	`}`
`118`	`117`