Restore EnsureAotCompatibility test

[kllysng]

The test is passing locally without issue. Running an ADO PR build to ensure it passes there too.

Update - test passed in ADO build with no issues, running again

Update again - passed again

[msbw2]

I think this might be a build step. At least it would make sense to be one to me instead of a unit test.

[keegan-caruso]

I think this might be a build step. At least it would make sense to be one to me instead of a unit test.

azure-activedirectory-identitymodel-extensions-for-dotnet/build/common.props

Line 78 in 90dfd31

<IsAotCompatible>true</IsAotCompatible>

[jmprieur]

LGTM
Thanks @kellyyangsong

Do we understand why it was skipped in the first place?

[kllysng]

LGTM Thanks @kellyyangsong

Do we understand why it was skipped in the first place?

@jmprieur yes, thanks for asking, I should have explained in a comment. It was skipped because it would timeout which for this test is taking longer than 3 min in the ADO build. I re-ran this many times and never saw it take longer than 50s to complete.

[kllysng]

I think this might be a build step. At least it would make sense to be one to me instead of a unit test.

I think this might be a build step. At least it would make sense to be one to me instead of a unit test.

azure-activedirectory-identitymodel-extensions-for-dotnet/build/common.props

Line 78 in 90dfd31

<IsAotCompatible>true</IsAotCompatible>

We have an analyzer that will cause warnings - but I still think this test is useful until we start treating warnings as errors.