#92 Fix UI tests

element-hq · Jun 26, 2022 · 30ce941 · 30ce941
1 parent 7704c31
commit 30ce941
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 12 deletions.
diff --git a/ElementX/Sources/Screens/RoomScreen/View/Style/TimelineStyle.swift b/ElementX/Sources/Screens/RoomScreen/View/Style/TimelineStyle.swift
@@ -24,17 +24,6 @@ enum TimelineStyle: String, CaseIterable {
     }
 }
 
-extension TimelineStyle: CustomStringConvertible {
-    var description: String {
-        switch self {
-        case .plain:
-            return ElementL10n.roomTimelineStylePlainLongDescription
-        case .bubbled:
-            return ElementL10n.roomTimelineStyleBubbledLongDescription
-        }
-    }
-}
-
 // MARK: - Environment
 
 private struct TimelineStyleKey: EnvironmentKey {

diff --git a/ElementX/Sources/Screens/Settings/View/Settings.swift b/ElementX/Sources/Screens/Settings/View/Settings.swift
@@ -99,12 +99,24 @@ struct Settings: View {
                             .tag(style)
                     }
                 }
+                .accessibilityIdentifier("timelineStylePicker")
             }
             .listRowBackground(rowBackgroundColor)
         }
     }
 }
 
+extension TimelineStyle: CustomStringConvertible {
+    var description: String {
+        switch self {
+        case .plain:
+            return ElementL10n.roomTimelineStylePlainLongDescription
+        case .bubbled:
+            return ElementL10n.roomTimelineStyleBubbledLongDescription
+        }
+    }
+}
+
 // MARK: - Previews
 
 struct Settings_Previews: PreviewProvider {

diff --git a/UITests/Sources/SettingsUITests.swift b/UITests/Sources/SettingsUITests.swift
@@ -26,7 +26,7 @@ class SettingsUITests: XCTestCase {
         XCTAssert(app.navigationBars["Settings"].exists)
         XCTAssert(app.buttons["reportBugButton"].exists)
         XCTAssertEqual(app.buttons["crashButton"].exists, BuildSettings.settingsCrashButtonVisible)
-        XCTAssertEqual(app.buttons["timelineStyleButton"].exists, BuildSettings.settingsShowTimelineStyle)
+        XCTAssertEqual(app.buttons["timelineStylePicker"].exists, BuildSettings.settingsShowTimelineStyle)
         XCTAssert(app.buttons["logoutButton"].exists)
     }