Hi Felipe,

> -----Original Message-----
> From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> Sent: jeudi 21 avril 2005 13:57
> To: [EMAIL PROTECTED]
> Subject: cvs commit: jakarta-cactus/documentation/docs/xdocs changes.xml

[snip]

>   +    /**
>   +     * Verifies an empty web was created when the source archive does
> not
>   +     * contain a web deployment descriptor but specifies the version.
>   +     *
>   +     * @throws Exception If an unexpected error occurs
>   +     */
>   +    public void testSrcFileWithoutWebXmlNewWebXml22() throws Exception
>   +    {
>   +           try
>   +           {
>   +                   executeTestTarget();
>   +           }
>   +           catch (BuildException e)
>   +           {

A very small detail: one of the best practices I've been using is to name
the expected exception "expected". I think it makes it a little more
readable.

Thanks
-Vincent

_________________________________________________________________
Do You Yahoo!? -- Une adresse @yahoo.fr gratuite et en français !
Yahoo! Mail : http://fr.mail.yahoo.com

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

Reply via email to