Hi Frantisek,

Frantisek Sodomka wrote:
> On Mar 19, 11:37 pm, Jason Sankey <ja...@zutubi.com> wrote:
>> I pretty much have it working for the test-clojure suite now, although
>> I'm sure the code could use review by a more experienced eye.  I've been
>> looking at adding the other two top-level suites (test-contrib and
>> datalog) too, but their test clojure scripts are structured slightly
>> differently such that requiring them leads them to run their tests
>> immediately.  It would be nice if some convention could be established
>> for a top-level suite to be defined/exposed, and in such a way that I
>> could wrap them with my script which defines special reporting methods
>> to spit out XML.
> 
> Nice!
> I took the liberty and changed the test-contrib and the datalog top-
> level tests to match test-clojure. Let me know if that helps or if
> something is broken :)

Sweet, this is perfect.  I have now cobbled together a proof-of-concept 
on my build server using my code to run the three top-level suites and 
produce JUnit-compatible reports.  I have kept this separate at the 
moment from the primary clojure-contrib project as there are issues I 
need to sort out:

1) At the moment I'm just copying in my extra code and an extra build 
file to Make Things Work.  I need a nicer way to layer this on.

2) A few things are hard coded which need not be.

3) Test-is reports results for all (is ...) expressions separately, but 
there is no good way to name them separately.  At the moment I am 
reporting on each (is ...) as a separate test case by using the test 
name plus line number, but this is clunky.  I think combining the 
results so one deftest == one JUnit report test case would be more sensible.

4) I'm still new to clojure, so I'm sure my code is far from optimal.

In any case, it's nice to have come this far already, and perhaps the 
hackish reports can even start to be useful.  You can take a look at an 
example build here:

http://pulse.zutubi.com/browse/projects/clojure-contrib-reports/builds/5/

ans see that it reports 801 tests, all passed ;).

Cheers,
Jason

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To post to this group, send email to clojure@googlegroups.com
To unsubscribe from this group, send email to 
clojure+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/clojure?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to