<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 25, 2015 at 9:42 AM Amar Takhar <<a href="mailto:verm@darkbeer.org">verm@darkbeer.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 2015-11-25 17:35 +0000, Mark Atwood wrote:<br>
> Ah, I misunderstood.<br>
><br>
> So, the state we are in now is:<br>
> - All the tests run?<br>
> - All the test pass?<br>
<br>
When you say "all" yes all the ones converted run and pass -- with exception to a<br>
few which I will get back to later they've been commented out.<br></blockquote><div><br></div><div>Ok.  Why were they commented out?  Did we comment them out, or were they commented out in NTP Classic?</div><div> </div><div>|> Why a new test suite?  What are you converting the current one into?</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The old test suite used Google Test which is C++.  The development model of the<br>
project changed to one incompatible with the project.  Right now I am converting<br>
it all to C using Unity.<br></blockquote><div><br></div><div>Ok.  Expand that a bit for me, please.   The development model of the Google Test project changed to one incompatible with the NTPsec project?  Or the development model of the NTPsec project changed to one incompatible with the Google Test project?</div><div><br></div><div>I am inclined agree to want the tests in standard C99 C instead of C++, but also it needs to bring a value-add, such as making it easier for non-core contributors to write new tests and run them.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The 'new' test suite involves re-arranging it into a more suitable format to<br>
edit and test.  We need to be setup better for regression testing as well.<br>
<br>
It's complicated since the C++ code uses classes and function overloading which<br>
is going to require a lot of refactoring.  I'm leaving these ones to the end<br>
<br>
>From the old C++ tests it's about 60% complete conversion wise.  I'll hopefully<br>
have this done 'soon' barring any strangeness which I've already hit..<br> <br></blockquote><div><br></div><div>Ok.  Do please email to <a href="mailto:devel@ntpsec.org">devel@ntpsec.org</a> each time you convert another test, and what the test was that you converted.</div><div><br></div><div>I'm pushing really hard on the tests because getting all the tests running is a good marker for 0.9.1, and also once we have that working, I'm going to ask you to go back and start the work to waf-ify the fork point and forward, so we can get bisect to work.</div><div><br></div><div>Thanks!</div><div><br></div><div>..m </div></div></div>