summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorAndrew Bulhak <andrew.bulhak@mullvad.net>2024-12-05 17:27:49 +0100
committerBug Magnet <marco.nikic@mullvad.net>2024-12-10 09:20:19 +0100
commit4a4933c9353444e47f170222c585d8e5bf61552e (patch)
treebb2d1e0ef4fdb2f71508216d73fd2f6c0753367c
parente6227d775b4408103ef483123e454b6ee6cbc096 (diff)
downloadmullvadvpn-4a4933c9353444e47f170222c585d8e5bf61552e.tar.xz
mullvadvpn-4a4933c9353444e47f170222c585d8e5bf61552e.zip
Replace circular .description definition with .asString
-rw-r--r--ios/MullvadVPN/Classes/AccessbilityIdentifier.swift6
-rw-r--r--ios/MullvadVPN/View controllers/AccountDeletion/AccountDeletionContentView.swift4
-rw-r--r--ios/MullvadVPN/View controllers/CreationAccount/Completed/SetupAccountCompletedContentView.swift4
-rw-r--r--ios/MullvadVPN/View controllers/CreationAccount/Welcome/WelcomeContentView.swift6
-rw-r--r--ios/MullvadVPN/View controllers/Settings/SelectableSettingsDetailsCell.swift2
-rw-r--r--ios/MullvadVPN/View controllers/VPNSettings/VPNSettingsCellFactory.swift2
-rw-r--r--ios/MullvadVPN/Views/CustomToggleStyle.swift2
-rw-r--r--ios/MullvadVPNUITests/Base/BaseUITestCase.swift2
-rw-r--r--ios/MullvadVPNUITests/ConnectivityTests.swift2
-rw-r--r--ios/MullvadVPNUITests/Pages/AccountPage.swift10
-rw-r--r--ios/MullvadVPNUITests/Pages/DeviceManagementPage.swift2
-rw-r--r--ios/MullvadVPNUITests/Pages/SelectLocationPage.swift8
-rw-r--r--ios/MullvadVPNUITests/Pages/VPNSettingsPage.swift2
-rw-r--r--ios/MullvadVPNUITests/XCUIElementQuery+Extensions.swift2
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]
}
}