List:Internals« Previous MessageNext Message »
From:Milos Prodanovic Date:July 27 2007 10:31am
Subject:Re: IPV6 data type patch
View as plain text  
Hi Chad,


Thanks for reply and useful suggestions,

I'll work on partial code test (data handling functions), and on
examples/tests of usage, error throw/catching, on the platform that I
have (x86, intel).

I'll post tests/examples to this thread as soon as I finish them.


Regards

Milos

On 7/27/07, Chad MILLER <cmiller@stripped> wrote:
> Hi Milos.
>
> On 17 Jul 2007, at 10:05, Milos Prodanovic wrote:
> > It would be nice to make some functionality tests, and I'm not sure
> > that I can make
> > tests of all possible 'use cases', but I'm glad to do tests if you
> > have some crucial test
> > that could show usability.
>
> Please do make tests.  For every check-in of code, we verify on every
> platform we support that the entire regression suite passes its
> tests.  Any patch is incomplete without tests that show its
> functionality and give proof that it's not broken accidentally in the
> future by someone else.  As the author of the code, you're better
> equipped to write its tests than anyone else.
>
> We also verify that all lines of code are covered by the test suite,
> so the tests should exercise every kind of branch in the tests.  Your
> test should intentionally create errors, and catch them.  It should
> show that the freaky edge cases work.  It should use new
> functionality in a greater context to show it works well with
> others.  It will receive a lot of attention, and serve as examples
> and reference for the documentation team to use when describing the
> functionality in the manual.
>
> In short, the tests are at least as important as the code.
>
> - chad
>
>
> After "make", "{ cd mysql-test; ./mtr
> $optional_specific_test_name; }".  Write new queries & directives at
> "mysql-test/t/${test_name}.test" and write the output at "mysql-test/
> r/${test_name}.result".  I suggest "type_net_addresses" is a good
> test name.  You may use "-record" to make the result, but don't
> blindly do it; recording a wrong result is much worse than not having
> a result.
>
> --
> Chad Miller, Software Developer                         chad@stripped
> MySQL Inc., www.mysql.com
> Orlando, Florida, USA                                13-20z,  UTC-0400
> Office: +1 408 213 6740                         sip:6740@stripped
>
>
>
>
Thread
IPV6 data type patchMilos Prodanovic16 Jul
  • Re: IPV6 data type patchStewart Smith16 Jul
    • Re: IPV6 data type patchDavid Li16 Jul
      • Re: IPV6 data type patchMilos Prodanovic17 Jul
        • Re: IPV6 data type patchStewart Smith18 Jul
        • Re: IPV6 data type patchChad MILLER27 Jul
          • Re: IPV6 data type patchMilos Prodanovic27 Jul