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.

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

Jump to bottom

PYIC-7785: Add API tests to complete a strategic app journey #2833

Closed
wants to merge 13 commits into from
Closed

PYIC-7785: Add API tests to complete a strategic app journey #2833

wants to merge 13 commits into from

Conversation

MikeCollingwood
Copy link
Contributor

@MikeCollingwood MikeCollingwood commented Dec 30, 2024 •
edited by jira bot
Loading

Proposed changes

What changed

  • Add steps to complete a strategic app journey in API tests

Why did it change

  • To improve API tests cover of the new async DCMAW feature

Issue tracking

@MikeCollingwood MikeCollingwood mentioned this pull request Dec 30, 2024
Merged
@MikeCollingwood
PYIC-7785: markups:
6b60910 
- Add after hook to cleanup DCMAW stub session state
- Improve step namings
- Add comments clarifying breaks in API tests
- Make callback step fetch oauthState if not already retrieved
@MikeCollingwood
PYIC-7785: Use After hook to cleanup Async DCMAW stub state
fba4601
@MikeCollingwood
PYIC-7785: Fix CRI step types
2649f7f
@MikeCollingwood MikeCollingwood marked this pull request as ready for review January 10, 2025 16:26
@MikeCollingwood MikeCollingwood requested review from a team as code owners January 10, 2025 16:26
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@DanCorderIPV
Copy link
Contributor

Closing as github seems to have got confused about what SHA to use when uploading pacts.
Will open a new identical PR instead.

@DanCorderIPV DanCorderIPV mentioned this pull request Jan 14, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MikeCollingwood @DanCorderIPV