That's fair. Dan, instead of going through the jira-to-you-to-svn pass again, how about you add the following javadoc to the class ...
/** * Regression test for CXF-959. This is a point test for consistent use of * namespace prefixes in generated WSDL/XMLSchema. This test could * be made into a more comprehensive functional test by exercising cases * such as multiple schema. */ > -----Original Message----- > From: Glen Mazza [mailto:[EMAIL PROTECTED] > Sent: Wednesday, September 05, 2007 6:20 PM > To: [email protected] > Subject: Re: svn commit: r573082 - > in/incubator/cxf/trunk/rt/databinding/aegis/src/test/java/org/apache/cxf /a > egis/namespaces: ./ NamespaceConfusionTest.java data/ data/Name.java > impl/impl/NameServiceImpl.java intf/ intf/NameService.java > > Am Mittwoch, den 05.09.2007, 21:59 +0000 schrieb [EMAIL PROTECTED]: > > Author: dkulp > > Date: Wed Sep 5 14:59:07 2007 > > New Revision: 573082 > > > > URL: http://svn.apache.org/viewvc?rev=573082&view=rev > > Log: > > [CXF-959] Add testcase provided by Benson Margulies > > > > + > > +public class NamespaceConfusionTest extends AbstractAegisTest { > > + > > I think it would be nice to have a comment for this class explaining > what it does/what it is for. By name alone, I don't think it is > sufficiently evident to the reader (at least not this one.) > > Glen >
