<div dir="ltr"><div><div>Hi Nick<br><br></div><div>I see no mistake in Cortez's flagging. He can <a href="https://github.com/yoursunny/ndn-cxx-code-style/blob/4d905504731ac1b8c65ad6c6ae4a845b6ea02266/rules0.js#L97">understand</a> both NFD's x.t.cpp and NLSR's test-x.cpp test file names.</div><div><br></div><div>As an example, the mistake in <a href="https://gerrit.named-data.net/#/c/4132/9/tests/test-name-prefix-table.cpp@24">https://gerrit.named-data.net/#/c/4132/9/tests/test-name-prefix-table.cpp@24</a> is that the include line for the unit under test should appear first. The line "<span style="font-family:monospace,monospace">#include "route/name-prefix-table.hpp"</span>" should appear before other #include lines.</div><div><br></div><div>The reason of requiring this include to appear first is to ensure the header can compile on its own. If the header omits any necessary includes, the build of name-prefix-table.cpp or test-name-prefix-table.cpp will break. Thus, this rule ensures that build breaks show up first for the people working on these files, not for innocent people in other packages.<br></div><div><br></div>Yours, Junxiao<br></div>Guardian of Mr Anthony M Cortez<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 22, 2017 at 11:47 AM, Nick Gordon <span dir="ltr"><<a href="mailto:nmgordon@memphis.edu" target="_blank">nmgordon@memphis.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">For NLSR, we have a particular naming convention with unit tests that differs slightly from NFD and ndn-cxx, which is test-*.cpp, as opposed to *.t.cpp. Our friendly neighborhood code robot, Mr. Cortez, repeatedly insists that we include the corresponding header, which test files usually do not have. Since the naming of test files is not mandated by the code styles, could the code style robot be extended to not flag those? It's tiring to remove his score, explain myself, and give a +1 CS score myself. Or, we should formalize a test file naming convention to have a reason to decline these kinds of requests.<br>
<br>
-Nick<br>
<br>
______________________________<wbr>_________________<br>
Nfd-dev mailing list<br>
<a href="mailto:Nfd-dev@lists.cs.ucla.edu" target="_blank">Nfd-dev@lists.cs.ucla.edu</a><br>
<a href="http://www.lists.cs.ucla.edu/mailman/listinfo/nfd-dev" rel="noreferrer" target="_blank">http://www.lists.cs.ucla.edu/m<wbr>ailman/listinfo/nfd-dev</a><br>
</blockquote></div><br></div>