-
Notifications
You must be signed in to change notification settings - Fork 77
Add test framework generation for mgmt sdk #1162
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
Conversation
As discussed offline, I will think if there is any alternative to downloading the test-utils package. Else, I will discuss with releasing it to npmjs to the package owner. Will update this issue soon. |
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.
Are we planning to add karma config to run tests against the browser?
Yes, but currently I want to make sure tests against node works |
hold on this PR as the public released @azure-tools/test-recorder package has some issues. |
test/smoke/generated/graphrbac-data-plane/src/operations/oAuth2PermissionGrantOperations.ts
Show resolved
Hide resolved
@joheredi @sarangan12 Could you help review the PR now ? the test recorder issue should have been resolved. |
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.
Looks good. But, I am seeing your comment that the test recorder has some issues. If that is resolved, this PR could be merged,
That issue has been resolved. |
No description provided.