Skip to content

Commit

Permalink
Fixes to enable CI and Release Pipelines to work reliably (#4919)
Browse files Browse the repository at this point in the history
A few fixes:
Disabling unreliable test: [Test MUXControls\.ReleaseTest\.RepeaterNoCrashTest is unreliable \#4899](#4899)

XamlIslandsTestApp was failing to install on test machines due to TargetPlatformMinVersion being tied to the version of SDK.

WinUI-CI was using a 'taefQuery' parameter to skip some tests (NugetPkgTestsOnly='True') when run as a framework package test app. However, this was not being plumbed through for the Release Pipeline, which resulted in test failures. The yml is refactored a bit to avoid this problem.
  • Loading branch information
kmahone authored Apr 28, 2021
1 parent 1bf8b63 commit fdb393e
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion build/AzurePipelinesTemplates/MUX-NugetReleaseTest-Job.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,8 @@ jobs:
testSuite: 'NugetTestSuite'
helixType: ${{ parameters.helixType }}
artifactName: ${{ parameters.buildArtifactName }}
taefQuery: ${{ parameters.taefQuery }}
${{if eq(parameters.useFrameworkPkg, 'true')}}:
taefQuery: "(not (@NugetPkgTestsOnly = 'true'))"
rerunPassesRequiredToAvoidFailure: 5
${{if ne(parameters.test_matrix, '') }}:
matrix: ${{ parameters.test_matrix }}
1 change: 0 additions & 1 deletion build/MUX-CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ jobs:
helixType: 'test/frpkg'
dependsOn: CreateNugetPackage
useFrameworkPkg: true
taefQuery: "(not (@NugetPkgTestsOnly = 'true'))"

- template: AzurePipelinesTemplates\MUX-ProcessTestResults-Job.yml
parameters:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ public void BasicTestCX()
}

[TestMethod]
[TestProperty("Ignore", "True")] // Issue #4899: Test MUXControls.ReleaseTest.RepeaterNoCrashTest is unreliable
public void RepeaterNoCrashTest()
{
var button = new Button(FindElement.ByName("AddItemsButton"));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
<PropertyGroup>
<ProjectGuid>4454b583-39a0-4a91-881c-2078fa26c447</ProjectGuid>
<TargetPlatformVersion>$(MuxSdkVersion)</TargetPlatformVersion>
<TargetPlatformMinVersion>$(MuxSdkVersion)</TargetPlatformMinVersion>
<TargetPlatformMinVersion>10.0.18362.0</TargetPlatformMinVersion>
<DefaultLanguage>en-US</DefaultLanguage>
<AppxPackageSigningEnabled>true</AppxPackageSigningEnabled>
<EntryPointProjectUniqueName>WpfApp.csproj</EntryPointProjectUniqueName>
Expand Down

0 comments on commit fdb393e

Please sign in to comment.