diff options
| author | Andrew Bulhak <andrew.bulhak@mullvad.net> | 2024-12-05 17:27:49 +0100 |
|---|---|---|
| committer | Bug Magnet <marco.nikic@mullvad.net> | 2024-12-10 09:20:19 +0100 |
| commit | 4a4933c9353444e47f170222c585d8e5bf61552e (patch) | |
| tree | bb2d1e0ef4fdb2f71508216d73fd2f6c0753367c | |
| parent | e6227d775b4408103ef483123e454b6ee6cbc096 (diff) | |
| download | mullvadvpn-4a4933c9353444e47f170222c585d8e5bf61552e.tar.xz mullvadvpn-4a4933c9353444e47f170222c585d8e5bf61552e.zip | |
Replace circular .description definition with .asString
14 files changed, 27 insertions, 27 deletions
diff --git a/ios/MullvadVPN/Classes/AccessbilityIdentifier.swift b/ios/MullvadVPN/Classes/AccessbilityIdentifier.swift index bb86dd95c1..0970ed889b 100644 --- a/ios/MullvadVPN/Classes/AccessbilityIdentifier.swift +++ b/ios/MullvadVPN/Classes/AccessbilityIdentifier.swift @@ -214,14 +214,14 @@ public enum AccessibilityIdentifier: Equatable { case unknown } -extension AccessibilityIdentifier: CustomStringConvertible { - public var description: String { +extension AccessibilityIdentifier { + public var asString: String { "\(self)" } } extension UIAccessibilityIdentification { func setAccessibilityIdentifier(_ value: AccessibilityIdentifier?) { - accessibilityIdentifier = value.map(\.description) + accessibilityIdentifier = value.map(\.asString) } } diff --git a/ios/MullvadVPN/View controllers/AccountDeletion/AccountDeletionContentView.swift b/ios/MullvadVPN/View controllers/AccountDeletion/AccountDeletionContentView.swift index b6f4843fd4..26d87a787c 100644 --- a/ios/MullvadVPN/View controllers/AccountDeletion/AccountDeletionContentView.swift +++ b/ios/MullvadVPN/View controllers/AccountDeletion/AccountDeletionContentView.swift @@ -376,9 +376,9 @@ class AccountDeletionContentView: UIView { @objc private func didPress(button: AppButton) { switch button.accessibilityIdentifier { - case AccessibilityIdentifier.deleteButton.description: + case AccessibilityIdentifier.deleteButton.asString: delegate?.didTapDeleteButton(contentView: self, button: button) - case AccessibilityIdentifier.cancelButton.description: + case AccessibilityIdentifier.cancelButton.asString: delegate?.didTapCancelButton(contentView: self, button: button) default: return } diff --git a/ios/MullvadVPN/View controllers/CreationAccount/Completed/SetupAccountCompletedContentView.swift b/ios/MullvadVPN/View controllers/CreationAccount/Completed/SetupAccountCompletedContentView.swift index 1562d6ecf7..818053e66b 100644 --- a/ios/MullvadVPN/View controllers/CreationAccount/Completed/SetupAccountCompletedContentView.swift +++ b/ios/MullvadVPN/View controllers/CreationAccount/Completed/SetupAccountCompletedContentView.swift @@ -142,9 +142,9 @@ class SetupAccountCompletedContentView: UIView { @objc private func tapped(button: AppButton) { switch button.accessibilityIdentifier { - case AccessibilityIdentifier.learnAboutPrivacyButton.description: + case AccessibilityIdentifier.learnAboutPrivacyButton.asString: delegate?.didTapPrivacyButton(view: self, button: button) - case AccessibilityIdentifier.startUsingTheAppButton.description: + case AccessibilityIdentifier.startUsingTheAppButton.asString: delegate?.didTapStartingAppButton(view: self, button: button) default: return } diff --git a/ios/MullvadVPN/View controllers/CreationAccount/Welcome/WelcomeContentView.swift b/ios/MullvadVPN/View controllers/CreationAccount/Welcome/WelcomeContentView.swift index 425d3b3eb9..db3145ac68 100644 --- a/ios/MullvadVPN/View controllers/CreationAccount/Welcome/WelcomeContentView.swift +++ b/ios/MullvadVPN/View controllers/CreationAccount/Welcome/WelcomeContentView.swift @@ -251,11 +251,11 @@ final class WelcomeContentView: UIView { @objc private func tapped(button: AppButton) { switch button.accessibilityIdentifier { - case AccessibilityIdentifier.purchaseButton.description: + case AccessibilityIdentifier.purchaseButton.asString: delegate?.didTapPurchaseButton(welcomeContentView: self, button: button) - case AccessibilityIdentifier.redeemVoucherButton.description: + case AccessibilityIdentifier.redeemVoucherButton.asString: delegate?.didTapRedeemVoucherButton(welcomeContentView: self, button: button) - case AccessibilityIdentifier.infoButton.description: + case AccessibilityIdentifier.infoButton.asString: delegate?.didTapInfoButton(welcomeContentView: self, button: button) default: return } diff --git a/ios/MullvadVPN/View controllers/Settings/SelectableSettingsDetailsCell.swift b/ios/MullvadVPN/View controllers/Settings/SelectableSettingsDetailsCell.swift index 8869d7f0cf..239a62c3d7 100644 --- a/ios/MullvadVPN/View controllers/Settings/SelectableSettingsDetailsCell.swift +++ b/ios/MullvadVPN/View controllers/Settings/SelectableSettingsDetailsCell.swift @@ -22,7 +22,7 @@ class SelectableSettingsDetailsCell: SelectableSettingsCell { .withRenderingMode(.alwaysOriginal) .withTintColor(.white) actionButton.configuration = actionButtonConfiguration - actionButton.accessibilityIdentifier = .openPortSelectorMenuButton + actionButton.setAccessibilityIdentifier(.openPortSelectorMenuButton) actionButton.addTarget( self, diff --git a/ios/MullvadVPN/View controllers/VPNSettings/VPNSettingsCellFactory.swift b/ios/MullvadVPN/View controllers/VPNSettings/VPNSettingsCellFactory.swift index 3e7733a105..99688515f2 100644 --- a/ios/MullvadVPN/View controllers/VPNSettings/VPNSettingsCellFactory.swift +++ b/ios/MullvadVPN/View controllers/VPNSettings/VPNSettingsCellFactory.swift @@ -81,7 +81,7 @@ final class VPNSettingsCellFactory: CellFactoryProtocol { cell.titleLabel.text = portString // TODO: replace this with a tagged AccessibilityIdentifier - cell.accessibilityIdentifier = "\(item.accessibilityIdentifier.description) (\(portString))" + cell.accessibilityIdentifier = "\(item.accessibilityIdentifier.asString) (\(portString))" cell.applySubCellStyling() case .wireGuardCustomPort: diff --git a/ios/MullvadVPN/Views/CustomToggleStyle.swift b/ios/MullvadVPN/Views/CustomToggleStyle.swift index e968a921f3..b11449b9fc 100644 --- a/ios/MullvadVPN/Views/CustomToggleStyle.swift +++ b/ios/MullvadVPN/Views/CustomToggleStyle.swift @@ -56,7 +56,7 @@ struct CustomToggleStyle: ToggleStyle { ) .opacity(disabled ? 0.4 : 1) } - .accessibilityIdentifier(accessibilityId?.description ?? "") + .accessibilityIdentifier(accessibilityId?.asString ?? "") .onTapGesture { toggle(configuration) } diff --git a/ios/MullvadVPNUITests/Base/BaseUITestCase.swift b/ios/MullvadVPNUITests/Base/BaseUITestCase.swift index e8ed5e5042..347d90cfbc 100644 --- a/ios/MullvadVPNUITests/Base/BaseUITestCase.swift +++ b/ios/MullvadVPNUITests/Base/BaseUITestCase.swift @@ -248,7 +248,7 @@ class BaseUITestCase: XCTestCase { // Ensure changelog is no longer shown _ = app - .otherElements[AccessibilityIdentifier.changeLogAlert.description] + .otherElements[AccessibilityIdentifier.changeLogAlert.asString] .waitForNonExistence(timeout: Self.shortTimeout) } diff --git a/ios/MullvadVPNUITests/ConnectivityTests.swift b/ios/MullvadVPNUITests/ConnectivityTests.swift index 673a8dc2db..909cdce159 100644 --- a/ios/MullvadVPNUITests/ConnectivityTests.swift +++ b/ios/MullvadVPNUITests/ConnectivityTests.swift @@ -67,7 +67,7 @@ class ConnectivityTests: LoggedOutUITestCase { .tapSelectLocationButton() let filterCloseButtons = self.app.buttons - .matching(identifier: AccessibilityIdentifier.relayFilterChipCloseButton.description) + .matching(identifier: AccessibilityIdentifier.relayFilterChipCloseButton.asString) .allElementsBoundByIndex for filterCloseButton in filterCloseButtons where filterCloseButton.isHittable { diff --git a/ios/MullvadVPNUITests/Pages/AccountPage.swift b/ios/MullvadVPNUITests/Pages/AccountPage.swift index 9d8326983f..86622b672f 100644 --- a/ios/MullvadVPNUITests/Pages/AccountPage.swift +++ b/ios/MullvadVPNUITests/Pages/AccountPage.swift @@ -18,27 +18,27 @@ class AccountPage: Page { } @discardableResult func tapRedeemVoucherButton() -> Self { - app.buttons[AccessibilityIdentifier.redeemVoucherButton.description].tap() + app.buttons[AccessibilityIdentifier.redeemVoucherButton.asString].tap() return self } @discardableResult func tapAdd30DaysTimeButton() -> Self { - app.buttons[AccessibilityIdentifier.purchaseButton.description].tap() + app.buttons[AccessibilityIdentifier.purchaseButton.asString].tap() return self } @discardableResult func tapRestorePurchasesButton() -> Self { - app.buttons[AccessibilityIdentifier.restorePurchasesButton.description].tap() + app.buttons[AccessibilityIdentifier.restorePurchasesButton.asString].tap() return self } @discardableResult func tapLogOutButton() -> Self { - app.buttons[AccessibilityIdentifier.logoutButton.description].tap() + app.buttons[AccessibilityIdentifier.logoutButton.asString].tap() return self } @discardableResult func tapDeleteAccountButton() -> Self { - app.buttons[AccessibilityIdentifier.deleteButton.description].tap() + app.buttons[AccessibilityIdentifier.deleteButton.asString].tap() return self } diff --git a/ios/MullvadVPNUITests/Pages/DeviceManagementPage.swift b/ios/MullvadVPNUITests/Pages/DeviceManagementPage.swift index 9b24b1276c..881d402b63 100644 --- a/ios/MullvadVPNUITests/Pages/DeviceManagementPage.swift +++ b/ios/MullvadVPNUITests/Pages/DeviceManagementPage.swift @@ -19,7 +19,7 @@ class DeviceManagementPage: Page { @discardableResult func tapRemoveDeviceButton(cellIndex: Int) -> Self { app - .otherElements.matching(identifier: AccessibilityIdentifier.deviceCell.description).element(boundBy: cellIndex) + .otherElements.matching(identifier: AccessibilityIdentifier.deviceCell.asString).element(boundBy: cellIndex) .buttons[AccessibilityIdentifier.deviceCellRemoveButton] .tap() diff --git a/ios/MullvadVPNUITests/Pages/SelectLocationPage.swift b/ios/MullvadVPNUITests/Pages/SelectLocationPage.swift index 5c6498487d..25a82588ea 100644 --- a/ios/MullvadVPNUITests/Pages/SelectLocationPage.swift +++ b/ios/MullvadVPNUITests/Pages/SelectLocationPage.swift @@ -30,7 +30,7 @@ class SelectLocationPage: Page { } @discardableResult func tapCountryLocationCellExpandButton(withIndex: Int) -> Self { - let cell = app.cells.containing(.any, identifier: AccessibilityIdentifier.countryLocationCell.description) + let cell = app.cells.containing(.any, identifier: AccessibilityIdentifier.countryLocationCell.asString) .element(boundBy: withIndex) let expandButton = cell.buttons[AccessibilityIdentifier.expandButton] expandButton.tap() @@ -38,7 +38,7 @@ class SelectLocationPage: Page { } @discardableResult func tapCityLocationCellExpandButton(withIndex: Int) -> Self { - let cell = app.cells.containing(.any, identifier: AccessibilityIdentifier.cityLocationCell.description) + let cell = app.cells.containing(.any, identifier: AccessibilityIdentifier.cityLocationCell.asString) .element(boundBy: withIndex) let expandButton = cell.buttons[AccessibilityIdentifier.expandButton] expandButton.tap() @@ -46,7 +46,7 @@ class SelectLocationPage: Page { } @discardableResult func tapRelayLocationCell(withIndex: Int) -> Self { - let cell = app.cells.containing(.any, identifier: AccessibilityIdentifier.relayLocationCell.description) + let cell = app.cells.containing(.any, identifier: AccessibilityIdentifier.relayLocationCell.asString) .element(boundBy: withIndex) cell.tap() return self @@ -79,7 +79,7 @@ class SelectLocationPage: Page { _ = app.buttons[.openCustomListsMenuButton].waitForExistence(timeout: BaseUITestCase.shortTimeout) let customListEllipsisButtons = app.buttons - .matching(identifier: AccessibilityIdentifier.openCustomListsMenuButton.description).allElementsBoundByIndex + .matching(identifier: AccessibilityIdentifier.openCustomListsMenuButton.asString).allElementsBoundByIndex // This is a workaround for an issue we have with the ellipsis showing up multiple times in the accessibility hieararchy even though in the view hierarchy there is only one // Only the actually visual one is hittable, so only the visible button will be tapped diff --git a/ios/MullvadVPNUITests/Pages/VPNSettingsPage.swift b/ios/MullvadVPNUITests/Pages/VPNSettingsPage.swift index 83999ab7b1..37bb4e75f6 100644 --- a/ios/MullvadVPNUITests/Pages/VPNSettingsPage.swift +++ b/ios/MullvadVPNUITests/Pages/VPNSettingsPage.swift @@ -19,7 +19,7 @@ class VPNSettingsPage: Page { _ subButtonAccessibilityIdentifier: AccessibilityIdentifier ) -> XCUIElement { let tableView = app.tables[AccessibilityIdentifier.vpnSettingsTableView] - let matchingCells = tableView.otherElements[cellAccessiblityIdentifier.description] + let matchingCells = tableView.otherElements[cellAccessiblityIdentifier.asString] let expandButton = matchingCells.buttons[AccessibilityIdentifier.expandButton] let lastCell = tableView.cells.allElementsBoundByIndex.last! tableView.scrollDownToElement(element: lastCell) diff --git a/ios/MullvadVPNUITests/XCUIElementQuery+Extensions.swift b/ios/MullvadVPNUITests/XCUIElementQuery+Extensions.swift index 3b70e275c6..e4363bf20e 100644 --- a/ios/MullvadVPNUITests/XCUIElementQuery+Extensions.swift +++ b/ios/MullvadVPNUITests/XCUIElementQuery+Extensions.swift @@ -15,6 +15,6 @@ extension XCUIElementQuery { } subscript(key: AccessibilityIdentifier) -> XCUIElement { - self[key.description] + self[key.asString] } } |
