Skip to content

Updated SignInOutRepeatedlyTest such that it verifies that the sessio…#198

Merged
caballoninja merged 3 commits intomasterfrom
Test/verifyUniqueSessionIdsOnLogins
Feb 7, 2025
Merged

Updated SignInOutRepeatedlyTest such that it verifies that the sessio…#198
caballoninja merged 3 commits intomasterfrom
Test/verifyUniqueSessionIdsOnLogins

Conversation

@BellringerQuinn
Copy link
Copy Markdown
Contributor

@BellringerQuinn BellringerQuinn commented Oct 17, 2024

…n ids each time are unique. In addition, exposed the session id in the SequenceAPI

This is done to verify that we won't be encountering sessionId mismatch errors due to re-using a session id on multiple auths

Docs Checklist

Please ensure you have addressed documentation updates if needed as part of this PR:

  • I have created a separate PR on the sequence docs repository for documentation updates: Link to docs PR
  • No documentation update is needed for this change.

…n ids each time are unique. In addition, exposed the session id in the SequenceAPI
Copy link
Copy Markdown
Contributor

@caballoninja caballoninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

andygruening
andygruening previously approved these changes Nov 8, 2024
Copy link
Copy Markdown
Contributor

@andygruening andygruening left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks tiptop!

@caballoninja caballoninja merged commit 98e27bd into master Feb 7, 2025
@caballoninja caballoninja deleted the Test/verifyUniqueSessionIdsOnLogins branch February 7, 2025 12:49
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.

3 participants