Re: [PEAR-QA] Taking over Net_DNS
[email protected] ("Daniel O'Connor")
| Newsgroups | php.pear.qa |
|---|---|
| Message-ID | <[email protected]> |
Oops! Try: clockwerx@clockwerx-desktop:~/pear/Net_DNS/trunk$ php tests/AllTests.php On Tue, Apr 13, 2010 at 4:03 PM, Daniel O'Connor <[email protected]>wrote: > Oneeee more round of picking on! > Re: unit tests - well; it's never too early to start some coverage. > I might put on my QA hat and make some test skeletons. > > At the moment, I'll apply the patch in its current form + some minor fixes > to make it a bit more testable. > > > > > On Tue, Apr 13, 2010 at 2:56 PM, Ian Pye <[email protected]> wrote: > >> Thanks for the encouragement. I re-submitted a patch, which removes >> some of the most egregious faults. I don't see any other Unit tests in >> the NET_dns package -- am I not looking in the right place? >> >> Ian >> >> On Sun, Apr 11, 2010 at 6:02 PM, Daniel O'Connor >> <[email protected]> wrote: >> > >> > >> > On Thu, Apr 8, 2010 at 8:57 AM, Matthew Fonda <[email protected]> >> > wrote: >> >> >> >> On Mon, Apr 5, 2010 at 12:07 PM, Ian Pye <[email protected]> wrote: >> >> >> >> > Hey, >> >> > >> >> > Here you go: https://pear.php.net/bugs/bug.php?id=11665 >> >> >> >> >> >> Hi Ian, >> >> >> >> Please ensure your patch adheres to the PEAR coding standards. >> >> See http://pear.php.net/manual/en/standards.php >> >> >> > >> > Just to expand on that - we're about to pick on you a lot re coding >> > standards and some of the approach you've taken when implementing your >> > class. >> > We're not being nasty; we're just looking for a few common traits that >> lead >> > to maintainable code. >> > >> > http://www.youtube.com/watch?v=wEhu57pih5w is a great watch to >> understand >> > why we're on about no-work-in-constructor, unit testings, why we think >> > global state is bad, etc. >> > >> > Please do give it another shot, we're keen to have patches :) >> > -Daniel >> > >> > >