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

HTTP test null refs in 'Build Browser wasm Release AllSubsets_Mono_EAT' CI leg #53647

Closed
stephentoub opened this issue Jun 2, 2021 · 4 comments · Fixed by #54932
Closed

HTTP test null refs in 'Build Browser wasm Release AllSubsets_Mono_EAT' CI leg #53647

stephentoub opened this issue Jun 2, 2021 · 4 comments · Fixed by #54932
Assignees
Labels
arch-wasm WebAssembly architecture area-System.Net.Http disabled-test The test is disabled in source code against the issue os-android trimming-for-aot `EnableAggressiveTrimming=true` used for running tests with AOT
Milestone

Comments

@stephentoub
Copy link
Member

[13:10:13] fail: [FAIL] System.Net.Http.Tests.HttpHeadersTest.NonValidated_ValidAndInvalidValues_DictionaryMembersWork
[13:10:13] info: System.NullReferenceException : Object reference not set to an instance of an object.
[13:10:13] info:    at System.Net.Http.Tests.HttpHeadersTest.NonValidated_ValidAndInvalidValues_DictionaryMembersWork()
[13:10:13] info:    at System.Reflection.RuntimeMethodInfo.Invoke(Object , BindingFlags , Binder , Object[] , CultureInfo )
[13:10:13] fail: console.error: /__w/1/s/src/mono/mono/utils/memfuncs.c:360 <disabled>

It doesn't repro in any of the other legs. From @lewing: "Ah, so the EAT leg aggressively trims the test itself so this is almost certainly trimming related."

@stephentoub stephentoub added arch-wasm WebAssembly architecture area-System.Net.Http disabled-test The test is disabled in source code against the issue labels Jun 2, 2021
@stephentoub stephentoub added this to the 6.0.0 milestone Jun 2, 2021
@ghost
Copy link

ghost commented Jun 2, 2021

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details
[13:10:13] fail: [FAIL] System.Net.Http.Tests.HttpHeadersTest.NonValidated_ValidAndInvalidValues_DictionaryMembersWork
[13:10:13] info: System.NullReferenceException : Object reference not set to an instance of an object.
[13:10:13] info:    at System.Net.Http.Tests.HttpHeadersTest.NonValidated_ValidAndInvalidValues_DictionaryMembersWork()
[13:10:13] info:    at System.Reflection.RuntimeMethodInfo.Invoke(Object , BindingFlags , Binder , Object[] , CultureInfo )
[13:10:13] fail: console.error: /__w/1/s/src/mono/mono/utils/memfuncs.c:360 <disabled>

It doesn't repro in any of the other legs. From @lewing: "Ah, so the EAT leg aggressively trims the test itself so this is almost certainly trimming related."

Author: stephentoub
Assignees: -
Labels:

arch-wasm, area-System.Net.Http, disabled-test

Milestone: 6.0.0

@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Jun 2, 2021
@karelz karelz removed the untriaged New issue has not been triaged by the area owner label Jun 3, 2021
@MaximLipnin
Copy link
Contributor

Same for Android AOT lane with EAT in #50510

@lewing
Copy link
Member

lewing commented Jun 17, 2021

cc @SamMonoRT

@lewing lewing added trimming-for-aot `EnableAggressiveTrimming=true` used for running tests with AOT os-android labels Jun 17, 2021
@naricc naricc self-assigned this Jun 22, 2021
@fanyang-mono fanyang-mono self-assigned this Jun 30, 2021
@fanyang-mono
Copy link
Member

Test passed when running locally with trimming.

@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jun 30, 2021
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Jul 1, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jul 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture area-System.Net.Http disabled-test The test is disabled in source code against the issue os-android trimming-for-aot `EnableAggressiveTrimming=true` used for running tests with AOT
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants