Skip to content

Adds blockUnmockedRequests property of mock response plugin config schema#1166

Merged
waldekmastykarz merged 2 commits intodotnet:mainfrom
bartizan:update-mock-response-plugin-schema
May 7, 2025
Merged

Adds blockUnmockedRequests property of mock response plugin config schema#1166
waldekmastykarz merged 2 commits intodotnet:mainfrom
bartizan:update-mock-response-plugin-schema

Conversation

@bartizan
Copy link
Copy Markdown
Contributor

@bartizan bartizan commented May 6, 2025

@bartizan bartizan marked this pull request as ready for review May 6, 2025 14:38
@bartizan bartizan requested a review from a team as a code owner May 6, 2025 14:38
Copy link
Copy Markdown
Collaborator

@waldekmastykarz waldekmastykarz left a comment

Choose a reason for hiding this comment

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

Could you please add it to all schemas? This feature has been around since v0.12. Thank you!

@waldekmastykarz
Copy link
Copy Markdown
Collaborator

I'll mark the PR as draft until you update the PR so that we don't merge it accidentally

@waldekmastykarz waldekmastykarz marked this pull request as draft May 7, 2025 06:28
@bartizan
Copy link
Copy Markdown
Contributor Author

bartizan commented May 7, 2025

The fix is applied to v0.25 and later.
For v0.24 and earlier there were no "Dev Proxy MockResponsePlugin config schema".
It looks like since v0.25 there was a breaking change and schema mockresponseplugin.schema.json has been renamed to mockresponseplugin.mocksfile.schema.json

@waldekmastykarz
Copy link
Copy Markdown
Collaborator

Of course! You're right! For older versions, we only had schemas for the separate files and not config sections. My bad. Thanks for double checking.

@waldekmastykarz waldekmastykarz marked this pull request as ready for review May 7, 2025 08:27
@waldekmastykarz waldekmastykarz added the pr-bugfix Fixes a bug label May 7, 2025
@waldekmastykarz waldekmastykarz merged commit 5001dd5 into dotnet:main May 7, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants