-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[MOB-10402] Fix Tests #876
Merged
Merged
Changes from all commits
Commits
Show all changes
141 commits
Select commit
Hold shift + click to select a range
d2620a3
[MOB-10402] Add test reports
sumeruchat 69f5241
[MOB-10402] Add test reports
sumeruchat 7b8e280
[MOB-10402] Add test reports
sumeruchat b706ad3
[MOB-10402] Fix
sumeruchat 4fc091c
[MOB-10402] Fix
sumeruchat 0cd2f26
[MOB-10402] Fix
sumeruchat f42781d
[MOB-10402] Fix
sumeruchat 7ed554b
[MOB-10402] Fix
sumeruchat 568ebe9
[MOB-10402] fix
sumeruchat 7579563
Merge branch 'master' into feature/MOB-10402-add-tests
sumeruchat 22343ea
[MOB-9233] Some work on in-app json only
sumeruchat d9c3230
[MOB-9233] add a test
sumeruchat 6a97003
[MOB-9233] Fixes
sumeruchat 474b3ce
[MOB-9233] Fixes
sumeruchat a2c02b4
[MOB-9233] Fixes
sumeruchat a3662cc
[MOB-9233] Fixes
sumeruchat 54af23a
[MOB-9233] Fixes
sumeruchat a9ba3bd
[MOB-9233] Fixes
sumeruchat 2a5b58c
[MOB-9233] Fixes
sumeruchat c75ddc6
[MOB-9233] Fixes
sumeruchat e1c66c3
[MOB-9233] Fixes
sumeruchat 1085d80
[MOB-9233] Fixes
sumeruchat fcc2ca3
[MOB-9233] Fixes
sumeruchat 60bf79b
[MOB-9233] Fixes
sumeruchat f181cd8
[MOB-9233] Fixes
sumeruchat 9e6df49
[MOB-9233] Fix tests
sumeruchat 49ad2db
[MOB-9233] Fix tests
sumeruchat 5d98e00
[MOB-9233] Fix tests
sumeruchat 0975447
[MOB-9233] Fix tests
sumeruchat e896aa6
[MOB-9233] Fix tests
sumeruchat 49cb033
[MOB-9233] Fix tests
sumeruchat 806a820
[MOB-9233] Fix tests
sumeruchat 72777bd
[MOB-9233] Fix tests
sumeruchat 063b8a0
[MOB-9233] Fix tests
sumeruchat dc8b1ac
[MOB-9233] Some fixes
sumeruchat e7b6346
[MOB-9233] Fixes
sumeruchat 97723f6
[MOB-9233] Fixes
sumeruchat 04d9eaf
[MOB-9233] Fixes
sumeruchat 3d031bf
[MOB-9233] Fixes
sumeruchat a5cb8eb
[MOB-9233] Fixes
sumeruchat 21f1245
[MOB-9233] Fixes
sumeruchat 10fed83
[MOB-9233] Fixes
sumeruchat dfa49c6
[MOB-9233] Fixes
sumeruchat ae48057
[MOB-9233] Fixes
sumeruchat 6f924fd
[MOB-9233] Fixes
sumeruchat d36f12a
[MOB-9233] Fixes
sumeruchat b4693af
[MOB-9233] Fixes
sumeruchat 2b5ea6d
[MOB-9233] Fixes
sumeruchat 9fb3461
[MOB-9233] Fixes
sumeruchat 2ed9d1f
[MOB-9233] Fixes
sumeruchat a8f89bb
[MOB-9233] Fixes
sumeruchat dea9f43
[MOB-9233] Fixes
sumeruchat 55d21f9
[MOB-9233] Fixes
sumeruchat ae81982
[MOB-9233] Fixes
sumeruchat 697a689
[MOB-9233] Fixes
sumeruchat 4ec5477
[MOB-9233] Fixes
sumeruchat e708ad5
[MOB-9233] Fixes
sumeruchat a5d578f
[MOB-9233] Fixes
sumeruchat e882c40
[MOB-9233] Fixes
sumeruchat ef3984c
[MOB-9233] Fixes
sumeruchat 090ef0d
[MOB-9233] Fixes
sumeruchat 537f1c3
[MOB-9233] Fixes
sumeruchat a9d5f1a
[MOB-9233] Fixes
sumeruchat a670415
[MOB-9233] Fixes
sumeruchat 7076c5a
[MOB-9233] Fixes
sumeruchat 9669e1c
[MOB-9233] Fixes
sumeruchat a1eac51
[MOB-9233] Fixes
sumeruchat 27e27e1
[MOB-9233] Fixes
sumeruchat 30246b6
[MOB-9233] Updated tests according to the new discussion
sumeruchat 7673242
[MOB-9233] Updated code according to the new discussion
sumeruchat ea1f39d
[MOB-9233] Updated code according to the new discussion
sumeruchat 1761de8
[MOB-9233] Updated code according to the new discussion
sumeruchat 55e5259
[MOB-9233] Updated code according to the new discussion
sumeruchat 7b665de
[MOB-9233] Updated code according to the new discussion
sumeruchat e47139e
[MOB-9233] Updated code according to the new discussion
sumeruchat b152aaa
[MOB-9233] Updated code according to the new discussion
sumeruchat 897fe2a
[MOB-9233] Updated code according to the new discussion
sumeruchat 9760232
[MOB-9233] Updated code according to the new discussion
sumeruchat ad83f13
[MOB-9233] Updated code according to the new discussion
sumeruchat f2cc52c
[MOB-9233] Updated code according to the new discussion
sumeruchat 01d21f2
[MOB-9233] Updated code according to the new discussion
sumeruchat 80e9797
[MOB-9233] Updated code according to the new discussion
sumeruchat c0387c6
[MOB-9233] Updated code according to the new discussion
sumeruchat 23b0ce8
[MOB-9233] Updated code according to the new discussion
sumeruchat c58b3e5
[MOB-9233] Updated code according to the new discussion
sumeruchat f3b7167
[MOB-9233] Updated code according to the new discussion
sumeruchat 8d06f34
[MOB-9233] Updated code according to the new discussion
sumeruchat 1d03e8d
[MOB-10364] Update according to new discussiom
sumeruchat c30a1c7
[MOB-10364] Update according to new discussiom
sumeruchat 886e650
[MOB-10364] Update according to new discussiom
sumeruchat d667aea
[MOB-10364] Update according to new discussiom
sumeruchat 3557212
[MOB-10364] Update according to new discussiom
sumeruchat 4a13c59
Fixes
sumeruchat 70dd26f
[MOB-9233] Updated code according to the new discussion
sumeruchat 67cb3fd
[MOB-9233] Updated code according to the new discussion
sumeruchat cc57564
[MOB-9233] Updated code according to the new discussion
sumeruchat 104478f
[MOB-9233] Updated code according to the new discussion
sumeruchat 5c9f46f
Update swift-sdk/Internal/in-app/InAppMessageParser.swift
sumeruchat 6a480df
Update swift-sdk/Internal/in-app/InAppManager.swift
sumeruchat 337ed90
[MOB-10364] Update according to new discussiom
sumeruchat f529335
Update iOS Simulator destination in run_test.sh for compatibility
sumeruchat 52e88ed
[MOB-10364] Update according to new discussiom
sumeruchat 5f6de08
[MOB-10364] Update according to new discussiom
sumeruchat 25c9ca0
[MOB-9233] Fix tests for json only in app messages (#883)
sumeruchat eab7521
[MOB-9446] Enhance push notification state tracking in SDKs (#881)
evantk91 a1ff081
[MOB-10605] prepares version 6.5.9 release (#885)
evantk91 78a58ae
[MOB-10951] Add mobile framework info to register token request (#884)
sumeruchat e1496f5
[MOB-10626] removes disable push call (#887)
evantk91 d520a4f
[MOB-10652] prepares for version 6.5.10 release (#888)
evantk91 0523ec2
[MOB-10592] Make IterableAPIMobileFrameworkInfo properties and initia…
sumeruchat b4aca70
[MOB-10709] prepares for version 6.5.11 release (#890)
evantk91 aaaa176
MOB-10770 Update iOS SDK initialization checks (#891)
joaodordio 7ec6b46
[MOB-10402] To generate an improved commit message, I need more conte…
sumeruchat cac6bfb
[MOB-10402] Update iOS simulator in CI workflow
sumeruchat 3672871
[MOB-10402] Add OS version to iOS Simulator config
sumeruchat aef1eb8
[MOB-10402] Select iPhone simulator dynamically
sumeruchat c953838
[MOB-10402] Fix simulator selection logic in CI workflow
sumeruchat cda03e8
[MOB-10402] Fix regex for iPhone model parsing
sumeruchat 4c94ce0
[MOB-10402] Update CI to use macos-latest runner
sumeruchat 3c5b19d
[MOB-10402] Fix device name regex in build script
sumeruchat 00b79d1
[MOB-10402] Add swift-sdk.xctestplan to project files
sumeruchat 6c61743
[MOB-10402] Add endpoint test plan to Xcode project
sumeruchat 1340898
[MOB-10402] Update macOS version in CI workflow
sumeruchat 575e19e
[MOB-10402] Update CI to use macOS-latest and iPhone 16 Pro
sumeruchat 76b5ba6
[MOB-10402] Update CI to run on macOS 15
sumeruchat 74f146e
[MOB-10402] Rename test plan to swift-sdk.xctestplan
sumeruchat 2a6b6ff
[MOB-10402] Merge master into feature/MOB-10402-add-tests
sumeruchat b7e0b04
[MOB-10402] Remove unused test plans from project files
sumeruchat d0c0098
[MOB-10402] Remove obsolete xctestplan files
sumeruchat c2a253a
[MOB-10402] Update e2e workflow to macOS 15
sumeruchat de5318d
[MOB-10402] Fix build output formatting with xcpretty
sumeruchat 5ed6479
[MOB-10402] Add result bundle path to xcodebuild command
sumeruchat e1d9fa7
[MOB-10402] Add test report generation to e2e workflow
sumeruchat f115f78
[MOB-10402] Remove redundant test report steps
sumeruchat 4603070
[MOB-10402] Add swift-sdk.xctestplan to project config
sumeruchat e6ca2db
[MOB-10402] Add endpoint-tests.xctestplan to project
sumeruchat 2123eb7
[MOB-10402] Fix xctestplan paths in project config
sumeruchat c42691e
[MOB-10402] Fix test plan path in xcscheme files
sumeruchat 73af51f
MOB-10998 Actually Fix Tests (#904)
sumeruchat 18eeca2
[MOB-10402] Add skipped tests to xctestplan
sumeruchat 68d68a6
Merge branch 'master' into feature/MOB-10402-add-tests
evantk91 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -150,19 +150,19 @@ class AuthManager: IterableAuthManagerProtocol { | |
|
||
pendingAuth = false | ||
|
||
// Set the new token first | ||
authToken = retrievedAuthToken | ||
storeAuthToken() | ||
|
||
if retrievedAuthToken != nil { | ||
let isRefreshQueued = queueAuthTokenExpirationRefresh(retrievedAuthToken, onSuccess: onSuccess) | ||
if !isRefreshQueued { | ||
onSuccess?(authToken) | ||
onSuccess?(retrievedAuthToken) // Use retrievedAuthToken instead of authToken | ||
} | ||
} else { | ||
handleAuthFailure(failedAuthToken: nil, reason: .authTokenNull) | ||
scheduleAuthTokenRefreshTimer(interval: getNextRetryInterval(), successCallback: onSuccess) | ||
} | ||
|
||
authToken = retrievedAuthToken | ||
|
||
storeAuthToken() | ||
} | ||
|
||
func handleAuthFailure(failedAuthToken: String?, reason: AuthFailureReason) { | ||
|
@@ -180,13 +180,13 @@ class AuthManager: IterableAuthManagerProtocol { | |
/// schedule a default timer of 10 seconds if we fall into this case | ||
scheduleAuthTokenRefreshTimer(interval: getNextRetryInterval(), successCallback: onSuccess) | ||
|
||
return true | ||
return false // Return false since we couldn't queue a valid refresh | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nice catch! |
||
} | ||
|
||
let timeIntervalToRefresh = TimeInterval(expirationDate) - dateProvider.currentDate.timeIntervalSince1970 - expirationRefreshPeriod | ||
if timeIntervalToRefresh > 0 { | ||
scheduleAuthTokenRefreshTimer(interval: timeIntervalToRefresh, isScheduledRefresh: true, successCallback: onSuccess) | ||
return true | ||
return true // Only return true when we successfully queue a refresh | ||
} | ||
return false | ||
} | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
{ | ||
"configurations" : [ | ||
{ | ||
"id" : "D4401795-4D16-4709-9480-E9D329384AB3", | ||
"name" : "Configuration 1", | ||
"options" : { | ||
|
||
} | ||
} | ||
], | ||
"defaultOptions" : { | ||
"codeCoverage" : false, | ||
"targetForVariableExpansion" : { | ||
"containerPath" : "container:swift-sdk.xcodeproj", | ||
"identifier" : "ACF560D220E443BF000AAC23", | ||
"name" : "host-app" | ||
} | ||
}, | ||
"testTargets" : [ | ||
{ | ||
"target" : { | ||
"containerPath" : "container:swift-sdk.xcodeproj", | ||
"identifier" : "AC28480624AA44C600C1FC7F", | ||
"name" : "endpoint-tests" | ||
} | ||
} | ||
], | ||
"version" : 1 | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This actually makes sense. Might have to match this one in Android later too if thats not happening..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree. We should add to Android.