Opened 4 years ago
Last modified 4 years ago
#5949 new enhancement
Add a 'needs-e2e-tests' keyword to trac
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Component: | Trac | Keywords: | 2nd-opinion 3rd-opinion-needed |
| Cc: |
Description (last modified by )
Trac currently has a very specific 'needs-unit-tests' keyword, and it could be useful to have the same to encourage end to end tests.
The simplest solution might be to add a 'needs-e2e-tests' keyword, but also open to other ideas.
Trac does seem to have a lot of keywords, and I also noticed there are requests for more (#5634), so an alternative would be to rename 'needs-unit-tests' a more generic 'needs-tests'.
Change History (7)
This ticket was mentioned in Slack in #core-test by talldanwp. View the logs.
4 years ago
#3
@
4 years ago
- Keywords 2nd-opinion added
I dropped a message about this ticket in the core-test slack channel, so hopefully there will be some extra opinions added soon. Maybe we could add a '3rd-opinion' keyword too? ;)
#4
@
4 years ago
- Keywords 3rd-opinion-needed added
Maybe we could add a '3rd-opinion' keyword too? ;)
Huh, you don't see that option? /s
#6
in reply to:
↑ 1
@
4 years ago
Replying to dd32:
Perhaps
needs-testingwould work withneeds-testsbetter if the former was renamed totesting-requiredtoo?
Just a few more options to consider:
needs-unit-tests→needs-automated-testsneeds-testing→needs-manual-testing
I like
needs-testsbetter than two separate labels, but I can see it being confused withneeds-testing.Isn't an
e2ejust a visual unit test though? ;)Perhaps
needs-testingwould work withneeds-testsbetter if the former was renamed totesting-requiredtoo?Adding keywords isn't an issue (other than the list growing), renaming them isn't as straight forward though, as it requires a string-replace on the Trac DB.