patternfly / patternfly/patternfly-react

Add additional Wizard tests for dynamic steps

Open
#10,781 5 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

PF Team Stale
Dominant language
TypeScript
Stars
862
Forks
392
Avg merge
4d 8h
Merged PRs (30d)
9

Description

Describe the enhancement or change
Followup to https://github.com/patternfly/patternfly-react/pull/10779 and https://github.com/patternfly/patternfly-react/pull/10748. Would be good to try and add tests for the behavior shown in these PRs to ensure as little breaks as possible in the future. The behavior includes dynamic steps working as intended (10748) as well as ensuring the Wizard doesn't unexpectedly reset (10779)

Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?

Any other information?


Jira Issue: PF-43

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing PRs 10779 and 10748, then locate the existing Wizard test suite and its dynamic-step coverage. Add tests for dynamic steps and for preventing unexpected Wizard resets, and run the relevant Wizard tests to verify both behaviors.

Written by the indexing model from the issue text.

Assessment

Tech stack
react, typescript
Domain
frontend, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.