[buddypress-trac] [BuddyPress Trac] #6223: Remove failing automated tests from the suite

buddypress-trac noreply at wordpress.org
Fri Feb 13 16:03:29 UTC 2015

#6223: Remove failing automated tests from the suite
 Reporter:  boonebgorges        |      Owner:
     Type:  defect (bug)        |     Status:  new
 Priority:  normal              |  Milestone:  2.3
Component:  Tools - Unit Tests  |    Version:
 Severity:  normal              |   Keywords:
 See https://core.trac.wordpress.org/ticket/30284

 A normal run of `phpunit` currently skips test that are annotated with
 `@ticket BP12345` if ticket 12345 is open. This practice has some negative
 consequences, especially because it forces a number of *passing* tests to
 be skipped. (Also, this behavior requires a ping to Trac, which causes
 issues like those described in #5910 [9053].)

 We should remove failing tests and stop skipping them when running
 `phpunit`. We should also agree to stop committing failing tests to the
 repo. Commit only those tests that pass, and leave failing tests as
 patches on the corresponding tickets. Those tests will then be committed
 with their corresponding fixes.

Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6223>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac

More information about the buddypress-trac mailing list