Skip to content

Ensure end to end tests will continue passing when multiple branches feature is released - #333

Merged
lfdebrux merged 1 commit into
mainfrom
ldeb-multiple-branches
Sep 16, 2026
Merged

lfdebrux merged 1 commit into
mainfrom
ldeb-multiple-branches

Conversation

@lfdebrux

Copy link
Copy Markdown
Contributor

What problem does this pull request solve?

Trello card: https://trello.com/c/TnXaNTsk/3245-fix-existing-end-to-end-tests-that-use-routing

When the multiple branches feature is released generally in forms-admin, the way to add or edit routes for a form will change.

This PR updates the code for creating a test form in the end to end specs to work with either the new "Edit question routes" page to add routes or the old journey when the multiple branches feature is not enabled, depending on what buttons are present on the "Add and edit your questions" page.

I tested this locally by enabling the multiple_branches feature flag for the end to end tests group.

Things to consider when reviewing

  • Ensure that you consider the wider context.
  • Does it work when run on your machine?
  • Is it clear what the code is doing?
  • Do the commit messages explain why the changes were made?
  • Has all relevant documentation been updated?

When the multiple branches feature is released generally in forms-admin,
the way to add or edit routes for a form will change.

This commit updates the code for creating a test form in the end to end
specs to work with either the new "Edit question routes" page to add
routes or the old journey when the multiple branches feature is not
enabled, depending on what buttons are present on the "Add and edit your
questions" page.
@lfdebrux
lfdebrux force-pushed the ldeb-multiple-branches branch from 7e6ba4b to 1a52ff7 Compare September 16, 2026 12:55
@lfdebrux
lfdebrux added this pull request to the merge queue Sep 16, 2026
Merged via the queue into main with commit f3e1df2 Sep 16, 2026
2 checks passed
@lfdebrux
lfdebrux deleted the ldeb-multiple-branches branch September 16, 2026 14:09
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