Skip to content

Fixes to v3 tests#75

Merged
fritzbrand merged 7 commits into
mainfrom
fixes-to-v3-tests
Aug 26, 2025
Merged

Fixes to v3 tests#75
fritzbrand merged 7 commits into
mainfrom
fixes-to-v3-tests

Conversation

@fritzbrand

@fritzbrand fritzbrand commented Aug 25, 2025

Copy link
Copy Markdown

Some minor updates to my previous PR.

  • Updated to use Flow_tester 1.5.1
  • Added a test template manually to all Onboarding test files that use the onboarding.csv file
  • Removed asserts on buttons during template send tests. Will need to rework the mocking of the buttons at some point in future. We need to standardise how we use buttons in journey's & tests, we currently have multiple implementations

Comment thread Onboarding/QA/tests/forms_test.exs Outdated
setup [:setup_flow]

describe "DMA Form" do
# TODO: I don't see the point of this test file, as 'dma-form_test.ex' already tests form functionality in much more depth

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.

Was this maybe the original test file from the base CMS Journeys that we had? (we usually copy them as a base and then edit them). I'm happy to delete it rather since we have the test coverage in another file.

end

describe "checkpoints" do
@tag :thisone

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.

We can probably remove this tag?

@fritzbrand
fritzbrand merged commit acc3e47 into main Aug 26, 2025
2 checks passed
@fritzbrand
fritzbrand deleted the fixes-to-v3-tests branch August 26, 2025 10:40
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