Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misc cucumber tweaks #239

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Misc cucumber tweaks #239

merged 1 commit into from
Dec 19, 2024

Conversation

mike182uk
Copy link
Member

@mike182uk mike182uk commented Dec 19, 2024

no refs

Misc cucumber tweaks:

  • Use docker compose run instead of docker compose up to get a TTY for live updating of progress + colorization of output. Also removes the annoying service name prefix from the output (making it easier to copy/paste undefined snippets)
    CleanShot 2024-12-19 at 11 51 29@2x
  • Use progress as the formatter so we can see that things are happening
  • Use JS based config instead of cmd line args
  • Added failFast to the config so we can stop on the first failure
  • Use cucumber-js bin directly to prevent double yarn run output at the end of the run

no refs

Misc cucumber tweaks:

- Use `docker compose run` instead of `docker compose up` to get a `TTY` for
live updating of progress + colorization of output. Also removes the annoying
service name prefix from the output
- Use `progress` as the formatter so we can see that things are happening
- Use JS based config instead of cmd line args for cucumber.js
- Added `failFast` to the config so we can stop on the first failure
@mike182uk mike182uk force-pushed the misc-cucumber-tweaks branch from 1f7b272 to 4f84c7f Compare December 19, 2024 11:52
@allouis allouis merged commit 46912ed into main Dec 19, 2024
2 checks passed
@allouis allouis deleted the misc-cucumber-tweaks branch December 19, 2024 12:43
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