-
Notifications
You must be signed in to change notification settings - Fork 564
Description
Android application type
Classic Xamarin.Android (MonoAndroid12.0, etc.)
Affected platform version
VS 17.3
Description
Commit e1af958 seems to have somehow affected Assembly.ReflectionOnlyLoad related API usage. We're seeing four new BCL test failures that are related:
https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=6270744&view=ms.vss-test-web.build-test-results-tab
For some reason, this API usage is no longer throwing -- we don't seem hit some of these expected code paths:
https://github.com/mono/mono/blob/5d2e3bc3b3c8184d35b2f7801e88d96470d367c4/mcs/class/corlib/System.Reflection/Assembly.cs#L556
https://github.com/mono/mono/blob/5d2e3bc3b3c8184d35b2f7801e88d96470d367c4/mono/metadata/icall.c#L2089
Steps to Reproduce
Use some Assembly.ReflectionOnlyLoad related reflection, example:
- https://github.com/mono/mono/blob/0e5e464cff103f104bee6beb22461ea7332fe102/mcs/class/corlib/Test/System.Reflection/AssemblyTest.cs#L735
- https://github.com/mono/mono/blob/5d2e3bc3b3c8184d35b2f7801e88d96470d367c4/mcs/class/corlib/Test/System.Reflection/FieldInfoTest.cs#L427
These no longer throw an InvalidOperationExceptions as expected.
Did you find any workaround?
No response
Relevant log output
No response