[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