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.

Sign up for GitHub

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

Jump to bottom

[Mono] System.Reflection.Metadata.Tests.MetadataReaderTests.GetAssemblyName is failing #80863

Closed
simonrozsival opened this issue Jan 19, 2023 · 3 comments · Fixed by #80878
Closed

[Mono] System.Reflection.Metadata.Tests.MetadataReaderTests.GetAssemblyName is failing #80863

simonrozsival opened this issue Jan 19, 2023 · 3 comments · Fixed by #80878
Assignees
@VSadov

Comments

@simonrozsival
Copy link
Member

The changes in #80581 cause the rolling build of runtime-extra-platforms to fail.

Ref #77697

On android-arm64, android-arm, tvos-arm64:

Assert.Equal() Failure
Expected: MSIL
Actual:   None

Stack trace
   at System.Reflection.Metadata.Tests.MetadataReaderTests.GetAssemblyName() in /_/src/libraries/System.Reflection.Metadata/tests/Metadata/MetadataReaderTests.cs:line 3098
   at System.Reflection.MethodInvoker.InterpretedInvoke(Object obj, IntPtr* args)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)

On android-x64, tvossimulator-x64:

Assert.Equal() Failure
Expected: MSIL
Actual:   Amd64

On android-x86:

Assert.Equal() Failure
Expected: MSIL
Actual:   X86
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Jan 19, 2023
@jkotas
Copy link
Member

jkotas commented Jan 19, 2023

cc @VSadov

@VSadov
Copy link
Member

VSadov commented Jan 19, 2023

This could be a result of intentional simplification of the ProcessorArchitecture computation. I will take a look.

@VSadov
Copy link
Member

VSadov commented Jan 19, 2023

Actually it looks like new test scenarios. Possibly a test issue.

@VSadov VSadov self-assigned this Jan 19, 2023
steveisok pushed a commit to steveisok/runtime that referenced this issue Jan 19, 2023
[iOS][Android] ActiveIssue tests that are failing on CI
2a6a4c4 
dotnet#80876

dotnet#80875

dotnet#80863
@steveisok steveisok mentioned this issue Jan 19, 2023
Merged
@VSadov VSadov mentioned this issue Jan 19, 2023
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Jan 19, 2023
@simonrozsival simonrozsival mentioned this issue Jan 20, 2023
Merged
@ghost ghost removed in-pr There is an active PR which will close this issue when it is merged untriaged New issue has not been triaged by the area owner labels Jan 20, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Feb 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@VSadov VSadov

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make GetAssemblyName test more robust. VSadov/runtime
3 participants
@simonrozsival @jkotas @VSadov