Skip to content

Conversation

@jhendrixMSFT
Copy link
Member

The track 1 modeler didn't correctly interpret the schema definition,
and now that we fixed the behavior in track 2 the codegen doesn't match
what the route expects.
Fix the schema definition to match the route and not break tests.

The track 1 modeler didn't correctly interpret the schema definition,
and now that we fixed the behavior in track 2 the codegen doesn't match
what the route expects.
Fix the schema definition to match the route and not break tests.
@jhendrixMSFT
Copy link
Member Author

jhendrixMSFT commented May 19, 2021

One sec, there is also something funky with body-complex.json. Let me take a look to see if it needs to be fixed too.

@jhendrixMSFT jhendrixMSFT merged commit 97f0e3a into Azure:master May 19, 2021
@jhendrixMSFT jhendrixMSFT deleted the ap_object branch May 19, 2021 20:07
iscai-msft added a commit to iscai-msft/autorest.testserver that referenced this pull request Aug 3, 2021
…nto add_media_types_no_accept_header

* 'main' of https://github.com/Azure/autorest.testserver:
  Change prepublish to prepack (Azure#297)
  Add LLC category (Azure#296)
  add test for grouped properties with reserved names (Azure#295)
  add lro parameterization with constant (Azure#293)
  LLC testing (Azure#292)
  Fix regression in types library (Azure#291)
  Fix PetAPObject schema definition to match route (Azure#290)
  fix any swagger (Azure#289)
  Fix version of wiremock (Azure#288)
  Added spec for Anything type.  (Azure#286)
  Fix: Invalid spec validation.json defining query param in path (Azure#283)
  add multiapi version no mixins (Azure#282)
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.

2 participants