Skip to content

fix: [SDK-4766] stabilize parallel user tests - #1718

Closed
onesignal-deploy wants to merge 1 commit into
mainfrom
fadi/sdk-4766
Closed

fix: [SDK-4766] stabilize parallel user tests#1718
onesignal-deploy wants to merge 1 commit into
mainfrom
fadi/sdk-4766

Conversation

@onesignal-deploy

Copy link
Copy Markdown
Collaborator

Description

One Line Summary

Reset resilient identifier storage and correct test setup so parallel SDK tests no longer inherit stale user state.

Details

Motivation

The reduced CI test plan intermittently reused file-backed identifiers between tests, causing IAM, Live Activities, and user tests to observe subscriptions or users created by earlier tests.

Scope

This changes test mocks and test setup only. Production SDK behavior and public APIs are unchanged.

Testing

Unit testing

  • Full UnitTestApp_TestPlan_Reduced test plan passes.
  • Original failing test set passes for five iterations with four parallel workers.
  • Focused user tests pass for ten iterations.
  • Focused Live Activities test passes for ten iterations.

Manual testing

Not applicable because this change only affects automated test isolation.

Affected code checklist

  • Notifications
    • Display
    • Open
    • Push Processing
    • Confirm Deliveries
  • Outcomes
  • Sessions
  • In-App Messaging
  • REST API requests
  • Public API changes

Checklist

Overview

  • I have filled out all REQUIRED sections above
  • PR does one thing
  • No public API changes

Testing

  • I have included test coverage for these changes, or explained why it is not needed
  • All automated tests pass, or I explained why that is not possible
  • Manual device testing is not applicable and is explained above

Final pass

  • Code is as readable as possible
  • I have reviewed this PR myself

Made with Cursor

@fadi-george fadi-george mentioned this pull request Aug 19, 2026
18 tasks
@fadi-george
fadi-george deleted the fadi/sdk-4766 branch August 19, 2026 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants