Re: [Cppunit-devel] New TextTestRunner and hierarchy example update

[Baptiste L. <bl...@cl...>]

> Um.  I think you'd better revisit this change.
>
> I wasn't completely happy with the old output, and one of my "TODO"
> items was to fix it and propose a patch.
>
> However, the new output is worse.  All I get is dots with no
> terminating newline:
   No, that's the way it's supposed to be (for me at least). It is also the
way it was done in Michael Feather's version. To me it makes sense:

    The dots are a show progress. A 'F' is printed to show that a test has
failed (equivalent to our progress becoming red). When all tests are run,
you got a report showing what failed (error report need to be made more
readable).

  Having test name and dot with new line would keep the screen scrolling.
This would be hard to use when you have more than a few tests.

  When we run test with the GUI test runner, we don't know which test is
running. On the other hand when a failure occurs, the full report is added
to the failure list. I guess we could dump the failure as soon it occured.
Though, I'm not sure that would be useful since failures would be spread
over many page of text which would keep scrolling as the other tests run.

  Also printing takes time, when you refactor and code test first, you run
the test every few seconds. You don't care which tests are run, you only
want to you if they run. You made sure that the test you added is run by
making it fails the first time.

  If that's not te way you to it, could you give your use case and the
output you'd like to have ?

> Whether to print test names or just dots should probably be made an
> option.  I liked having the names output.
  That could be done by giving the instance of the testresult to use to the
testrunner. But I'm not sure how to manage the lifecycle of the test result
(run() may be called more than once but testresult only support one run).
Suggestions ?

> Seems okay, but someone broke "make distcheck" by removing a file from the
> directory without updating the corresponding Makefile.am.
   That would be me. I moved TypeInfoHolder.h from src to include. I knew
this need to be updated when adding strange file (dsp/dsw), but I though it
automatically retreived .cpp/.h (that would make sense to me). I fixed it.

>
> -S
>
> --
> by Rocket to the Moon,
> by Airplane to the Rocket,
> by Taxi to the Airport,
> by Frontdoor to the Taxi,
> by throwing back the blanket and laying down the legs ...
> - They Might Be Giants
>
>
> _______________________________________________
> Cppunit-devel mailing list
> Cpp...@li...
> http://lists.sourceforge.net/lists/listinfo/cppunit-devel
>