[icedtea-web] reviewer needed - added engine for lunching reproducers
Omair Majid
omajid at redhat.com
Fri May 6 16:32:10 PDT 2011
Hi Jiri,
On 05/05/2011 12:05 PM, Jiri Vanek wrote:
> Makefile - automated compile and copy for most of reproducers, with
> possibility to add more complex reproducers which will take care of
> compiling themselves (I have in this way also "repaired" your
> reproducers you send me, but because this patch is about engine itself
> they are not commited - but are working quite fine :) )
> When all reproducers are prepared, junit testsuite is lunched - after
> program is "installed". junit takes location of installed javaws and
> location of prepared reproducers (directory jars, jnlps and other
> resources).
>
> Inside junit suite virtual server is lunched and serves all resources.
> During the testsuite individual jnlps are executed, and their
> stderr/out/return code can be asserted. (There are helepers method for
> this in used virtual server which provides to tester exactly std out/err
> return code). Also possibility to timeout process is added.
> At the end the report - same as was for unit tests - is generated (if my
> previous patch is used :). There is also possibility of including
> stdout/err of tests into xml-report files.
>
Sorry for taking so long to respond to this.
I have got to admit, this is great! I have lots of little nitpicks [1],
but we can argue about those _after_ you commit an initial version.
I only thing I dont like is how test information is spread out. I would
love if this was more jtreg-like: you place everything (jnlp file and
the associated java file (and possibly a makefile or other meta
information)) in one directory. I dont like that you have to add junit
test methods (to another location). You have a big thumbs up from me if
you can fix this.
Thanks again for doing this!
Cheers,
Omair
[1] including file paths/names, variable names, spacing and other details ;)
More information about the distro-pkg-dev
mailing list