[wp-trac] [WordPress Trac] #17749: Faster and phpunit runner-compatible unit tests

WordPress Trac wp-trac at lists.automattic.com
Tue Dec 6 10:57:43 UTC 2011


#17749: Faster and phpunit runner-compatible unit tests
------------------------------+-------------------------
 Reporter:  nbachiyski        |       Owner:  nbachiyski
     Type:  feature request   |      Status:  closed
 Priority:  normal            |   Milestone:  3.3
Component:  Unit Tests        |     Version:
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |
------------------------------+-------------------------

Comment (by nacin):

 Replying to [comment:34 mrtorrent]:
 > I'm really disappointed that the first contributions I've made to
 WordPress have been met with such reluctance, when I thought I was just
 chipping in a few small, non-controversial fixes that would help up the
 standard of the code.

 I'm sorry your first contribution fell by the wayside. We'd love to see
 more of them. But once we hit release candidate stage, as dd32 has
 explained, we have strict rules about what can/should and cannot/shouldn't
 be committed to core. I opened #19456 which will be committed once
 development for 3.4 opens.

 > It seems like a fear of breaking anything, which comes from a lack of
 thorough test coverage to give people peace of mind when they make a
 change, and which would make WordPress a more robust piece of software.

 > But that is being prevented by bad code like this which has been hanging
 around for a long time and was written by people who didn't know better at
 the time.

 I think there are plenty of examples of bad code in WordPress that bind
 our hands for future development, but this line is not one of them.

 > Manual testing and clinging to an aged codebase is not healthy.

 You're preaching to the choir.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/17749#comment:36>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list