-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Update Web Pub Sub test recordings #35848
Conversation
e50722f
to
1838458
Compare
API change check API changes are not detected in this pull request. |
1838458
to
5ab572d
Compare
@ParameterizedTest | ||
@MethodSource("getTokenOptions") | ||
@DoNotRecord |
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.
did we ad @DonotRecord here?
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.
Yeah, The recording is always empty because no network calls are made when using a connection string. Seemed wasteful to have 7 recordings that are empty being uploaded. ;/
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.
If this class isn't recording anything we should make this a normal unit tests and remove the extends TestProxyTestBase
.
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.
The other tests in this class do make network calls. :/
...-webpubsub/src/test/java/com/azure/messaging/webpubsub/WebPubSubServiceAsyncClientTests.java
Outdated
Show resolved
Hide resolved
...aging-webpubsub/src/test/java/com/azure/messaging/webpubsub/WebPubSubServiceClientTests.java
Show resolved
Hide resolved
5a35b92
to
11195fb
Compare
11195fb
to
0127dd6
Compare
...-webpubsub/src/test/java/com/azure/messaging/webpubsub/WebPubSubServiceAsyncClientTests.java
Outdated
Show resolved
Hide resolved
...-webpubsub/src/test/java/com/azure/messaging/webpubsub/WebPubSubServiceAsyncClientTests.java
Show resolved
Hide resolved
...-webpubsub/src/test/java/com/azure/messaging/webpubsub/WebPubSubServiceAsyncClientTests.java
Show resolved
Hide resolved
c1c0043
to
e7422a1
Compare
e7422a1
to
f4d7487
Compare
Description
Related: #33342
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines