Skip to content

Conversation

@carlos-granados
Copy link
Contributor

No description provided.

@carlos-granados carlos-granados force-pushed the refactor-snippets-tests branch 2 times, most recently from 8f1b84c to 4f00aab Compare December 10, 2025 13:14
$this->answerString = $answerString;

$this->options = '--format-settings=\'{"timer": false}\'';
$this->options = '--no-colors --format-settings=\'{"timer": false}\'';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This step overrides any option set for behat previously, so we need to make sure that it uses the no-colors option

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense - but would it be clearer if it didn't override the options (so it behaves consistently with the normal "I run Behat" step? I suppose that might impact on other scenarios that are depending on this behaviour, so we could leave it for now,

Copy link
Contributor

@acoulton acoulton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @carlos-granados one small comment / query but happy with it as-is if you are.

@carlos-granados carlos-granados merged commit deefbc8 into Behat:3.x Dec 10, 2025
43 of 44 checks passed
@carlos-granados carlos-granados deleted the refactor-snippets-tests branch December 10, 2025 14:11
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