<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 17, 2017 at 6:08 PM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
  Comments would be a new feature so belong in a new branch, so create a branch off of master, call it say<br>
scott/test-harness-comments then after you tested it (by adding at least one comment to a current example) and then make a pull request and we can get it into master very quickly.</blockquote><div><br></div><div>This works</div><div><br></div><div><div>--- a/config/testparse.py</div><div>+++ b/config/testparse.py</div><div>@@ -98,6 +98,7 @@ def parseTest(testStr,srcfile):</div><div>   subdict={}</div><div>   for line in striptest.split("\n"):</div><div>     if not line.strip(): continue</div><div>+    if len(line.split(":")) < 2: continue</div><div>     var=line.split(":")[0].strip()</div><div>     val=line.split(":")[1].strip()</div><div>     # Start by seeing if we are in a subtest</div></div><div><br></div><div>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-HOEnZb"><font color="#888888"><br>
   Barry<br>
</font></span><div class="gmail-HOEnZb"><div class="gmail-h5"><br>
> On Jan 17, 2017, at 5:29 PM, Scott Kruger <<a href="mailto:kruger@txcorp.com">kruger@txcorp.com</a>> wrote:<br>
><br>
><br>
><br>
><br>
> The "This" is not at the same indentation level as the "test:"<br>
><br>
> According the YAML standard, comments begin with #.<br>
> I did not implement comments.<br>
><br>
> Where do you want the fix pushed?<br>
><br>
> Scott<br>
><br>
><br>
> On 1/17/17 4:19 PM, Barry Smith wrote:<br>
>><br>
>>   I don't see any difference in the spacing between the two cases? We definitely don't want white space or lack of white space to matter in the definitions, that is too hard to manage.<br>
>><br>
>><br>
>>> On Jan 17, 2017, at 5:04 PM, Scott Kruger <<a href="mailto:kruger@txcorp.com">kruger@txcorp.com</a>> wrote:<br>
>>><br>
>>><br>
>>><br>
>>> It has to do with the spacing.  Looking at src/sys/examples/test/ex1.c<br>
>>><br>
>>> This works:<br>
>>> /*TEST<br>
>>><br>
>>>  This test does foo<br>
>>>  test:<br>
>>>     filter: egrep "(PETSC ERROR)" | egrep "(main|CreateError|Error Created)" | cut -f1,2,3,4,5,6 -d" "<br>
>>><br>
>>><br>
>>> TEST*/<br>
>>><br>
>>> which I can see by doing this:<br>
>>> ------------------------------<wbr>------------------<br>
>>> gabrielle 81: ../../../../config/testparse.<wbr>py -t ex1.c -v 1<br>
>>><br>
>>><br>
>>> ex1.c<br>
>>>  runex1<br>
>>>     filter: egrep "(PETSC ERROR)" | egrep "(main|CreateError|Error Created)" | cut -f1,2,3,4,5,6 -d" "<br>
>>> ------------------------------<wbr>------------------<br>
>>><br>
>>> but this does not:<br>
>>><br>
>>> /*TEST<br>
>>><br>
>>> This test does foo<br>
>>>  test:<br>
>>>     filter: egrep "(PETSC ERROR)" | egrep "(main|CreateError|Error Created)" | cut -f1,2,3,4,5,6 -d" "<br>
>>><br>
>>><br>
>>> TEST*/<br>
>>><br>
>>> I would suggest that using # as a comment delimiter would be the best approach to enabling comments.<br>
>>><br>
>>> Scott<br>
>>><br>
>>><br>
>>> On 1/17/17 3:53 PM, Matthew Knepley wrote:<br>
>>>> On Tue, Jan 17, 2017 at 4:31 PM, Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a><br>
>>>> <mailto:<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>>> wrote:<br>
>>>><br>
>>>>   On Tue, 17 Jan 2017, Matthew Knepley wrote:<br>
>>>><br>
>>>>   > Also, how do I put comments inside these blocks to tell me what the test is<br>
>>>>   > about?<br>
>>>><br>
>>>>   Looks like everything other than the define format is ignored..<br>
>>>><br>
>>>><br>
>>>> It does not look that way to me:<br>
>>>><br>
>>>> /usr/bin/python ./config/gmakegentest.py --petsc-arch=arch-c-exodus-<wbr>master<br>
>>>> Traceback (most recent call last):<br>
>>>> File "./config/gmakegentest.py", line 733, in <module><br>
>>>>   main(petsc_arch=opts.petsc_<wbr>arch, output=opts.output,<br>
>>>> verbose=opts.verbose, single_ex=opts.single_<wbr>executable)<br>
>>>> File "./config/gmakegentest.py", line 718, in main<br>
>>>><br>
>>>> dataDict=pEx.walktree(os.path.<wbr>join(pEx.petsc_dir,'src'),<wbr>action="genPetscTests")<br>
>>>> File "./config/gmakegentest.py", line 613, in walktree<br>
>>>>   eval("self."+action+"(root,<wbr>dirs,files,dataDict)")<br>
>>>> File "<string>", line 1, in <module><br>
>>>> File "./config/gmakegentest.py", line 593, in genPetscTests<br>
>>>>   dataDict[root].update(<wbr>testparse.parseTestFile(<wbr>fullex))<br>
>>>> File "/PETSc3/petsc/petsc-dev/<wbr>config/testparse.py", line 175, in<br>
>>>> parseTestFile<br>
>>>>   testDict[basename]=parseTests(<wbr>testString,srcfile)<br>
>>>> File "/PETSc3/petsc/petsc-dev/<wbr>config/testparse.py", line 140, in<br>
>>>> parseTests<br>
>>>>   testname,subdict=parseTest(<wbr>test,srcfile)<br>
>>>> File "/PETSc3/petsc/petsc-dev/<wbr>config/testparse.py", line 102, in parseTest<br>
>>>>   val=line.split(":")[1].strip()<br>
>>>> IndexError: list index out of range<br>
>>>><br>
>>>><br>
>>>>  Matt<br>
>>>><br>
>>>><br>
>>>><br>
>>>>   Satish<br>
>>>><br>
>>>><br>
>>>><br>
>>>><br>
>>>> --<br>
>>>> What most experimenters take for granted before they begin their<br>
>>>> experiments is infinitely more interesting than any results to which<br>
>>>> their experiments lead.<br>
>>>> -- Norbert Wiener<br>
>>><br>
>>> --<br>
>>> Tech-X Corporation               <a href="mailto:kruger@txcorp.com">kruger@txcorp.com</a><br>
>>> 5621 Arapahoe Ave, Suite A       Phone: <a href="tel:%28720%29%20974-1841" value="+17209741841">(720) 974-1841</a><br>
>>> Boulder, CO 80303                Fax:   <a href="tel:%28303%29%20448-7756" value="+13034487756">(303) 448-7756</a><br>
>><br>
><br>
> --<br>
> Tech-X Corporation               <a href="mailto:kruger@txcorp.com">kruger@txcorp.com</a><br>
> 5621 Arapahoe Ave, Suite A       Phone: <a href="tel:%28720%29%20974-1841" value="+17209741841">(720) 974-1841</a><br>
> Boulder, CO 80303                Fax:   <a href="tel:%28303%29%20448-7756" value="+13034487756">(303) 448-7756</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener</div>
</div></div>