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

[mono-2019-02][Xamarin.Android.BCL_Tests] Copy xunit tests to IntermediateOutputPath #2976

Merged

Conversation

lambdageek
Copy link
Member

so that their references can be resolved

Otherwise, any referenced assemblies that are only used by the xunit testsuites
will not be included in the APK.


I'm not sure if this is the best way - with this the XUnit test assemblies all end up inside the APK in the assemblies/ folder (the same way that all the NUnit test assemblies already do) - which means they're duplicated in the Xamarin.Android.BCL_Test.apk - they're included once more in the bcl-tests.zip that's embedded as a resource in the assemblies/Xamarin.Android.Bcl_Tests.dll inside the APK.

But I've been told that that's a know issue that will be dealt with separately.

so that their references can be resolved

Otherwise, any referenced assemblies that are only used by the xunit testsuites
will not be included in the APK.
@lambdageek lambdageek requested a review from grendello as a code owner April 12, 2019 16:29
@lambdageek lambdageek requested a review from alexischr April 12, 2019 16:29
@alexischr alexischr added the full-mono-integration-build For PRs; run a full build (~6-10h for mono bumps), not the faster PR subset (~2h for mono bumps) label Apr 12, 2019
@alexischr alexischr merged commit 01811ad into dotnet:mono-2019-02 Apr 16, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Jan 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
full-mono-integration-build For PRs; run a full build (~6-10h for mono bumps), not the faster PR subset (~2h for mono bumps)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants