Remove unneeded temporary UI test ignores (#17980)
This commit is contained in:
parent
b97c737a47
commit
bab7693b9d
|
@ -31,7 +31,6 @@ import org.mozilla.fenix.ui.robots.mDevice
|
|||
*
|
||||
*/
|
||||
|
||||
@Ignore("Temp disable - reader view page detection issues: https://github.com/mozilla-mobile/fenix/issues/9688 ")
|
||||
class ReaderViewTest {
|
||||
private lateinit var mockWebServer: MockWebServer
|
||||
private var readerViewNotification: ViewVisibilityIdlingResource? = null
|
||||
|
@ -105,8 +104,7 @@ class ReaderViewTest {
|
|||
|
||||
@Test
|
||||
fun verifyReaderViewToggle() {
|
||||
// New three-dot menu design does not have readerview
|
||||
if (!FeatureFlags.toolbarMenuFeature) {
|
||||
// New three-dot menu design does not have readerview appearance menu item
|
||||
val readerViewPage =
|
||||
TestAssetHelper.getLoremIpsumAsset(mockWebServer)
|
||||
|
||||
|
@ -128,11 +126,13 @@ class ReaderViewTest {
|
|||
mDevice.waitForIdle()
|
||||
}
|
||||
|
||||
if (!FeatureFlags.toolbarMenuFeature) {
|
||||
browserScreen {
|
||||
verifyPageContent(estimatedReadingTime)
|
||||
}.openThreeDotMenu {
|
||||
verifyReaderViewAppearance(true)
|
||||
}.closeBrowserMenuToBrowser { }
|
||||
}
|
||||
|
||||
navigationToolbar {
|
||||
toggleReaderView()
|
||||
|
@ -141,9 +141,9 @@ class ReaderViewTest {
|
|||
verifyReaderViewAppearance(false)
|
||||
}.close { }
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
@Ignore("To be re-implemented in https://github.com/mozilla-mobile/fenix/issues/17971")
|
||||
fun verifyReaderViewAppearanceFontToggle() {
|
||||
val readerViewPage =
|
||||
TestAssetHelper.getLoremIpsumAsset(mockWebServer)
|
||||
|
@ -184,6 +184,7 @@ class ReaderViewTest {
|
|||
}
|
||||
|
||||
@Test
|
||||
@Ignore("To be re-implemented in https://github.com/mozilla-mobile/fenix/issues/17971")
|
||||
fun verifyReaderViewAppearanceFontSizeToggle() {
|
||||
val readerViewPage =
|
||||
TestAssetHelper.getLoremIpsumAsset(mockWebServer)
|
||||
|
@ -230,6 +231,7 @@ class ReaderViewTest {
|
|||
}
|
||||
|
||||
@Test
|
||||
@Ignore("To be re-implemented in https://github.com/mozilla-mobile/fenix/issues/17971")
|
||||
fun verifyReaderViewAppearanceColorSchemeChange() {
|
||||
val readerViewPage =
|
||||
TestAssetHelper.getLoremIpsumAsset(mockWebServer)
|
||||
|
|
|
@ -233,7 +233,6 @@ class SmokeTest {
|
|||
}
|
||||
|
||||
@Test
|
||||
@Ignore("To be re-implemented in https://github.com/mozilla-mobile/fenix/issues/17798")
|
||||
// Verifies the Synced tabs menu opens from a tab's 3 dot menu
|
||||
fun openMainMenuSyncedTabsItemTest() {
|
||||
homeScreen {
|
||||
|
|
Loading…
Reference in New Issue
Block a user