Hi (Felipe),

I don't like too much the test case names (test*) as they don't represent
anything. Couldn't we replace them with what they are doing? For example
instead of test06, I would prefer testIncludeResourcesInCactifiedWar. What
do you think?

Same comment for the descriptive items in the project.xml. For test06 it
currently says:

  <id>test-cactus-maven-plugin-06</id>
  <name>Simple project</name>
  <description>Test for CACTUS-132</description>

I think the following would be better:

  <id>test-cactus-maven-plugin-testIncludeResourceInCactifiedWar</id>
  <name>Test project</name>
  <description>Test inclusion of resources in the cactified
WAR</description>

What do you think?

Also, there are problems of extra empty lines in almost all the files in
test*. Could you please check this out?

Thanks!
-Vincent


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to