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

Re: [Xen-devel] [PATCH RFC V2 09/10] libxl/gentest.py: test JSON parser



On Tue, 2014-04-22 at 16:49 +0100, Wei Liu wrote:
> On Tue, Apr 22, 2014 at 04:47:22PM +0100, Ian Campbell wrote:
> > On Thu, 2014-04-17 at 12:13 +0100, Wei Liu wrote:
> > > @@ -240,20 +241,36 @@ int main(int argc, char **argv)
> > >          exit(1);
> > >      }
> > >  """)
> > > -    f.write("    printf(\"Testing TYPE_to_json()\\n\");\n")
> > > +    f.write("    printf(\"Testing TYPE_to/from_json()\\n\");\n")
> > >      f.write("    printf(\"----------------------\\n\");\n")
> > >      f.write("    printf(\"\\n\");\n")
> > >      for ty in [t for t in types if t.json_gen_fn is not None]:
> > >          arg = ty.typename + "_val"
> > >          f.write("    %s_rand_init(%s);\n" % (ty.typename, \
> > >              ty.pass_arg(arg, isref=False, passby=idl.PASS_BY_REFERENCE)))
> > > +        if not isinstance(ty, idl.Enumeration):
> > > +            f.write("    %s_init(%s_new);\n" % (ty.typename, \
> > > +                ty.pass_arg(arg, isref=False, 
> > > passby=idl.PASS_BY_REFERENCE)))
> > >          f.write("    s = %s_to_json(ctx, %s);\n" % \
> > >                  (ty.typename, ty.pass_arg(arg, isref=False)))
> > >          f.write("    printf(\"%%s: %%s\\n\", \"%s\", s);\n" % 
> > > ty.typename)
> > >          f.write("    if (s == NULL) abort();\n")
> > > +        f.write("    rc = %s_from_json(ctx, &%s_val_new, s);\n" % \
> > > +                (ty.typename, ty.typename))
> > > +        f.write("    if (rc) abort();\n")
> > > +        f.write("    new_s = %s_to_json(ctx, %s_new);\n" % \
> > > +                (ty.typename, ty.pass_arg(arg, isref=False)))
> > > +        f.write("    if (new_s == NULL) abort();\n")
> > > +        f.write("    if (strcmp(s, new_s)) {\n")
> > > +        f.write("        printf(\"Huh? Regenerated string different from 
> > > original string.\\n\");\n")
> > > +        f.write("        printf(\"Regenerated string: %s\\n\", 
> > > new_s);\n")
> > 
> > This could use some hint as to which struct went wrong. Perhaps
> > including the original string too for comparison?
> > 
> > 
> 
> It's already printed a few line above. :-)

Ah, I missed it in the noise. Great.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.