Skip to content

[Tests] Incremental build tests need AssertCommercialBuild() for Fast Deployment #11419

@jonathanpeppers

Description

@jonathanpeppers

Several incremental build tests fail on the public dnceng-public pipeline because they rely on Fast Deployment, which is only available in commercial builds. These tests should call AssertCommercialBuild() with a comment explaining why.

Failing tests from build 1427200:

Test Failed Assertion
ResolveLibraryProjectImports(NativeAOT) _UpdateAndroidResgen should be skipped
ManifestMergerIncremental(MonoVM) _ManifestMerger should have been skipped
BuildApplicationWithAssetPack(True,NativeAOT) _GenerateAndroidAssetsDir should have been skipped
AndroidResourceChange(NativeAOT) _CompileJava should have been skipped
CheckDontUpdateResourceIfNotNeeded(True,CoreCLR) _UpdateAndroidResgen should have been skipped
OverlayManifestIncrementalBuildTest(CoreCLR) _ManifestMerger should have been skipped
BuildAotApplicationAndmlts("arm64-v8a",False,False) _CompileJava target should be skipped

All of these are incremental build tests that verify certain MSBuild targets are skipped on the second build. They fail on public CI because without Fast Deployment, additional targets run during the incremental build.

Fix: Each test should call AssertCommercialBuild() with a comment explaining that Fast Deployment is needed for the incremental assertions to hold.

Metadata

Metadata

Assignees

No one assigned

    Labels

    needs-triageIssues that need to be assigned.

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions