commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 30545] - ConfigruationDynaBean Patch
Date Tue, 10 Aug 2004 09:34:31 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=30545>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30545

ConfigruationDynaBean Patch





------- Additional Comments From epugh@upstate.com  2004-08-10 09:34 -------
Hi, 

This is looking really good.  Could you also submit some unit tests?  If you
look at the other configurations you can see how they have fairly comprehensive
test suites.  Look at DataConfigurationTest for example.  Right now the
JCoverage tool says Configuraiton is at 88% which is a nice place to be!  

Also, what do you think about putting the support classes in o.a.c.c.dynabean? 
or putting all of it in .dynabean?  Also, is this basically two patches?  One
for dynabean, and one for the map to be used in expressions?

Unit tests also help people like me understand how to *use* the new code!

Eric

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message