Skip to content

Conversation

@dmytrokirpa
Copy link
Contributor

@dmytrokirpa dmytrokirpa commented Nov 11, 2025

Previous Behavior

We found in #35346 (comment) that some WebComponents VR tests were logging errors quietly without causing the pipeline to fail.

New Behavior

  • Added --bailOnStoriesError flag to the test-vr script in package.json for improved error handling.
  • Updated checkbox story to use Array.from as HTMLCollection doesn't have forEach method
  • Added missing id attribute to the long text link story for proper element targeting.
  • Fixed submenu items targeting in the menu list story by adding aria-haspopup attributes.
  • Updated radio group story to use fluent-radio selectors for as there are no elements with the [role=radio] attribute
  • Changed slider story to focus on fluent-slider instead of [role="slider"] as there are no elements such elements

Related Issue(s)

@github-actions
Copy link

📊 Bundle size report

✅ No changes found

@github-actions
Copy link

Pull request demo site: URL

@dmytrokirpa dmytrokirpa enabled auto-merge (squash) November 11, 2025 13:34
@dmytrokirpa
Copy link
Contributor Author

Could I get a review for this one, please, @microsoft/fui-wc?

@dmytrokirpa dmytrokirpa disabled auto-merge November 14, 2025 17:09
@dmytrokirpa dmytrokirpa merged commit 6cbe729 into microsoft:master Nov 15, 2025
12 checks passed
@dmytrokirpa dmytrokirpa deleted the fix/vr-tests-web-components/bail-on-stories-error branch November 15, 2025 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants