-
Notifications
You must be signed in to change notification settings - Fork 57
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.
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
add testcases #2822
add testcases #2822
Conversation
@@ -209,3 +209,76 @@ async def test_nested_proxy_resources_list_by_top_level_tracked_resource(client) | |||
assert result.name == "nested" | |||
assert result.type == "Azure.ResourceManager.Models.Resources/topLevelTrackedResources/top/nestedProxyResources" | |||
assert result.system_data.created_by == "AzureSDK" | |||
|
|||
|
|||
@pytest.mark.asyncio |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ChenxiJiang333 Please run pnpm change add
to add changelog.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@msyyc updated
) | ||
|
||
|
||
# @pytest.mark.asyncio |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we delete these commented out tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is not bad to keep it. After we fix the codegen, we could open the commented code directly.
thank you @ChenxiJiang333 for contributing! |
Add testcases for several cadl-ranch scenarios