Skip to content

Commit 0366867

Browse files
committed
Merge upstream dev into fix/add-ios17-authflowtester
2 parents 322ceb6 + daf0b35 commit 0366867

3 files changed

Lines changed: 18 additions & 6 deletions

File tree

.github/workflows/pr.yaml

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,9 +138,20 @@ jobs:
138138

139139
ui-tests-pr:
140140
needs: [test-orchestrator]
141+
strategy:
142+
fail-fast: false
143+
matrix:
144+
ios: [^26, ^18]
145+
include:
146+
- ios: ^26
147+
xcode: ^26
148+
- ios: ^18
149+
xcode: ^16
141150
uses: ./.github/workflows/reusable-ui-test-workflow.yaml
142151
with:
143152
is_pr: true
153+
ios: ${{ matrix.ios }}
154+
xcode: ${{ matrix.xcode }}
144155
pr_test: "AuthFlowTesterUITests/LegacyLoginTests/testCAOpaque_DefaultScopes_WebServerFlow"
145156
short_timeout: "2"
146157
long_timeout: "7"

libs/SalesforceSDKCore/SalesforceSDKCore/Classes/Identity/SFIdentityCoordinator.m

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -135,11 +135,6 @@ - (NSString *)validateParameters
135135
if ([invalidParameters length] > 0) [invalidParameters appendString:@", "];
136136
[invalidParameters appendString:@"identity URL"];
137137
}
138-
SFScopeParser *scopeParser = [[SFScopeParser alloc] initWithScopes:self.credentials.scopes];
139-
if (![scopeParser hasIdentityScope]) {
140-
if ([invalidParameters length] > 0) [invalidParameters appendString:@", "];
141-
[invalidParameters appendString:@"identity scope"];
142-
}
143138

144139
NSString *invalidParametersError = nil;
145140
if ([invalidParameters length] > 0) {

native/SampleApps/AuthFlowTester/AuthFlowTesterUITests/PageObjects/LoginPageObject.swift

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,13 @@ class LoginPageObject {
221221
}
222222

223223
private func advancedAuthCloseButton() -> XCUIElement {
224-
return app.otherElements["TopBrowserBar"].buttons["Close"]
224+
let topBar = app.otherElements["TopBrowserBar"]
225+
let closeButton = topBar.buttons["Close"]
226+
if closeButton.exists {
227+
return closeButton
228+
}
229+
// Earlier iOS versions use "Cancel"
230+
return topBar.buttons["Cancel"]
225231
}
226232

227233
// MARK: - Actions

0 commit comments

Comments
 (0)