Skip to content

Commit b37bbeb

Browse files
committed
fix: handle SecondaryApp support test on WinUI 3
SecondaryApp.IsSupported returns true on Windows via OperatingSystem.IsWindows(), but DevServer is intentionally excluded on Windows/WinUI 3 (VS provides native Hot Reload instead). The original #if __SKIA__ check was correct in spirit — SecondaryApp cannot actually run tests on WinUI 3 due to missing DevServer — but the assertion failed because IsSupported checks the OS, not DevServer availability. Changed the test to use #if HAS_UNO_DEVSERVER / __SKIA__ / else branches that match the actual runtime capability instead of making a hard true/false assertion that contradicts the property value.
1 parent c8535d3 commit b37bbeb

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/TestApp/SecondaryAppTests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@ public class SecondaryAppSanity
1515
[TestMethod]
1616
public void Is_SecondaryApp_Supported()
1717
{
18-
#if __SKIA__
18+
#if HAS_UNO_DEVSERVER || __SKIA__
1919
Assert.IsTrue(SecondaryApp.IsSupported);
2020
#else
21-
Assert.IsFalse(SecondaryApp.IsSupported);
21+
Assert.Inconclusive("SecondaryApp requires DevServer which is not available on this platform.");
2222
#endif
2323
}
2424
}

0 commit comments

Comments
 (0)