Split Wizard up into testable components #122
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As the Wizard is a core part of onboarding new users to Kusk and therefore it should be well written and tested to increase confidence when releasing it.
This PR splits the wizard up into smaller components that we can test in isolation / mock where the real thing isn't required
Changes
flowAmbassador
. This separates concerns as now the wizard doesn't need to care about what flow to run, it hands that off to another component.Checklist