Skip to content

enzyme -> RTL: convert the Wizard component suite - #473

Merged
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:feature/rtl-components-wizard
Jun 18, 2026
Merged

enzyme -> RTL: convert the Wizard component suite#473
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:feature/rtl-components-wizard

Conversation

@blaipr

@blaipr blaipr commented Jun 16, 2026

Copy link
Copy Markdown
Contributor
SUMMARY

Converts the Wizard component test suite (components/Wizard) from enzyme to React Testing Library, continuing the enzyme → RTL migration (components, one directory per PR).

Wizard.test.js migrated off enzyme onto renderWithContexts: the PF Wizard step content/nav is asserted in the rendered DOM (the step label renders in nav + content, so the content paragraph is scoped). Behaviour and assertions are preserved.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • UI
ADDITIONAL INFORMATION

npm test for components/Wizard: all passing. ESLint clean (--no-ignore). No production code changed — test-only.

Migrate components/Wizard off enzyme onto renderWithContexts (React Testing
Library). The PF Wizard step content is asserted in its body portal;
behaviour and assertions are preserved.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Migrates the Wizard component’s unit test from Enzyme to React Testing Library (RTL), aligning with the ongoing UI test migration strategy and using the shared renderWithContexts helper for consistent provider setup.

Changes:

  • Replaced Enzyme mount usage with RTL renderWithContexts for rendering.
  • Updated assertions to validate Wizard step content and the step’s navigation entry via DOM queries.

@cigamit
cigamit merged commit 44a62b0 into ctrliq:main Jun 18, 2026
@cigamit cigamit self-assigned this Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants