Enable more tests to run on all 3 runtimes, part 1

[grendello]

No description provided.

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[jonathanpeppers]

I don't know if NUnit automatically finds all the enums for you, if you do: [Values] AndroidRuntime runtime

Let's see what the test run does, but I wonder if NUnit will siliently error and skip this test?

Or if it works fine, today I learned!

[grendello]

If you decorate a bool or an enum parameter with [Values], it will enumerate all the possible values, yep :) Very handy

[jonathanpeppers]

Maybe we should add a helper method for AotAssemblies, where you pass in the AndroidRuntime? And it returns true for Mono and false for the others.

[grendello]

I was considering doing that, yeah, but then I decided that tests should be explicit as much as possible in what they do (even if it's repetitive/verbose), they should be treated as separate entities without hidden code that does something.
I made an exception in the IgnoreUnsupportedConfiguration helper method because what it does can change in the future and it would be a real shame to have to go through all the tests and change them individually.
Another exception to this, but justified I think, is the SetRuntime method when called for NativeAOT, that's just environment setup unrelated to any particular test as such, so I think it's fine.

[jonathanpeppers]

I believe the way NUnit works, Assert.Ignore() throws an exception that tells NUnit to skip.

So, I think you could make this method void and you don't need an if-return statement in each test.

[grendello]

Yeah, Assert.Ignore() throws an exception, but I wanted to make the action explicit for human readers - thus the if-return pattern. It's verbose and repetitive, but makes it clear what's the intention and consequence. The actual action of ignoring via Assert.Ignore is hidden from the person reading a specific unit test. It is implied by the name, but without checking what IgnoreUnsupportedConfiguration actually does, they don't know the consequences.
Unit tests are a peculiar thing, and I think verbosity (and code repetition) are perfectly fine there, even desired.

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[grendello]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Copilot]

Pull Request Overview

This PR enables more tests to run across all three Android runtimes (MonoVM, CoreCLR, and NativeAOT) by adding runtime parameters to test methods and implementing runtime-specific configuration handling.

Key Changes

• Added helper methods in ProjectExtensions.cs to configure projects for specific runtimes, with special handling for NativeAOT
• Introduced IgnoreUnsupportedConfiguration() in BaseTest.cs to skip tests for unsupported runtime/configuration combinations
• Updated 40+ test methods in BuildTest.cs to accept an AndroidRuntime parameter and appropriately handle each runtime

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File	Description
ProjectExtensions.cs	Refactored SetRuntime() with new overload for XamarinAndroidApplicationProject and extracted helper methods for NativeAOT handling
BaseTest.cs	Added IgnoreUnsupportedConfiguration() to validate and skip unsupported runtime/build configurations (NativeAOT requires release, CoreCLR doesn't support AOT)
BuildTest.cs	Expanded test coverage by adding AndroidRuntime parameter to numerous tests, with appropriate runtime configuration and validation logic