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.

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

Fix SDK's EndpointDiscovery and S3 Upload Manager unstable tests #2286

Merged
merged 2 commits into from Nov 19, 2018

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented Nov 18, 2018

Fixes EndpointDiscovery mock client's test to consistently pass. The
previous test would periodically fail based on timing.

Fixes S3 Upload Manager's unstable unexpected EOF test to be stable.

@jasdel jasdel requested a review from xibz November 18, 2018 00:31
@jasdel jasdel self-assigned this Nov 18, 2018
Fixes the EndpointDiscovery mock client's test to consistently pass. The
previous test would periodically fail based on timing.
@jasdel jasdel changed the title codegentest: Fixup EndpointDiscovery model test Fix SDK's EndpointDiscovery and S3 Upload Manager unstable tests Nov 19, 2018
@jasdel jasdel merged commit c03c10f into aws:master Nov 19, 2018
@jasdel jasdel deleted the fixup/EndpointDescTest branch November 19, 2018 23:22
jasdel added a commit to jasdel/aws-sdk-go-v2 that referenced this pull request Dec 3, 2018
Fixes the S3 Upload Manager's unstable EOF detection test.

Related aws/aws-sdk-go#2286, aws#240
@diehlaws diehlaws added needs-review This issue or pull request needs review from a core team member. and removed review-needed labels Jan 4, 2019
lucix-aws added a commit that referenced this pull request Sep 20, 2023
lucix-aws added a commit that referenced this pull request Sep 21, 2023
lucix-aws added a commit that referenced this pull request Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review This issue or pull request needs review from a core team member.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants