Skip to content
This repository was archived by the owner on Jan 21, 2025. It is now read-only.

fix explode query check of mockapi for routes test case #742

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

tadelesh
Copy link
Member

@tadelesh tadelesh commented Oct 9, 2024

Cadl Ranch Contribution Checklist:

  • I have written a scenario spec
  • I have meaningful @scenario names. Someone can look at the list of scenarios and understand what I'm covering.
  • I have written a mock API
  • I have used @scenarioDocs for extra scenario description and to tell people how to pass my mock api check.

fix: #731

Copy link

changeset-bot bot commented Oct 9, 2024

🦋 Changeset detected

Latest commit: 4a89d3d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@azure-tools/cadl-ranch-specs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@tadelesh tadelesh merged commit 23279d1 into main Oct 10, 2024
9 checks passed
@tadelesh tadelesh deleted the fix_routes_mockapi branch October 10, 2024 02:58
github-merge-queue bot pushed a commit to microsoft/typespec that referenced this pull request Oct 10, 2024
…s test case (#4680)

A fix has been made to the routes test case in the `cadl-ranch`
repository at Azure/cadl-ranch#742. This was
done to fix Azure/cadl-ranch#731.

This PR is to bring those changes to the `typespec` repository.
`test:e2e` script is working fine.

Please review and approve. Thanks
swatkatz pushed a commit to swatkatz/typespec that referenced this pull request Nov 5, 2024
…s test case (microsoft#4680)

A fix has been made to the routes test case in the `cadl-ranch`
repository at Azure/cadl-ranch#742. This was
done to fix Azure/cadl-ranch#731.

This PR is to bring those changes to the `typespec` repository.
`test:e2e` script is working fine.

Please review and approve. Thanks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Returned 400 for query explode=true case with path explode/array?param=a&param=b
2 participants