Skip to content

Synchronize to fix crashes in the user module #1554

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 7, 2025
Merged

Synchronize to fix crashes in the user module #1554

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
acd3904
Add concurrency test
nan-li Apr 28, 2025
3106f4d
Add lock to OSModelStore models
nan-li Apr 21, 2025
d7cb25c
When sub ID is nullified, don't make server call
nan-li Apr 22, 2025
468b955
Synchronize the User Manager start up process
nan-li Apr 28, 2025
c348b3c
nits: simplify code
nan-li Apr 28, 2025
5fc5ca9
Add some additional checks and guards
nan-li Apr 28, 2025
735e755
update some user tests that now fail
nan-li Apr 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Add concurrency test 
* Add test to start user manager from multiple threads
  • Loading branch information
@nan-li
nan-li committed Apr 29, 2025
commit acd39044efbc0a2db3e46f2f1163f8a8cb3bc294
9 changes: 9 additions & 0 deletions iOS_SDK/OneSignalSDK/OneSignalUserTests/UserConcurrencyTests.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -278,4 +278,13 @@ final class UserConcurrencyTests: XCTestCase {
identityModel.clearData()
}
}

func testConcurrentStart() throws {
DispatchQueue.concurrentPerform(iterations: 5_000) { _ in
OneSignalUserManagerImpl.sharedInstance.start()
}
DispatchQueue.concurrentPerform(iterations: 5_000) { _ in
_ = OneSignalUserManagerImpl.sharedInstance.user
}
}
}