Skip to content

[release/9.0][browser] Disable debugger tests on CI (#108178)#128132

Open
akoeplinger wants to merge 1 commit into
dotnet:release/9.0-stagingfrom
akoeplinger:backport-disable-debugger-tests
Open

[release/9.0][browser] Disable debugger tests on CI (#108178)#128132
akoeplinger wants to merge 1 commit into
dotnet:release/9.0-stagingfrom
akoeplinger:backport-disable-debugger-tests

Conversation

@akoeplinger
Copy link
Copy Markdown
Member

@akoeplinger akoeplinger commented May 13, 2026

Backport of #108178 to release/9.0-staging

Fixes #128130

@akoeplinger
Copy link
Copy Markdown
Member Author

Test-only change that we did in main already, marking Servicing-approved.

@akoeplinger akoeplinger added the Servicing-approved Approved for servicing release label May 13, 2026
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @steveisok, @thaystg, @dotnet/dotnet-diag
See info in area-owners.md if you want to be subscribed.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR backports the change to disable (and remove CI wiring for) Mono WASM/browser debugger tests on the release/9.0 branch, addressing CI failures caused by environment/ICU differences (issue #128130).

Changes:

  • Removes WASM debugger test submission/build wiring to Helix and eliminates the wasmdebuggertests scenario.
  • Updates browser/wasm Helix targets and validation to drop the WasmDebuggerTests scenario.
  • Removes pipeline templates and path-trigger subsets that previously scheduled debugger test jobs.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/mono/wasi/Makefile Removes the Helix submission target for debugger tests.
src/mono/browser/Makefile Removes Helix build/submission targets for debugger tests.
src/mono/browser/debugger/Wasm.Debugger.Tests/wasm.helix.targets Deletes debugger-test-specific Helix work item generation.
src/mono/browser/debugger/Wasm.Debugger.Tests/Wasm.Debugger.Tests.csproj Removes the HelixTargetsFile hook.
src/libraries/tests.proj Stops building the debugger test traversal projects and adjusts wasm sample gating.
src/libraries/sendtohelix.proj Removes special-casing for the debugger test scenario and host matrix.
src/libraries/sendtohelix-browser.targets Removes WasmDebuggerTests scenario support and updates scenario validation error text.
eng/testing/tests.browser.targets Removes WasmDebuggerTests scenario support and updates scenario validation error text.
eng/pipelines/runtime.yml Removes scheduling of wasm debugger tests from main runtime pipeline.
eng/pipelines/extra-platforms/runtime-extra-platforms-wasm.yml Removes wasm debugger jobs from extra-platforms/wasm pipelines.
eng/pipelines/common/templates/wasm-debugger-tests.yml Deletes the dedicated wasm debugger tests pipeline template.
eng/pipelines/common/evaluate-default-paths.yml Removes the wasmdebuggertests path subset used for CI triggering.
Comments suppressed due to low confidence (1)

eng/pipelines/extra-platforms/runtime-extra-platforms-wasm.yml:62

  • With the debugger test jobs removed from this template, the debuggerTestsOnly mode now results in an empty pipeline (e.g., eng/pipelines/runtime-wasm-dbgtests.yml passes debuggerTestsOnly: true, but all job groups are gated on ne(parameters.debuggerTestsOnly, true) and excludeOptional defaults to true). Either delete/update runtime-wasm-dbgtests.yml and remove the debuggerTestsOnly parameter/conditions here, or repoint that pipeline to a remaining job group so it still runs something meaningful.
  # Disabled for now
  #- template: /eng/pipelines/coreclr/perf-wasm-jobs.yml
    #parameters:
      #runProfile: 'v8'
      #onlySanityCheck: true

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/mono/wasi/Makefile
Comment thread src/libraries/tests.proj
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-Debugger-mono Servicing-approved Approved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants