Skip to content

[Windows] Review currently failing Windows tests #15530

Open

Description

Description

As part of #15527 I've skipped the tests listed below. All of these tests were failing on the original PR. We don't necessarily need to fix these issues for .NET8 but we should establish if we should fix these for .NET8 or just fix the tests themselves.

In addition after merging #15629 we're now running the DeviceTests for Windows, these have been disabled because they were failing and we should look into them.

Core
At the time of writing the whole Core project test runs have been disabled because it would crash while running, so that is something we need to fix, other than that, these tests are failing:

  • WebViewPlaysHtml5Video
  • Dashed Stroke Initializes Correctly (all variations)
  • Background Initializes Correctly (all variations)
  • Stroke Initializes Correctly (all variations)
  • CancelButtonColorInitializeCorrectly
  • StrokeShape Initializes Correctly (all variations)
  • ThumbColorUpdatesCorrectly
  • CursorPositionDoesntResetWhenNativeTextValueChanges
  • BackgroundUpdatesCorrectly
  • SettingSemanticHintMakesElementAccessible
  • DisconnectHandlerDoesntCrash
  • SettingSemanticDescriptionMakesElementAccessible
  • Enable Core DeviceTests Project (Enable WinUI Core Tests on CI #15885)
  • CursorPositionDoesntResetWhenNativeTextValueChanges
  • SettingSemanticDescriptionMakesElementAccessible (Activity Indicator)
  • SettingSemanticHintMakesElementAccessible (Activity Indicatr)
  • SwipeViewHandlerTests

BlazorWebView
#16187

Essentials

  • CanOpenUri (sms, tel and mailto)
  • CanOpen (sms, tel and mailto)
  • IsSupported (App Actions)
  • Vibrate
  • Vibrate_Cancel
  • Charge_Level

Controls
Just like Core, the Controls projects also crashes somewhere along the test run and never completes. We need to investigate why and enable it for running on CI again

  • Enable Controls DeviceTests Project

Unpackaged
Then we need to run this full suite also for unpackaged apps. I think we can make this a variation by leveraging the device parameter that is in place for iOS and Android and add that to the matrix for Windows

  • Enable unpackaged DeviceTests Project

Random

  • Read Windows app package ID from the csproj instead of providing it as a parameter
  • Check if we still need the RunHeadless property
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions