Skip to content

[tests] More network resilience for tests - #2954

Merged
jonpryor merged 1 commit into
dotnet:masterfrom
grendello:xa/ssltest-network
Apr 11, 2019
Merged

jonpryor merged 1 commit into
dotnet:masterfrom
grendello:xa/ssltest-network

Conversation

@grendello

Copy link
Copy Markdown
Contributor

Context: 608d950
Context: 34dfc6d

SslTests notoriously fail for reasons not related to the subject of the test,
thus creating false negatives when our bots flag the related builds/PRs as red
while the failure can/should be ignored.

Wrap the tests in code which ignores the most common failures we can safely
ignore.

@grendello
grendello requested a review from jonpryor as a code owner April 10, 2019 18:14
Context: 608d950
Context: 34dfc6d

SslTests notoriously fail for reasons not related to the subject of the test,
thus creating false negatives when our bots flag the related builds/PRs as red
while the failure can/should be ignored.

Wrap the tests in code which ignores the most common failures we can safely
ignore.

Additionally, slightly improve connection error handling in
AndroidHttpClientHandler tests.
@grendello
grendello force-pushed the xa/ssltest-network branch from 605cb74 to f281771 Compare April 11, 2019 08:27
@jonpryor
jonpryor merged commit 9cd0b02 into dotnet:master Apr 11, 2019
@grendello
grendello deleted the xa/ssltest-network branch April 11, 2019 20:13
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants