Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WinUI 3 tests failing at random in CI #347

Open
24 tasks
Arlodotexe opened this issue Feb 27, 2024 · 3 comments
Open
24 tasks

WinUI 3 tests failing at random in CI #347

Arlodotexe opened this issue Feb 27, 2024 · 3 comments
Assignees
Labels
bug Something isn't working regression What was working is now broke

Comments

@Arlodotexe
Copy link
Member

Describe the bug

The WinUI 3 tests that are run in the CI for build validation are sometimes failing with the following error:

  Passed RequestsAsync (20,5) [4 ms]
  Passed RequestsAsync (5,5) [< 1 ms]
  Passed RequestsAsync (20,5,2500,1000,1000,1000,1000) [6 s]
The active test run was aborted. Reason: Unable to communicate with test host process.
Terminating app with process ID '14816'.
Results File: C:\a\Windows\Windows\TestResults\WinAppSdk.trx

The active Test Run was aborted because the host process exited unexpectedly. Please inspect the call stack above, if available, to get more information about where the exception originated from.
The test running when the crash occurred: 
StackedNotificationsBehaviorExperiment.Tests.StackedNotificationsBehaviorTestClass.NotificationBehaviorSingleTest_StackedNotificationsBehaviorTestClass_Test

This test may, or may not be the source of the crash.

This seems to be happening more or less at random and can sometimes be resolved with a rerun. However, this has been happening more often in the last few weeks and is increasingly becoming a blocker for other work.

Steps to reproduce

Exact repro steps are unknown. Seems to only be happening in the CI and hasn't been reproduced locally.

Expected behavior

Consistent test results, no failure if there's no problem.

Screenshots

No response

Code Platform

  • UWP
  • WinAppSDK / WinUI 3
  • Web Assembly (WASM)
  • Android
  • iOS
  • MacOS
  • Linux / GTK

Windows Build Number

  • Windows 10 1809 (Build 17763)
  • Windows 10 1903 (Build 18362)
  • Windows 10 1909 (Build 18363)
  • Windows 10 2004 (Build 19041)
  • Windows 10 20H2 (Build 19042)
  • Windows 10 21H1 (Build 19043)
  • Windows 10 21H2 (Build 19044)
  • Windows 10 22H2 (Build 19045)
  • Windows 11 21H2 (Build 22000)
  • Other (specify)

Other Windows Build number

No response

App minimum and target SDK version

  • Windows 10, version 1809 (Build 17763)
  • Windows 10, version 1903 (Build 18362)
  • Windows 10, version 1909 (Build 18363)
  • Windows 10, version 2004 (Build 19041)
  • Windows 10, version 2104 (Build 20348)
  • Windows 11, version 22H2 (Build 22000)
  • Other (specify)

Other SDK version

No response

Visual Studio Version

No response

Visual Studio Build Number

No response

Device form factor

No response

Additional context

No response

Help us help you

Yes, I'd like to be assigned to work on this item.

@Arlodotexe Arlodotexe added bug Something isn't working regression What was working is now broke labels Feb 27, 2024
@Arlodotexe Arlodotexe self-assigned this Feb 27, 2024
@Arlodotexe Arlodotexe moved this to 🏗 In progress in Toolkit 8.x Feb 27, 2024
@Arlodotexe Arlodotexe moved this from 🏗 In progress to 👀 In review in Toolkit 8.x Mar 1, 2024
@Arlodotexe Arlodotexe moved this from 👀 In review to 🛑 Blocked in Toolkit 8.x Mar 5, 2024
@Arlodotexe
Copy link
Member Author

Looks like the cleanup done in #348 has slowed down the frequency of the test error, but not eliminated it altogether.

@Arlodotexe
Copy link
Member Author

In #453 (comment), we caught the issue happening on a UWP test run. This may not be specific to WinUI 3.

@michael-hawker
Copy link
Member

Part of the instability seems to be unexpected exceptions in tests, like #589 where the exception is unhandled and therefore the test process gets hung.

In that case it's using the old setup vs. our new generators/helpers. So, it could be good to look how these behave with the new setups and if we can better ensure that any exceptions are caught within the test setup and reported as test fails explicitly instead of killing the test process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression What was working is now broke
Projects
Status: 🏗 In progress
Development

No branches or pull requests

2 participants