Re: [PATCH 02/12] test: utils: Add helpers for automatic numbering of test cases

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 29, 2015 at 14:23:52 -0500, John Ferlan wrote:
> 
> I don't see this being used anywhere in this set of patches...  I assume
> you have some other upcoming patch series that will use it...
> 
> NITs
> 
> On 01/28/2015 05:30 AM, Peter Krempa wrote:
> > Adding or reordering test cases is usually a pain due to static test
> > case names that are then passed to virtTestRun(). To ease the numbering
> > of test cases, this patch adds two simple helpers that generate the test
> > names according to the order they are run. The test name can be
> > configured via the reset function.
> > 
> > This will allow us to freely add test cases in mid of test groups
> 
> s/in mid of/in the middle of/
> 
> > without the need to re-number the rest of test cases.
> > ---
> >  tests/testutils.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++
> >  tests/testutils.h |  3 +++
> >  2 files changed, 49 insertions(+)
> > 
> > diff --git a/tests/testutils.c b/tests/testutils.c
> > index 9a79f98..c7d2615 100644
> > --- a/tests/testutils.c
> > +++ b/tests/testutils.c
> > @@ -986,3 +986,49 @@ virDomainXMLOptionPtr virTestGenericDomainXMLConfInit(void)
> >                                   &virTestGenericPrivateDataCallbacks,
> >                                   NULL);
> >  }
> > +
> > +
> > +static int virtTestCounter;
> > +static char virtTestCounterStr[128];
> > +static char *virtTestCounterPrefixEndOffset;
> > +
> > +
> > +/**
> > + * virtTestCounterReset:
> > + * @prefix: name of the test group
> > + *
> > + * Resets the counter and sets up the test group name to use with
> > + * virtTestCounterNext(). This function is not thread safe.
> > + */
> > +void
> > +virtTestCounterReset(const char *prefix)
> > +{
> > +    virtTestCounter = 0;
> 
> Not that it'd happen, but if the prefix was larger than 128
> characters... ;-)

I can document it if it would help. I wanted to avoid memory allocation
in the testsuite so that this code can't introduce a failure.

A worst case scenario here should be that the test names will be
truncated which will make it less obvious in case of a failure. In such
case we can always bump the size of the string.

> 

Peter

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]