Re: [Cppunit-devel] New TextTestRunner and hierarchy example update
Brought to you by:
blep
From: Steve M. R. <ste...@vi...> - 2001-05-20 15:19:26
|
On Sun, May 20, 2001 at 11:54:07AM +0200, Baptiste Lepilleur wrote: > > 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). Yes, sure. This is one option for output, I agree. I think there ought to be a newline after all the dots, that's all. As it stands now, if all the tests succeed, no further output is produced. I also think that some folks (well, me anyway) may want a more verbose output style. I'm not suggesting that the default change. Just that one has an option. I'll have a look and suggest a patch, maybe sometime this week. > > 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). Nope. The Makefile.am files are the master record of the files required to build the project. They must be manually edited for all changes. -- 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 |