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

Test failed: DiagnosticsTest / SendAsync_HttpTracingEnabled_Succeeds #1477

Closed
davidsh opened this issue Sep 2, 2017 · 3 comments · Fixed by #54240
Closed

Test failed: DiagnosticsTest / SendAsync_HttpTracingEnabled_Succeeds #1477

davidsh opened this issue Sep 2, 2017 · 3 comments · Fixed by #54240
Labels
area-System.Net.Http disabled-test The test is disabled in source code against the issue os-linux Linux OS (any supported distro) test-bug Problem in test source code (most likely)
Milestone

Comments

@davidsh
Copy link
Contributor

davidsh commented Sep 2, 2017

Test failed during PR run:
https://mc.dot.net/#/user/davidsh/pr~2Fjenkins~2Fdotnet~2Fcorefx~2Fmaster~2F/test~2Ffunctional~2Fcli~2F/169daaad2062ab623fdafa738c3fdbc778d3652e

DiagnosticsTest/SendAsync_HttpTracingEnabled_Succeeds (Linux) (ManagedHandler)
Unhandled Exception of Type Xunit.Sdk.TrueException
Message :
Exit code was 139 but it should have been 42
Expected: True
Actual: False

@Priya91 Priya91 changed the title Diagnostics test failed: DiagnosticsTest/SendAsync_HttpTracingEnabled_Succeeds [ManagedHandler] Diagnostics test failed: DiagnosticsTest/SendAsync_HttpTracingEnabled_Succeeds Dec 8, 2017
@Priya91
Copy link
Contributor

Priya91 commented Dec 8, 2017

@davidsh You mention it is managed handler in the description, but the test result doesn't looks like managed handler?

@Priya91 Priya91 changed the title [ManagedHandler] Diagnostics test failed: DiagnosticsTest/SendAsync_HttpTracingEnabled_Succeeds Diagnostics test failed: DiagnosticsTest/SendAsync_HttpTracingEnabled_Succeeds Dec 8, 2017
@davidsh
Copy link
Contributor Author

davidsh commented Dec 8, 2017

@davidsh You mention it is managed handler in the description, but the test result doesn't looks like managed handler?

It is ManagedHandler.

Ref: https://mc.dot.net/#/user/davidsh/pr~2Fjenkins~2Fdotnet~2Fcorefx~2Fmaster~2F/test~2Ffunctional~2Fcli~2F/169daaad2062ab623fdafa738c3fdbc778d3652e/workItem/System.Net.Http.Functional.Tests

See:
image

@karelz karelz changed the title Diagnostics test failed: DiagnosticsTest/SendAsync_HttpTracingEnabled_Succeeds Test failed: DiagnosticsTest / SendAsync_HttpTracingEnabled_Succeeds Dec 24, 2017
@karelz karelz changed the title Test failed: DiagnosticsTest / SendAsync_HttpTracingEnabled_Succeeds [ManagedHandler] Test failed: DiagnosticsTest / SendAsync_HttpTracingEnabled_Succeeds Dec 24, 2017
@karelz karelz changed the title [ManagedHandler] Test failed: DiagnosticsTest / SendAsync_HttpTracingEnabled_Succeeds Test failed: DiagnosticsTest / SendAsync_HttpTracingEnabled_Succeeds May 16, 2018
davidsh referenced this issue in davidsh/corefx Apr 30, 2019
Some of these tests were disabled a long time ago and only on Linux.
Since that time, the tests were improved.

I re-enabled the tests plus some others that appear to be stable now.

Closes #23209
Closes #23771
@karelz
Copy link
Member

karelz commented Oct 10, 2019

Triage: The tests need to run isolated - now we have option to do that.

@karelz karelz transferred this issue from dotnet/corefx Jan 9, 2020
@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added area-System.Net.Http untriaged New issue has not been triaged by the area owner labels Jan 9, 2020
@karelz karelz added bug disabled-test The test is disabled in source code against the issue os-linux Linux OS (any supported distro) and removed untriaged New issue has not been triaged by the area owner labels Jan 9, 2020
@karelz karelz added this to the Future milestone Jan 9, 2020
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jun 15, 2021
@MihaZupan MihaZupan added test-bug Problem in test source code (most likely) and removed bug labels Jun 15, 2021
@MihaZupan MihaZupan modified the milestones: Future, 6.0.0 Jun 15, 2021
@ghost ghost added in-pr There is an active PR which will close this issue when it is merged and removed in-pr There is an active PR which will close this issue when it is merged labels Jun 16, 2021
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jun 16, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jul 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Net.Http disabled-test The test is disabled in source code against the issue os-linux Linux OS (any supported distro) test-bug Problem in test source code (most likely)
Projects
None yet
5 participants