Skip to content

chore(tests)#6363

Merged
johnjenkins merged 2 commits intostenciljs:mainfrom
jttommy:chore/wdio-snapshot-tests
Aug 4, 2025
Merged

chore(tests)#6363
johnjenkins merged 2 commits intostenciljs:mainfrom
jttommy:chore/wdio-snapshot-tests

Conversation

@jttommy
Copy link
Copy Markdown
Contributor

@jttommy jttommy commented Aug 4, 2025

Uses document tree verification instead of snapshot tests to confirm rendering works for two failing tests. Tests were failing due to some limitations with the testing framework. Specifically that numerous snapshots of the same segment of HTML were not allowed in certain use cases

What is the current behavior?

Two WDIO tests were failing due to framework limitations on toMatchInlineSnapshot

GitHub Issue Number: N/A

What is the new behavior?

The tests now verify attribute on the DOM tree for correctness.

Documentation

Does this introduce a breaking change?

  • Yes
  • No

Testing

Other information

Uses document tree verification instead of snapshot tests to confirm rendering works for two failing tests.
Tests were failing due to some limitations with the testing framework.  Specifically that numerous snapshots of the same segment
of HTML were not allowed in certain use cases
@jttommy jttommy requested a review from a team as a code owner August 4, 2025 17:40
I've run the formatter and linter.  Fixed a linting error in package.json where Windows machines will not run eslint if the targets are wrapped in single quotes
@johnjenkins johnjenkins enabled auto-merge August 4, 2025 21:18
Copy link
Copy Markdown
Contributor

@johnjenkins johnjenkins left a comment

Choose a reason for hiding this comment

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

LGTM!

@johnjenkins johnjenkins added this pull request to the merge queue Aug 4, 2025
Merged via the queue into stenciljs:main with commit 6b41e88 Aug 4, 2025
69 checks passed
@jttommy jttommy deleted the chore/wdio-snapshot-tests branch September 30, 2025 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants