<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">In my latest patch, I went back to <div class=""><br class=""></div><div class=""><blockquote type="cite" class=""><div lang="EN-US" link="#0563C1" vlink="#954F72" class=""><div class="WordSection1"><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span class="" style="font-family: Consolas;">BOOST_AUTO_TEST_SUITE(Security)<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span class="" style="font-family: Consolas;">BOOST_AUTO_TEST_SUITE(Pib)<o:p class=""></o:p></span></div><div style="margin: 0in 0in 0.0001pt; font-size: 11pt; font-family: Calibri, sans-serif;" class=""><span class="" style="font-family: Consolas;">BOOST_AUTO_TEST_SUITE(TestPib)</span></div></div></div></blockquote><div class=""><br class=""></div><div class="">and used the full name of the class “security::Pib” in the test cases to avoid name conflict, I think this could be a general solution, because the name of test suite is alway capitalized, but the namespace is not.</div><div class=""><br class=""></div><div class="">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; border-spacing: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; border-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-stroke-width: 0px;"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Best Regards,</div><div class=""><br class=""></div><div class="">Yingdi </div></div></span></div></span></div>
</div>
<br class=""><div><blockquote type="cite" class=""><div class="">On Sep 16, 2015, at 7:01 AM, Junxiao Shi <<a href="mailto:shijunxiao@email.arizona.edu" class="">shijunxiao@email.arizona.edu</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><meta http-equiv="Content-Type" content="text/html; charset=utf-8" class=""><meta name="Generator" content="Microsoft Word 15 (filtered medium)" class=""><style class=""><!--
/* Font Definitions */
@font-face
{font-family:Wingdings;
panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
{font-family:SimSun;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Consolas;
panose-1:2 11 6 9 2 2 4 3 2 4;}
@font-face
{font-family:"\@SimSun";
panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:#0563C1;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:#954F72;
text-decoration:underline;}
p
{mso-style-priority:99;
margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
/* List Definitions */
@list l0
{mso-list-id:542407464;
mso-list-type:hybrid;
mso-list-template-ids:384317004 67698703 67698691 67698693 67698689 67698691 67698693 67698689 67698691 67698693;}
@list l0:level1
{mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;}
@list l0:level2
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l0:level3
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l0:level4
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l0:level5
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l0:level6
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l0:level7
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l0:level8
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l0:level9
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l1
{mso-list-id:775757141;
mso-list-type:hybrid;
mso-list-template-ids:253647854 67698689 67698691 67698693 67698689 67698691 67698693 67698689 67698691 67698693;}
@list l1:level1
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l1:level2
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l1:level3
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l1:level4
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l1:level5
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l1:level6
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l1:level7
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l1:level8
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l1:level9
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l2
{mso-list-id:1292446187;
mso-list-type:hybrid;
mso-list-template-ids:-2135683586 67698689 67698691 67698693 67698689 67698691 67698693 67698689 67698691 67698693;}
@list l2:level1
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l2:level2
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l2:level3
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l2:level4
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l2:level5
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l2:level6
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
@list l2:level7
{mso-level-number-format:bullet;
mso-level-text:\F0B7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Symbol;}
@list l2:level8
{mso-level-number-format:bullet;
mso-level-text:o;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Courier New";}
@list l2:level9
{mso-level-number-format:bullet;
mso-level-text:\F0A7;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;}
ol
{margin-bottom:0in;}
ul
{margin-bottom:0in;}
--></style><div lang="EN-US" link="#0563C1" vlink="#954F72" class=""><div class="WordSection1"><p class="">Dear folks</p><p class=""><o:p class=""> </o:p></p><p class="">Since 20150901 we have adopted a new naming conventions for unit test suites: <a href="http://redmine.named-data.net/projects/nfd/wiki/UnitTesting/6" class="">http://redmine.named-data.net/projects/nfd/wiki/UnitTesting/6</a> </p><p class="">For example, a test suite for ndn::util::Signal should be named: Util/TestSignal.</p><p class=""><o:p class=""> </o:p></p><p class="">Under this rule, a test suite for ndn::security::pib::Pib should be named: Security/Pib/TestPib.</p><p class="">In Boost.Test, this is written as:</p><p class=""><span style="font-family:Consolas" class="">BOOST_AUTO_TEST_SUITE(Security)<o:p class=""></o:p></span></p><p class=""><span style="font-family:Consolas" class="">BOOST_AUTO_TEST_SUITE(Pib)<o:p class=""></o:p></span></p><p class=""><span style="font-family:Consolas" class="">BOOST_AUTO_TEST_SUITE(TestPib)<o:p class=""></o:p></span></p><p class=""><o:p class=""> </o:p></p><p class="">Each of these BOOST_AUTO_TEST_SUITE macros would declare a class named after the test suite name.</p><p class="">This causes the identifier “Pib” ambiguous: it refers to not only the class under test (ndn::security::pib::Pib), but also the second level test suite.</p><p class=""><o:p class=""> </o:p></p><p class="">Yingdi gave two proposals:</p><p style="margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo3" class=""><!--[if !supportLists]--><span style="mso-list:Ignore" class="">1.<span style="font:7.0pt "Times New Roman"" class=""> </span></span><!--[endif]-->name the test suite as Security/PibDir/TestPib</p><p style="margin-left:.5in;text-indent:-.25in;mso-list:l0 level1 lfo3" class=""><!--[if !supportLists]--><span style="mso-list:Ignore" class="">2.<span style="font:7.0pt "Times New Roman"" class=""> </span></span><!--[endif]-->name the test suite as security/pib/TestPib</p><p class=""><o:p class=""> </o:p></p><p class="">My proposal is:</p><p style="margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo2" class=""><!--[if !supportLists]--><span style="font-family:Symbol" class=""><span style="mso-list:Ignore" class="">·<span style="font:7.0pt "Times New Roman"" class=""> </span></span></span><!--[endif]-->name the test suite as Security/TestPib/TestPib</p><p style="margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo2" class=""><!--[if !supportLists]--><span style="font-family:Symbol" class=""><span style="mso-list:Ignore" class="">·<span style="font:7.0pt "Times New Roman"" class=""> </span></span></span><!--[endif]-->due to the existence of Pib class, all test suites for ndn::security::pib::* should have its second level named “TestPib” rather than “Pib”</p><p class=""><o:p class=""> </o:p></p><p class="">My reason is:</p><p style="margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo2" class=""><!--[if !supportLists]--><span style="font-family:Symbol" class=""><span style="mso-list:Ignore" class="">·<span style="font:7.0pt "Times New Roman"" class=""> </span></span></span><!--[endif]-->Yingdi’s second proposal resolves the name conflict between test suite name and class name, but it creates another conflict: “pib” refers to both the test suite and the namespace ndn::security::pib.</p><p style="margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo2" class=""><!--[if !supportLists]--><span style="font-family:Symbol" class=""><span style="mso-list:Ignore" class="">·<span style="font:7.0pt "Times New Roman"" class=""> </span></span></span><!--[endif]-->Although Yingdi’s first proposal prevents the name conflict, it’s using two different rules to resolve the conflict: (a) append “Dir” (b) prepend “Test”. The purpose for prepending “Test” to the lowest-level test suite name is to resolve the name conflict, so we should reuse the same rule for resolving all conflicts.</p><p style="margin-left:.5in;text-indent:-.25in;mso-list:l2 level1 lfo2" class=""><!--[if !supportLists]--><span style="font-family:Symbol" class=""><span style="mso-list:Ignore" class="">·<span style="font:7.0pt "Times New Roman"" class=""> </span></span></span><!--[endif]-->Having “TestPib” on two levels should be fine, because this class name is never referred to in the testing code.</p><p class=""><o:p class=""> </o:p></p><p class=""><o:p class=""> </o:p></p><p class="">Yours, Junxiao</p></div></div>_______________________________________________<br class="">Nfd-dev mailing list<br class=""><a href="mailto:Nfd-dev@lists.cs.ucla.edu" class="">Nfd-dev@lists.cs.ucla.edu</a><br class="">http://www.lists.cs.ucla.edu/mailman/listinfo/nfd-dev<br class=""></div></blockquote></div><br class=""></div></body></html>