Skip to content

Commit 82a5066

Browse files
Fix for failing nightly iOS end to end tests
1 parent f36bd2e commit 82a5066

File tree

6 files changed

+19
-37
lines changed

6 files changed

+19
-37
lines changed

ios/Configurations/UITests.xcconfig.template

-2
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ IOS_DEVICE_PIN_CODE =
77
TEST_DEVICE_IDENTIFIER_UUID =
88

99
// Mullvad accounts used by UI tests
10-
NO_TIME_ACCOUNT_NUMBER[config=Debug] =
11-
NO_TIME_ACCOUNT_NUMBER[config=Staging] =
1210
HAS_TIME_ACCOUNT_NUMBER[config=Debug] =
1311
HAS_TIME_ACCOUNT_NUMBER[config=Staging] =
1412
FIVE_WIREGUARD_KEYS_ACCOUNT_NUMBER =

ios/MullvadVPN.xcodeproj/project.pbxproj

-4
Original file line numberDiff line numberDiff line change
@@ -637,7 +637,6 @@
637637
8556EB562B9B0AC500D26DD4 /* RevokedDevicePage.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8556EB552B9B0AC500D26DD4 /* RevokedDevicePage.swift */; };
638638
855D9F5B2B63E56B00D7C64D /* ProblemReportPage.swift in Sources */ = {isa = PBXBuildFile; fileRef = 855D9F5A2B63E56B00D7C64D /* ProblemReportPage.swift */; };
639639
8587A05D2B84D43100152938 /* ChangeLogAlert.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8587A05C2B84D43100152938 /* ChangeLogAlert.swift */; };
640-
8590896C2B61763B003AF5F5 /* LoggedInWithoutTimeUITestCase.swift in Sources */ = {isa = PBXBuildFile; fileRef = 859089682B61763B003AF5F5 /* LoggedInWithoutTimeUITestCase.swift */; };
641640
8590896F2B61763B003AF5F5 /* LoggedOutUITestCase.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8590896B2B61763B003AF5F5 /* LoggedOutUITestCase.swift */; };
642641
85A42B862BB1D627007BABF7 /* XCUIElement+Extensions.swift in Sources */ = {isa = PBXBuildFile; fileRef = 85A42B852BB1D627007BABF7 /* XCUIElement+Extensions.swift */; };
643642
85B267612B849ADB0098E3CD /* mullvad-api.h in Headers */ = {isa = PBXBuildFile; fileRef = 85B267602B849ADB0098E3CD /* mullvad-api.h */; };
@@ -1896,7 +1895,6 @@
18961895
8556EB552B9B0AC500D26DD4 /* RevokedDevicePage.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = RevokedDevicePage.swift; sourceTree = "<group>"; };
18971896
855D9F5A2B63E56B00D7C64D /* ProblemReportPage.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ProblemReportPage.swift; sourceTree = "<group>"; };
18981897
8587A05C2B84D43100152938 /* ChangeLogAlert.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ChangeLogAlert.swift; sourceTree = "<group>"; };
1899-
859089682B61763B003AF5F5 /* LoggedInWithoutTimeUITestCase.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = LoggedInWithoutTimeUITestCase.swift; sourceTree = "<group>"; };
19001898
859089692B61763B003AF5F5 /* LoggedInWithTimeUITestCase.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = LoggedInWithTimeUITestCase.swift; sourceTree = "<group>"; };
19011899
8590896A2B61763B003AF5F5 /* BaseUITestCase.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = BaseUITestCase.swift; sourceTree = "<group>"; };
19021900
8590896B2B61763B003AF5F5 /* LoggedOutUITestCase.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = LoggedOutUITestCase.swift; sourceTree = "<group>"; };
@@ -3603,7 +3601,6 @@
36033601
isa = PBXGroup;
36043602
children = (
36053603
8590896A2B61763B003AF5F5 /* BaseUITestCase.swift */,
3606-
859089682B61763B003AF5F5 /* LoggedInWithoutTimeUITestCase.swift */,
36073604
859089692B61763B003AF5F5 /* LoggedInWithTimeUITestCase.swift */,
36083605
8590896B2B61763B003AF5F5 /* LoggedOutUITestCase.swift */,
36093606
8518F6372B60157E009EB113 /* LoggedInWithoutTimeUITestCase.swift */,
@@ -5638,7 +5635,6 @@
56385635
852969362B4E9724007EAD4C /* AccessbilityIdentifier.swift in Sources */,
56395636
85E3BDE52B70E18C00FA71FD /* Networking.swift in Sources */,
56405637
85C7A2E92B89024B00035D5A /* SettingsTests.swift in Sources */,
5641-
8590896C2B61763B003AF5F5 /* LoggedInWithoutTimeUITestCase.swift in Sources */,
56425638
8590896F2B61763B003AF5F5 /* LoggedOutUITestCase.swift in Sources */,
56435639
85557B202B5FBBD700795FE1 /* AccountPage.swift in Sources */,
56445640
852969352B4E9270007EAD4C /* LoginPage.swift in Sources */,

