-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[release/8.0-rc1] [mono] Fix InlineArray tests on android platforms #90679
[release/8.0-rc1] [mono] Fix InlineArray tests on android platforms #90679
Conversation
Tagging subscribers to this area: @directhex Issue DetailsBackport of #90583 to release/8.0-rc1 /cc @kotlarmilos Customer ImpactTestingRiskIMPORTANT: If this backport is for a servicing release, please verify that:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@carlossanlop this is just a test infrastructure change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CI failure is known: |
Description
Backport of #90583 to release/8.0-rc1
This PR enables class loader runtime tests on android platforms on the CI.
Runtime tests for mobile targets are grouped into bundles, where each bundle contains multiple entry points. All assemblies are copied to the root directory of a given bundle on android platforms, meaning if two or more assemblies within one test group have the same names, one will overwrite the other, which could potentially lead to runtime errors.
The fix for this issue, an assembly is renamed to be unique in the test app bundle.
Customer Impact
The issue was discovered by a team member.
Testing
Manual testing and automated testing on the CI were performed, ensuring that the tests are passing on android platforms. The issue has been reproduced locally, and has been fixed.
Risk
Low risk. This change enables class loader runtime tests on android platforms on the CI.
The change doesn't affect code that ships in a NuGet package.