Azure / cadl-ranch

Cadl Scenarios for client generations
https://azure.github.io/cadl-ranch/
MIT License
8 stars 33 forks source link

specs, add test for encode integer as string #656

Closed weidongxu-microsoft closed 3 months ago

weidongxu-microsoft commented 4 months ago

Fix https://github.com/Azure/cadl-ranch/issues/634

Not include case of "query" and "header", because they are always string on wire.

"body property" tests on

  1. safeint (int64) as required property
  2. uint32 as optional property

Cadl Ranch Contribution Checklist:

changeset-bot[bot] commented 4 months ago

🦋 Changeset detected

Latest commit: d9d7949906862bb2e98dd72bea68f76bd387a467

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

weidongxu-microsoft commented 3 months ago

shall we change the folder to numeric? the previous tests for encode is named by original type name.

Make sense. Moved to encode/numeric folder (and namespace).