Skip to content
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.

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

create unique identifier for pended test wait continuation #15

Merged
merged 2 commits into from
Aug 7, 2024

Conversation

karlnaden
Copy link
Collaborator

Summary

Resolves issue 13 where the wait dialog for the temporary subscription placeholder was not session specific, meaning that concurrent sessions could interrupt each other. The wait dialog now generates a unique key that is used which will be unique to the session.

Testing Guidance

Create a PAS Server suite session and run group 1.3 Successful Pended Workflow using the Prior Authorization Support Reference Implementation preset. Verify that when the wait dialog appears,

  1. hovering over the "click here" link shows a url with a random "token" parameter at the end
  2. clicking on the "click here" link successfully continues the test.

Copy link
Contributor

@vanessuniq vanessuniq left a comment

Choose a reason for hiding this comment

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

Work as expected

Copy link
Contributor

@arscan arscan left a comment

Choose a reason for hiding this comment

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

Tried it out and works as advertised.

When I mucked with the token it didn't find the route as expected.

Subsequent test runs changed the token as expected.

@karlnaden karlnaden merged commit c938a5f into main Aug 7, 2024
3 checks passed
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.

resume for pended workflow is not session specific
3 participants