Add /mediatypes/binaryBodyThreeContentTypes handler#339
Conversation
| }; | ||
| } | ||
| }); | ||
| app.post("/mediatypes/binaryBodyThreeContentTypes", "MediaTypesThreeContentTypes", (req) => { |
There was a problem hiding this comment.
i think this looks good, have you verified the test works locally? Also can you bump the package.json version number, we bump the version number for each test added to trigger a new release and allow people to pin to prior version numbers if necessary
There was a problem hiding this comment.
I have now 😅 I had to test the text/plain route manually because autorest.go doesn't generate code for it; turns out express needed parser config for it as well.
chlowell
force-pushed
the
chlowell/missing-mediatypes-route
branch
from
e1ee5b7 to
3b4f666
Compare
package.json
Outdated
| { | ||
| "name": "@microsoft.azure/autorest.testserver", | ||
| "version": "3.2.3", | ||
| "version": "3.3.0", |
There was a problem hiding this comment.
Need to update that one again. Just increase by a patch value when tweaking the tests. It is a bit annoying you have to keep making sure your pr version bump is always up to date with the latest version of package.json
Co-authored-by: Timothee Guerin <tiguerin@microsoft.com>
3 participants
#327 added the route to
swagger/media_types.jsonbut not a request handler for it.