ios/MullvadVPNUITests/AccountTests.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ class AccountTests: LoggedOutUITestCase {
5858
func testLogin() throws {
5959
LoginPage(app)
6060
.tapAccountNumberTextField()
61-
.enterText(self.noTimeAccountNumber)
61+
.enterText(self.hasTimeAccountNumber)
6262
.tapAccountNumberSubmitButton()
6363
.verifySuccessIconShown()
6464
.verifyDeviceLabelShown()

ios/MullvadVPNUITests/RelayTests.swift

+18-2
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,22 @@ class RelayTests: LoggedInWithTimeUITestCase {
2727
}
2828

2929
func testAdBlockingViaDNS() throws {
30+
// Undo enabling block ads in teardown
31+
addTeardownBlock {
32+
HeaderBar(self.app)
33+
.tapSettingsButton()
34+
35+
SettingsPage(self.app)
36+
.tapVPNSettingsCell()
37+
38+
VPNSettingsPage(self.app)
39+
.tapDNSSettingsCell()
40+
41+
DNSSettingsPage(self.app)
42+
.tapDNSContentBlockersHeaderExpandButton()
43+
.tapBlockAdsSwitch()
44+
}
45+
3046
HeaderBar(app)
3147
.tapSettingsButton()
3248

@@ -44,7 +60,7 @@ class RelayTests: LoggedInWithTimeUITestCase {
4460
TunnelControlPage(app)
4561
.tapSecureConnectionButton()
4662

47-
allowAddVPNConfigurations() // Allow adding VPN configurations iOS permission
63+
allowAddVPNConfigurationsIfAsked() // Allow adding VPN configurations iOS permission
4864

4965
TunnelControlPage(app)
5066
.waitForSecureConnectionLabel()
@@ -205,7 +221,7 @@ class RelayTests: LoggedInWithTimeUITestCase {
205221
TunnelControlPage(app)
206222
.tapSecureConnectionButton()
207223

208-
allowAddVPNConfigurations()
224+
allowAddVPNConfigurationsIfAsked()
209225

210226
HeaderBar(app)
211227
.tapSettingsButton()

ios/MullvadVPNUITests/Test base classes/BaseUITestCase.swift

-2
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@ class BaseUITestCase: XCTestCase {
1717
// swiftlint:disable force_cast
1818
let displayName = Bundle(for: BaseUITestCase.self)
1919
.infoDictionary?["DisplayName"] as! String
20-
let noTimeAccountNumber = Bundle(for: BaseUITestCase.self)
21-
.infoDictionary?["NoTimeAccountNumber"] as! String
2220
let hasTimeAccountNumber = Bundle(for: BaseUITestCase.self)
2321
.infoDictionary?["HasTimeAccountNumber"] as! String
2422
let fiveWireGuardKeysAccountNumber = Bundle(for: BaseUITestCase.self)

ios/MullvadVPNUITests/Test base classes/LoggedInWithoutTimeUITestCase.swift

-26
This file was deleted.

0 commit comments

Comments
 (0)