Re: [PATCH] pynfs: fix reporting of ignored tests

["Mkrtchyan, Tigran" <tigran.mkrtchyan@desy.de>]

----- Original Message -----
> From: "Frank Filz" <ffilzlnx@mindspring.com>
> To: "J. Bruce Fields" <bfields@fieldses.org>, "Tigran Mkrtchyan" <tigran.mkrtchyan@desy.de>
> Cc: linux-nfs@vger.kernel.org
> Sent: Monday, October 7, 2013 6:05:08 PM
> Subject: RE: [PATCH] pynfs: fix reporting of ignored tests
> 
> Tigran,
> 
Hi Frank,

> You also modified DSESS9005, should that actually be split into two separate
> test cases?

I was thinking to add one with compound: SEQ, PUTROOTFH, DESTROY_SESSION

Tigran.

> 
> Hmm, looks like a set of 4.1 tests that are not part of all that I need to
> see how they fare against Ganesha.
> 
> Frank
> 
> > -----Original Message-----
> > From: linux-nfs-owner@vger.kernel.org [mailto:linux-nfs-
> > owner@vger.kernel.org] On Behalf Of J. Bruce Fields
> > Sent: Monday, October 7, 2013 8:05 AM
> > To: Tigran Mkrtchyan
> > Cc: linux-nfs@vger.kernel.org
> > Subject: Re: [PATCH] pynfs: fix reporting of ignored tests
> > 
> > On Sat, Oct 05, 2013 at 10:27:15PM +0200, Tigran Mkrtchyan wrote:
> > > currently, ignored tests reported as failure, which makes fixing them
> > > quite hard.
> > 
> > Thanks.  And it's not counting warnings either?  I wonder if printresults
> and
> > xml_printresults should be sharing some code?
> > 
> > --b.
> > 
> > >
> > > Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>
> > > ---
> > >  nfs4.1/testmod.py | 8 +++++++-
> > >  1 file changed, 7 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/nfs4.1/testmod.py b/nfs4.1/testmod.py index
> > > 3d74efe..68855de 100644
> > > --- a/nfs4.1/testmod.py
> > > +++ b/nfs4.1/testmod.py
> > > @@ -439,6 +439,7 @@ def printresults(tests, opts, file=None):
> > >  def xml_printresults(tests, file_name, suite='all'):
> > >      with open(file_name, 'w') as fd:
> > >          failures = 0
> > > +        skipped = 0
> > >          total_time = 0
> > >          doc = xml.dom.minidom.Document()
> > >          testsuite = doc.createElement("testsuite") @@ -455,14 +456,19
> > > @@ def xml_printresults(tests, file_name, suite='all'):
> > >              testcase.setAttribute("time", str(t.time_taken))
> > >
> > >              total_time += t.time_taken
> > > -            if t.result not in (TEST_PASS, TEST_WARN):
> > > +            if t.result == TEST_FAIL:
> > >                  failures += 1
> > >                  failure = doc.createElement("failure")
> > >                  failure.setAttribute("message", t.result.msg)
> > >                  err = doc.createCDATASection(''.join(t.result.tb))
> > >                  failure.appendChild(err)
> > >                  testcase.appendChild(failure)
> > > +            elif t.result == TEST_OMIT:
> > > +                skipped += 1
> > > +                skip = doc.createElement("skipped")
> > > +                testcase.appendChild(skip)
> > >
> > >          testsuite.setAttribute("failures", str(failures))
> > > +        testsuite.setAttribute("skipped", str(skipped))
> > >          testsuite.setAttribute("time", str(total_time))
> > >          fd.write(doc.toprettyxml(indent="  "))
> > > --
> > > 1.8.3.1
> > >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the
> > body of a message to majordomo@vger.kernel.org More majordomo info at
> > http://vger.kernel.org/majordomo-info.html
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>