-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[TT-14358] Fix mock response fixtures in 5.8.0 branch #6967
Conversation
A JIRA Issue ID is missing from your branch name, PR title and PR description! 🦄 Your branch: fix-mock-response-fixture Your PR title: Fix mock response fixtures Your PR description: DescriptionRelated IssueMotivation and ContextHow This Has Been TestedScreenshots (if appropriate)Types of changes
Checklist
If this is your first time contributing to this repository - welcome! Please refer to jira-lint to get started. Without the JIRA Issue ID in your branch name you would lose out on automatic updates to JIRA via SCM; some GitHub status checks might fail. Valid sample branch names:‣ feature/shiny-new-feature--mojo-10' |
API Changes no api changes detected |
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨No code suggestions found for the PR. |
A JIRA Issue ID is missing from your branch name, PR title and PR description! 🦄 Your branch: fix-mock-response-fixture Your PR title: Fix mock response fixtures Your PR description: ### **User description** DescriptionRelated IssueMotivation and ContextHow This Has Been TestedScreenshots (if appropriate)Types of changes
Checklist
PR Type
Description
Changes walkthrough 📝
If this is your first time contributing to this repository - welcome! Please refer to jira-lint to get started. Without the JIRA Issue ID in your branch name you would lose out on automatic updates to JIRA via SCM; some GitHub status checks might fail. Valid sample branch names:‣ feature/shiny-new-feature--mojo-10' |
|
User description
Description
Related Issue
https://tyktech.atlassian.net/browse/TT-14358
Motivation and Context
How This Has Been Tested
Screenshots (if appropriate)
Types of changes
Checklist
PR Type
Description
Removed legacy conflict markers in fixture file.
Replaced mock response entries with structured YAML arrays.
Updated multiple test fixture segments for proper API response simulation.
Changes walkthrough 📝
mock_response.yml
Update and standardize mock response fixtures.
apidef/oas/testdata/fixtures/mock_response.yml