Making WordPress.org

Opened 3 years ago

Last modified 3 years ago

#5949 new enhancement

Add a 'needs-e2e-tests' keyword to trac

Reported by: talldanwp's profile talldanwp Owned by:
Milestone: Priority: normal
Component: Trac Keywords: 2nd-opinion 3rd-opinion-needed
Cc:

Description (last modified by SergeyBiryukov)

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)

#1 follow-up: @dd32
3 years ago

I like needs-tests better than two separate labels, but I can see it being confused with needs-testing.

Isn't an e2e just a visual unit test though? ;)

Perhaps needs-testing would work with needs-tests better if the former was renamed to testing-required too?

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.

This ticket was mentioned in Slack in #core-test by talldanwp. View the logs.


3 years ago

#3 @talldanwp
3 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 @dd32
3 years ago

  • Keywords 3rd-opinion-needed added

Maybe we could add a '3rd-opinion' keyword too? ;)

Huh, you don't see that option? /s

#5 @SergeyBiryukov
3 years ago

  • Description modified (diff)

#6 in reply to: ↑ 1 @SergeyBiryukov
3 years ago

Replying to dd32:

Perhaps needs-testing would work with needs-tests better if the former was renamed to testing-required too?

Just a few more options to consider:

  • needs-unit-testsneeds-automated-tests
  • needs-testingneeds-manual-testing

This ticket was mentioned in Slack in #meta by tellyworth. View the logs.


3 years ago

Note: See TracTickets for help on using tickets.