You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by sebb <se...@gmail.com> on 2009/07/25 19:29:46 UTC

Re: svn commit: r797804 - in /commons/proper/codec/trunk/src: java/org/apache/commons/codec/ test/org/apache/commons/codec/

On 25/07/2009, ggregory@apache.org <gg...@apache.org> wrote:
> Author: ggregory
>  Date: Sat Jul 25 17:27:04 2009
>  New Revision: 797804
>
>  URL: http://svn.apache.org/viewvc?rev=797804&view=rev
>  Log:
>  Javadocs. Make our exceptions fit the Java 1.4 pattern. 100%/100% line/branch code coverage for the main package.
>
>  Added:
>     commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExceptionTest.java
>     commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExceptionTest.java
>  Modified:
>     commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderException.java
>     commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderException.java
>     commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredCharsetNamesTest.java
>
>  Modified: commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderException.java
>  URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderException.java?rev=797804&r1=797803&r2=797804&view=diff
>  ==============================================================================
>  --- commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderException.java (original)
>  +++ commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderException.java Sat Jul 25 17:27:04 2009
>  @@ -33,7 +33,18 @@
>      private static final long serialVersionUID = 1L;
>
>      /**
>  -     * Creates a DecoderException.
>  +     * Constructs a new exception with <code>null</code> as its detail message. The cause is not initialized, and may
>  +     * subsequently be initialized by a call to {@link #initCause}.
>  +     *
>  +     * @since 1.4
>  +     */
>  +    public DecoderException() {
>  +        super();
>  +    }
>  +
>  +    /**
>  +     * Constructs a new exception with the specified detail message. The cause is not initialized, and may subsequently
>  +     * be initialized by a call to {@link #initCause}.
>       *
>       * @param message
>       *            The detail message which is saved for later retrieval by the {@link #getMessage()} method.
>  @@ -43,28 +54,35 @@
>      }
>
>      /**
>  -     * Creates a DecoderException.
>  +     * Constructsa new exception with the specified detail message and cause.
>  +     *
>  +     * <p>
>  +     * Note that the detail message associated with <code>cause</code> is not automatically incorporated into this
>  +     * exception's detail message.
>  +     * </p>
>       *
>  +     * @param message
>  +     *            The detail message which is saved for later retrieval by the {@link #getMessage()} method.
>       * @param cause
>       *            The cause which is saved for later retrieval by the {@link #getCause()} method. A <code>null</code>
>       *            value is permitted, and indicates that the cause is nonexistent or unknown.
>       * @since 1.4
>       */
>  -    public DecoderException(Throwable cause) {
>  -        super(cause);
>  +    public DecoderException(String message, Throwable cause) {
>  +        super(message, cause);
>      }
>
>      /**
>  -     * Creates a DecoderException.
>  +     * Constructs a new exception with the specified cause and a detail message of <code>(cause==null ?
>  +     * null : cause.toString())</code> (which typically contains the class and detail message of <code>cause</code>).
>  +     * This constructor is useful for exceptions that are little more than wrappers for other throwables.
>       *
>  -     * @param message
>  -     *            The detail message which is saved for later retrieval by the {@link #getMessage()} method.
>       * @param cause
>       *            The cause which is saved for later retrieval by the {@link #getCause()} method. A <code>null</code>
>       *            value is permitted, and indicates that the cause is nonexistent or unknown.
>       * @since 1.4
>       */
>  -    public DecoderException(String message, Throwable cause) {
>  -        super(message, cause);
>  +    public DecoderException(Throwable cause) {
>  +        super(cause);
>      }
>   }
>
>  Modified: commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderException.java
>  URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderException.java?rev=797804&r1=797803&r2=797804&view=diff
>  ==============================================================================
>  --- commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderException.java (original)
>  +++ commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderException.java Sat Jul 25 17:27:04 2009
>  @@ -35,7 +35,18 @@
>      private static final long serialVersionUID = 1L;
>
>      /**
>  -     * Creates a new instance of this exception with an useful message.
>  +     * Constructs a new exception with <code>null</code> as its detail message. The cause is not initialized, and may
>  +     * subsequently be initialized by a call to {@link #initCause}.
>  +     *
>  +     * @since 1.4
>  +     */
>  +    public EncoderException() {
>  +        super();
>  +    }
>  +
>  +    /**
>  +     * Constructs a new exception with the specified detail message. The cause is not initialized, and may subsequently
>  +     * be initialized by a call to {@link #initCause}.
>       *
>       * @param message
>       *            a useful message relating to the encoder specific error.
>  @@ -45,28 +56,35 @@
>      }
>
>      /**
>  -     * Creates a EncoderException.
>  +     * Constructs a new exception with the specified detail message and cause.
>  +     *
>  +     * <p>
>  +     * Note that the detail message associated with <code>cause</code> is not automatically incorporated into this
>  +     * exception's detail message.
>  +     * </p>
>       *
>  +     * @param message
>  +     *            The detail message which is saved for later retrieval by the {@link #getMessage()} method.
>       * @param cause
>       *            The cause which is saved for later retrieval by the {@link #getCause()} method. A <code>null</code>
>       *            value is permitted, and indicates that the cause is nonexistent or unknown.
>       * @since 1.4
>       */
>  -    public EncoderException(Throwable cause) {
>  -        super(cause);
>  +    public EncoderException(String message, Throwable cause) {
>  +        super(message, cause);
>      }
>
>      /**
>  -     * Creates a EncoderException.
>  +     * Constructs a new exception with the specified cause and a detail message of <code>(cause==null ?
>  +     * null : cause.toString())</code> (which typically contains the class and detail message of <code>cause</code>).
>  +     * This constructor is useful for exceptions that are little more than wrappers for other throwables.
>       *
>  -     * @param message
>  -     *            The detail message which is saved for later retrieval by the {@link #getMessage()} method.
>       * @param cause
>       *            The cause which is saved for later retrieval by the {@link #getCause()} method. A <code>null</code>
>       *            value is permitted, and indicates that the cause is nonexistent or unknown.
>       * @since 1.4
>       */
>  -    public EncoderException(String message, Throwable cause) {
>  -        super(message, cause);
>  +    public EncoderException(Throwable cause) {
>  +        super(cause);
>      }
>   }
>
>  Added: commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExceptionTest.java
>  URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExceptionTest.java?rev=797804&view=auto
>  ==============================================================================
>  --- commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExceptionTest.java (added)
>  +++ commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExceptionTest.java Sat Jul 25 17:27:04 2009
>  @@ -0,0 +1,49 @@
>  +/*
>  + * Copyright (C) 1993-2003 SEAGULL

Huh?

>  + *
>  + * DecoderException.java
>  + * Created on Jul 25, 2009, 9:28:09 AM
>  + *
>  + */
>  +
>  +package org.apache.commons.codec;
>  +
>  +import junit.framework.TestCase;
>  +
>  +/**
>  + * Tests DecoderException.
>  + *
>  + * @author <a href="mailto:ggregory@seagullsw.com">Gary Gregory</a>
>  + * @version $Id: $
>  + */
>  +public class DecoderExceptionTest extends TestCase {
>  +
>  +    private static final String MSG = "TEST";
>  +
>  +    private static final Throwable t = new Exception();
>  +
>  +    public void testConstructor0() {
>  +        DecoderException e = new DecoderException();
>  +        assertNull(e.getMessage());
>  +        assertNull(e.getCause());
>  +    }
>  +
>  +    public void testConstructorString() {
>  +        DecoderException e = new DecoderException(MSG);
>  +        assertEquals(MSG, e.getMessage());
>  +        assertNull(e.getCause());
>  +    }
>  +
>  +    public void testConstructorStringThrowable() {
>  +        DecoderException e = new DecoderException(MSG, t);
>  +        assertEquals(MSG, e.getMessage());
>  +        assertEquals(t, e.getCause());
>  +    }
>  +
>  +    public void testConstructorThrowable() {
>  +        DecoderException e = new DecoderException(t);
>  +        assertEquals(t.getClass().getName(), e.getMessage());
>  +        assertEquals(t, e.getCause());
>  +    }
>  +
>  +}
>
>  Added: commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExceptionTest.java
>  URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExceptionTest.java?rev=797804&view=auto
>  ==============================================================================
>  --- commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExceptionTest.java (added)
>  +++ commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExceptionTest.java Sat Jul 25 17:27:04 2009
>  @@ -0,0 +1,49 @@
>  +/*
>  + * Copyright (C) 1993-2003 SEAGULL
>  + *

Huh?

>  + * DecoderException.java
>  + * Created on Jul 25, 2009, 9:28:09 AM
>  + *
>  + */
>  +
>  +package org.apache.commons.codec;
>  +
>  +import junit.framework.TestCase;
>  +
>  +/**
>  + * Tests EncoderException.
>  + *
>  + * @author <a href="mailto:ggregory@seagullsw.com">Gary Gregory</a>
>  + * @version $Id: $
>  + */
>  +public class EncoderExceptionTest extends TestCase {
>  +
>  +    private static final String MSG = "TEST";
>  +
>  +    private static final Throwable t = new Exception();
>  +
>  +    public void testConstructor0() {
>  +        EncoderException e = new EncoderException();
>  +        assertNull(e.getMessage());
>  +        assertNull(e.getCause());
>  +    }
>  +
>  +    public void testConstructorString() {
>  +        EncoderException e = new EncoderException(MSG);
>  +        assertEquals(MSG, e.getMessage());
>  +        assertNull(e.getCause());
>  +    }
>  +
>  +    public void testConstructorStringThrowable() {
>  +        EncoderException e = new EncoderException(MSG, t);
>  +        assertEquals(MSG, e.getMessage());
>  +        assertEquals(t, e.getCause());
>  +    }
>  +
>  +    public void testConstructorThrowable() {
>  +        EncoderException e = new EncoderException(t);
>  +        assertEquals(t.getClass().getName(), e.getMessage());
>  +        assertEquals(t, e.getCause());
>  +    }
>  +
>  +}
>
>  Modified: commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredCharsetNamesTest.java
>  URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredCharsetNamesTest.java?rev=797804&r1=797803&r2=797804&view=diff
>  ==============================================================================
>  --- commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredCharsetNamesTest.java (original)
>  +++ commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredCharsetNamesTest.java Sat Jul 25 17:27:04 2009
>  @@ -28,6 +28,13 @@
>   */
>   public class RequiredCharsetNamesTest extends TestCase {
>
>  +    /**
>  +     * We could make the constructor private in the future, it's a matter a style.
>  +     */
>  +    public void testConstructor() {
>  +        new RequiredCharsetNames();
>  +    }
>  +
>      public void testIso8859_1() {
>          Assert.assertEquals("ISO-8859-1", RequiredCharsetNames.ISO_8859_1);
>      }
>
>
>

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


RE: svn commit: r797804 - in /commons/proper/codec/trunk/src: java/org/apache/commons/codec/ test/org/apache/commons/codec/

Posted by Gary Gregory <GG...@seagullsoftware.com>.
Oops, fixing...

> -----Original Message-----
> From: sebb [mailto:sebbaz@gmail.com]
> Sent: Saturday, July 25, 2009 10:30 AM
> To: dev@commons.apache.org
> Subject: Re: svn commit: r797804 - in /commons/proper/codec/trunk/src:
> java/org/apache/commons/codec/ test/org/apache/commons/codec/
> 
> On 25/07/2009, ggregory@apache.org <gg...@apache.org> wrote:
> > Author: ggregory
> >  Date: Sat Jul 25 17:27:04 2009
> >  New Revision: 797804
> >
> >  URL: http://svn.apache.org/viewvc?rev=797804&view=rev
> >  Log:
> >  Javadocs. Make our exceptions fit the Java 1.4 pattern. 100%/100%
> line/branch code coverage for the main package.
> >
> >  Added:
> >
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExcept
> ionTest.java
> >
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExcept
> ionTest.java
> >  Modified:
> >
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderExcept
> ion.java
> >
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderExcept
> ion.java
> >
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredChars
> etNamesTest.java
> >
> >  Modified:
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderExcept
> ion.java
> >  URL:
> http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apach
> e/commons/codec/DecoderException.java?rev=797804&r1=797803&r2=797804&view=
> diff
> >
> ==========================================================================
> ====
> >  ---
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderExcept
> ion.java (original)
> >  +++
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/DecoderExcept
> ion.java Sat Jul 25 17:27:04 2009
> >  @@ -33,7 +33,18 @@
> >      private static final long serialVersionUID = 1L;
> >
> >      /**
> >  -     * Creates a DecoderException.
> >  +     * Constructs a new exception with <code>null</code> as its detail
> message. The cause is not initialized, and may
> >  +     * subsequently be initialized by a call to {@link #initCause}.
> >  +     *
> >  +     * @since 1.4
> >  +     */
> >  +    public DecoderException() {
> >  +        super();
> >  +    }
> >  +
> >  +    /**
> >  +     * Constructs a new exception with the specified detail message.
> The cause is not initialized, and may subsequently
> >  +     * be initialized by a call to {@link #initCause}.
> >       *
> >       * @param message
> >       *            The detail message which is saved for later retrieval
> by the {@link #getMessage()} method.
> >  @@ -43,28 +54,35 @@
> >      }
> >
> >      /**
> >  -     * Creates a DecoderException.
> >  +     * Constructsa new exception with the specified detail message and
> cause.
> >  +     *
> >  +     * <p>
> >  +     * Note that the detail message associated with <code>cause</code>
> is not automatically incorporated into this
> >  +     * exception's detail message.
> >  +     * </p>
> >       *
> >  +     * @param message
> >  +     *            The detail message which is saved for later
> retrieval by the {@link #getMessage()} method.
> >       * @param cause
> >       *            The cause which is saved for later retrieval by the
> {@link #getCause()} method. A <code>null</code>
> >       *            value is permitted, and indicates that the cause is
> nonexistent or unknown.
> >       * @since 1.4
> >       */
> >  -    public DecoderException(Throwable cause) {
> >  -        super(cause);
> >  +    public DecoderException(String message, Throwable cause) {
> >  +        super(message, cause);
> >      }
> >
> >      /**
> >  -     * Creates a DecoderException.
> >  +     * Constructs a new exception with the specified cause and a
> detail message of <code>(cause==null ?
> >  +     * null : cause.toString())</code> (which typically contains the
> class and detail message of <code>cause</code>).
> >  +     * This constructor is useful for exceptions that are little more
> than wrappers for other throwables.
> >       *
> >  -     * @param message
> >  -     *            The detail message which is saved for later
> retrieval by the {@link #getMessage()} method.
> >       * @param cause
> >       *            The cause which is saved for later retrieval by the
> {@link #getCause()} method. A <code>null</code>
> >       *            value is permitted, and indicates that the cause is
> nonexistent or unknown.
> >       * @since 1.4
> >       */
> >  -    public DecoderException(String message, Throwable cause) {
> >  -        super(message, cause);
> >  +    public DecoderException(Throwable cause) {
> >  +        super(cause);
> >      }
> >   }
> >
> >  Modified:
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderExcept
> ion.java
> >  URL:
> http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/java/org/apach
> e/commons/codec/EncoderException.java?rev=797804&r1=797803&r2=797804&view=
> diff
> >
> ==========================================================================
> ====
> >  ---
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderExcept
> ion.java (original)
> >  +++
> commons/proper/codec/trunk/src/java/org/apache/commons/codec/EncoderExcept
> ion.java Sat Jul 25 17:27:04 2009
> >  @@ -35,7 +35,18 @@
> >      private static final long serialVersionUID = 1L;
> >
> >      /**
> >  -     * Creates a new instance of this exception with an useful
> message.
> >  +     * Constructs a new exception with <code>null</code> as its detail
> message. The cause is not initialized, and may
> >  +     * subsequently be initialized by a call to {@link #initCause}.
> >  +     *
> >  +     * @since 1.4
> >  +     */
> >  +    public EncoderException() {
> >  +        super();
> >  +    }
> >  +
> >  +    /**
> >  +     * Constructs a new exception with the specified detail message.
> The cause is not initialized, and may subsequently
> >  +     * be initialized by a call to {@link #initCause}.
> >       *
> >       * @param message
> >       *            a useful message relating to the encoder specific
> error.
> >  @@ -45,28 +56,35 @@
> >      }
> >
> >      /**
> >  -     * Creates a EncoderException.
> >  +     * Constructs a new exception with the specified detail message
> and cause.
> >  +     *
> >  +     * <p>
> >  +     * Note that the detail message associated with <code>cause</code>
> is not automatically incorporated into this
> >  +     * exception's detail message.
> >  +     * </p>
> >       *
> >  +     * @param message
> >  +     *            The detail message which is saved for later
> retrieval by the {@link #getMessage()} method.
> >       * @param cause
> >       *            The cause which is saved for later retrieval by the
> {@link #getCause()} method. A <code>null</code>
> >       *            value is permitted, and indicates that the cause is
> nonexistent or unknown.
> >       * @since 1.4
> >       */
> >  -    public EncoderException(Throwable cause) {
> >  -        super(cause);
> >  +    public EncoderException(String message, Throwable cause) {
> >  +        super(message, cause);
> >      }
> >
> >      /**
> >  -     * Creates a EncoderException.
> >  +     * Constructs a new exception with the specified cause and a
> detail message of <code>(cause==null ?
> >  +     * null : cause.toString())</code> (which typically contains the
> class and detail message of <code>cause</code>).
> >  +     * This constructor is useful for exceptions that are little more
> than wrappers for other throwables.
> >       *
> >  -     * @param message
> >  -     *            The detail message which is saved for later
> retrieval by the {@link #getMessage()} method.
> >       * @param cause
> >       *            The cause which is saved for later retrieval by the
> {@link #getCause()} method. A <code>null</code>
> >       *            value is permitted, and indicates that the cause is
> nonexistent or unknown.
> >       * @since 1.4
> >       */
> >  -    public EncoderException(String message, Throwable cause) {
> >  -        super(message, cause);
> >  +    public EncoderException(Throwable cause) {
> >  +        super(cause);
> >      }
> >   }
> >
> >  Added:
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExcept
> ionTest.java
> >  URL:
> http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apach
> e/commons/codec/DecoderExceptionTest.java?rev=797804&view=auto
> >
> ==========================================================================
> ====
> >  ---
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExcept
> ionTest.java (added)
> >  +++
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/DecoderExcept
> ionTest.java Sat Jul 25 17:27:04 2009
> >  @@ -0,0 +1,49 @@
> >  +/*
> >  + * Copyright (C) 1993-2003 SEAGULL
> 
> Huh?
> 
> >  + *
> >  + * DecoderException.java
> >  + * Created on Jul 25, 2009, 9:28:09 AM
> >  + *
> >  + */
> >  +
> >  +package org.apache.commons.codec;
> >  +
> >  +import junit.framework.TestCase;
> >  +
> >  +/**
> >  + * Tests DecoderException.
> >  + *
> >  + * @author <a href="mailto:ggregory@seagullsw.com">Gary Gregory</a>
> >  + * @version $Id: $
> >  + */
> >  +public class DecoderExceptionTest extends TestCase {
> >  +
> >  +    private static final String MSG = "TEST";
> >  +
> >  +    private static final Throwable t = new Exception();
> >  +
> >  +    public void testConstructor0() {
> >  +        DecoderException e = new DecoderException();
> >  +        assertNull(e.getMessage());
> >  +        assertNull(e.getCause());
> >  +    }
> >  +
> >  +    public void testConstructorString() {
> >  +        DecoderException e = new DecoderException(MSG);
> >  +        assertEquals(MSG, e.getMessage());
> >  +        assertNull(e.getCause());
> >  +    }
> >  +
> >  +    public void testConstructorStringThrowable() {
> >  +        DecoderException e = new DecoderException(MSG, t);
> >  +        assertEquals(MSG, e.getMessage());
> >  +        assertEquals(t, e.getCause());
> >  +    }
> >  +
> >  +    public void testConstructorThrowable() {
> >  +        DecoderException e = new DecoderException(t);
> >  +        assertEquals(t.getClass().getName(), e.getMessage());
> >  +        assertEquals(t, e.getCause());
> >  +    }
> >  +
> >  +}
> >
> >  Added:
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExcept
> ionTest.java
> >  URL:
> http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apach
> e/commons/codec/EncoderExceptionTest.java?rev=797804&view=auto
> >
> ==========================================================================
> ====
> >  ---
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExcept
> ionTest.java (added)
> >  +++
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/EncoderExcept
> ionTest.java Sat Jul 25 17:27:04 2009
> >  @@ -0,0 +1,49 @@
> >  +/*
> >  + * Copyright (C) 1993-2003 SEAGULL
> >  + *
> 
> Huh?
> 
> >  + * DecoderException.java
> >  + * Created on Jul 25, 2009, 9:28:09 AM
> >  + *
> >  + */
> >  +
> >  +package org.apache.commons.codec;
> >  +
> >  +import junit.framework.TestCase;
> >  +
> >  +/**
> >  + * Tests EncoderException.
> >  + *
> >  + * @author <a href="mailto:ggregory@seagullsw.com">Gary Gregory</a>
> >  + * @version $Id: $
> >  + */
> >  +public class EncoderExceptionTest extends TestCase {
> >  +
> >  +    private static final String MSG = "TEST";
> >  +
> >  +    private static final Throwable t = new Exception();
> >  +
> >  +    public void testConstructor0() {
> >  +        EncoderException e = new EncoderException();
> >  +        assertNull(e.getMessage());
> >  +        assertNull(e.getCause());
> >  +    }
> >  +
> >  +    public void testConstructorString() {
> >  +        EncoderException e = new EncoderException(MSG);
> >  +        assertEquals(MSG, e.getMessage());
> >  +        assertNull(e.getCause());
> >  +    }
> >  +
> >  +    public void testConstructorStringThrowable() {
> >  +        EncoderException e = new EncoderException(MSG, t);
> >  +        assertEquals(MSG, e.getMessage());
> >  +        assertEquals(t, e.getCause());
> >  +    }
> >  +
> >  +    public void testConstructorThrowable() {
> >  +        EncoderException e = new EncoderException(t);
> >  +        assertEquals(t.getClass().getName(), e.getMessage());
> >  +        assertEquals(t, e.getCause());
> >  +    }
> >  +
> >  +}
> >
> >  Modified:
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredChars
> etNamesTest.java
> >  URL:
> http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apach
> e/commons/codec/RequiredCharsetNamesTest.java?rev=797804&r1=797803&r2=7978
> 04&view=diff
> >
> ==========================================================================
> ====
> >  ---
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredChars
> etNamesTest.java (original)
> >  +++
> commons/proper/codec/trunk/src/test/org/apache/commons/codec/RequiredChars
> etNamesTest.java Sat Jul 25 17:27:04 2009
> >  @@ -28,6 +28,13 @@
> >   */
> >   public class RequiredCharsetNamesTest extends TestCase {
> >
> >  +    /**
> >  +     * We could make the constructor private in the future, it's a
> matter a style.
> >  +     */
> >  +    public void testConstructor() {
> >  +        new RequiredCharsetNames();
> >  +    }
> >  +
> >      public void testIso8859_1() {
> >          Assert.assertEquals("ISO-8859-1",
> RequiredCharsetNames.ISO_8859_1);
> >      }
> >
> >
> >
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org


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