<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 29, 2016 at 5:58 PM, Xinliang David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">To clarify, it is not 128 iterations, but creating a symbol table with<br>
128 entries -- which is a reasonable size.</blockquote><div><br></div><div>We don't generally test on "realistic" sized inputs in the regression suite. We write targeted tests for functionality. Broad testing is done in the test-suite and other integration level testing.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"> Test coverage wise, it is probably the same as a 3-entry symtab.<br></blockquote><div><br>Then let's use a 3-entry symtab.<br><br>(why 3? Because it tests the boundaries (first and last) and one "normal" case of a non-boundary value - while the boundaries probably aren't interesting in this algorithm, it's cheap enough to just follow that common practice in test case design)<br><br>I'm also curious about the padding parameter - what does it do? Choose how many null characters go between each value? What effect does that have/why is that a tuning parameter? (understanding what it's for can help us choose appropriate test cases/coverage for that functionality)<br><br>(choosing appropriate test cases is a fairly broad area of knowledge, I can't seem to find a canonical reference on this, but "test case design", "choosing test cases", etc gives several results with the sort of discussion here: <a href="http://www.cdf.toronto.edu/~heap/148/F13/Lectures/W4/ChoosingTestCases.html">http://www.cdf.toronto.edu/~heap/148/F13/Lectures/W4/ChoosingTestCases.html</a> - the point being to choose minimal, representative samples from the space of all possible inputs)</div></div></div></div>