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

[buddypress-trac]

#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