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.

Sign up for GitHub

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

Jump to bottom

review keyword chain readability in test scenarios #1386

Open
rdeltour opened this issue Dec 1, 2022 · 0 comments
Open

review keyword chain readability in test scenarios #1386

rdeltour opened this issue Dec 1, 2022 · 0 comments
Assignees
@rdeltour
Labels
newcomers friendly 😊 This issue can be picked up by newcomers to the project! priority: low To be processed when time allows status: ready for implem The issue is ready to be implemented type: tests The issue is related to the test suite

Comments

@rdeltour
Copy link
Member

rdeltour commented Dec 1, 2022

The removal of some errors/warning means some scenarios that were previously written as "When … Then … And …" now read "When … And …", which would be more understandable as "When … Then …".

This is mostly cosmetics (since keywords can be used interchangeably), but would improve test readability.

We need to make a pass over all the tests to fix that wherever that happens.
@rdeltour rdeltour added priority: low To be processed when time allows status: ready for implem The issue is ready to be implemented type: tests The issue is related to the test suite newcomers friendly 😊 This issue can be picked up by newcomers to the project! labels Dec 1, 2022
@rdeltour rdeltour self-assigned this Dec 1, 2022
This was referenced Dec 1, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdeltour rdeltour

Labels
newcomers friendly 😊 This issue can be picked up by newcomers to the project! priority: low To be processed when time allows status: ready for implem The issue is ready to be implemented type: tests The issue is related to the test suite
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rdeltour