Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Test roundtrip correctness of TypeSpec and swagger input for Mgmt #4821

Closed
12 tasks done
live1206 opened this issue Jun 13, 2024 · 1 comment
Closed
12 tasks done

Test roundtrip correctness of TypeSpec and swagger input for Mgmt #4821

live1206 opened this issue Jun 13, 2024 · 1 comment
Assignees
@live1206 @HarveyLink
Labels
Mgmt This issue is related to a management-plane library. v3 Version 3 of AutoRest C# generator.

Comments

@live1206
Copy link
Member

live1206 commented Jun 13, 2024 •
edited by ArthurMa1978
Loading

@live1206 live1206 added the v3 Version 3 of AutoRest C# generator. label Jun 13, 2024
@live1206
Copy link
Member Author

live1206 commented Jul 1, 2024 •
edited
Loading

@HarveyLink Please help with the testing here.

A few things before the testing:

  • Check if ArmResourcePatchAsync/ArmResourcePatchSync is used, update it to ArmCustomPatchAsync/ArmCustomPatchSync according to example
  • Remove list operation according to example
  • Add renaming with clientName, refer to informatica for example
  • If you have updated typsepc, run "tsp compile main.tsp" to generate the latest swagger

Actual testing steps:

  • Switch to a new branch in azure-sdk-for-net
  • Generate .NET SDK from latest swagger: if there is a ArmResourcePatchAsync, do this change
  • Check in the generated SDK from latest swagger for latter comparison
  • Add csharp option to tspconfig.yaml, refer to informatica for example
  • Add tsp-location.yaml to the SDK folder, refer to informatica for example and rename existing autorest.md to autorest.md.bak
  • run dotnet build /t:GenerateCode /p:LocalSpecRepo="path-to-your-local-spec" to generate SDK from typespec
  • summarize the difference

@live1206 live1206 assigned HarveyLink and unassigned live1206 and pshao25 Jul 1, 2024
@ArthurMa1978 ArthurMa1978 added the Mgmt This issue is related to a management-plane library. label Jul 5, 2024
@live1206 live1206 self-assigned this Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@live1206 live1206

@HarveyLink HarveyLink

Labels
Mgmt This issue is related to a management-plane library. v3 Version 3 of AutoRest C# generator.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@live1206 @ArthurMa1978 @HarveyLink @pshao25