Skip to content

test(heading): avoid newSpecPage usage to ease Lumina migration#10431

Merged
jcfranco merged 1 commit intodevfrom
jcfranco/avoid-newSpecPage-usage
Sep 28, 2024
Merged

test(heading): avoid newSpecPage usage to ease Lumina migration#10431
jcfranco merged 1 commit intodevfrom
jcfranco/avoid-newSpecPage-usage

Conversation

@jcfranco
Copy link
Member

@jcfranco jcfranco commented Sep 27, 2024

Related Issue: #10349

Summary

Adds stopgap VNode/rendering assertion util to Heading.spec.tsx. Once #10310 lands, we can revisit this and use better test tooling.

@jcfranco jcfranco added the skip visual snapshots Pull requests that do not need visual regression testing. label Sep 27, 2024
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

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

👍

@jcfranco jcfranco merged commit 9312e29 into dev Sep 28, 2024
@jcfranco jcfranco deleted the jcfranco/avoid-newSpecPage-usage branch September 28, 2024 00:12
},
) {
expect(vnode).toEqual(
expect.objectContaining({
Copy link
Member

@maxpatiiuk maxpatiiuk Sep 28, 2024

Choose a reason for hiding this comment

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

TIL: I didn't know this was a thing!

supported in vitest too

benelan added a commit that referenced this pull request Sep 30, 2024
…estone-estimates

* origin/dev: (26 commits)
  test: fix `window.getComputedStyle` arguments (#10424)
  test(card): remove redundant spy and setup wrapper element (#10429)
  test(heading): avoid `newSpecPage` usage to ease Lumina migration (#10431)
  refactor(themed): drop broken, unused, regexp arg support (#10425)
  docs: component token description consistency (#10430)
  refactor: prevent mixed Sass mixed declaration warnings (#10426)
  deprecate: deprecate `enforce-ref-last-prop` rule (#10421)
  chore(actions): add custom `deprecate` type to semantic-pr check (#10427)
  chore(issue templates): add arcgis data pipelines to team dropdown (#10418)
  fix(tooltip): closed tooltips should not reappear (#10420)
  feat: add dashboard-graph (#10417)
  fix(input-time-zone): fix region mode quirks after update (#10413)
  fix(text-area): ensure border-color token doesn't override invalid styles (#10390)
  ci(changelog): add deprecation commits to changelog automation (#10346)
  fix: properly set aria attributes on components (#10404)
  feat: add parcel parameter (#10384)
  feat(alert): apply --calcite-alert-corner-radius to internal close button (#10388)
  feat(dialog, panel): Add css properties for background-color (#10387)
  fix: remove aria-disabled from components where necessary (#10374)
  feat(action-group, block, panel): add `menuPlacement` and `menuFlipPlacements` properties (#10249)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip visual snapshots Pull requests that do not need visual regression testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants