You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Gary Gregory <ga...@gmail.com> on 2012/10/13 17:35:34 UTC

Re: svn commit: r1397859 - in /commons/proper/csv/trunk: pom.xml src/test/java/org/apache/commons/csv/perf/PerformanceTest.java

On Sat, Oct 13, 2012 at 11:30 AM, <se...@apache.org> wrote:

> Author: sebb
> Date: Sat Oct 13 15:30:07 2012
> New Revision: 1397859
>
> URL: http://svn.apache.org/viewvc?rev=1397859&view=rev
> Log:
> Drop @Ignore from perf/PerformanceTest; instead add to pom surefire
> excludes
> This makes it possible to run the test without needing to edit the source
> file.
>

Nice touch.
:)
Gary

>
> Modified:
>     commons/proper/csv/trunk/pom.xml
>
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/perf/PerformanceTest.java
>
> Modified: commons/proper/csv/trunk/pom.xml
> URL:
> http://svn.apache.org/viewvc/commons/proper/csv/trunk/pom.xml?rev=1397859&r1=1397858&r2=1397859&view=diff
>
> ==============================================================================
> --- commons/proper/csv/trunk/pom.xml (original)
> +++ commons/proper/csv/trunk/pom.xml Sat Oct 13 15:30:07 2012
> @@ -120,7 +120,16 @@ CSV files of various types.
>            </descriptors>
>            <tarLongFileMode>gnu</tarLongFileMode>
>          </configuration>
> -     </plugin>
> +      </plugin>
> +      <plugin>
> +        <groupId>org.apache.maven.plugins</groupId>
> +        <artifactId>maven-surefire-plugin</artifactId>
> +        <configuration>
> +          <excludes>
> +            <exclude>**/perf/PerformanceTest.java</exclude>
> +          </excludes>
> +        </configuration>
> +      </plugin>
>      </plugins>
>    </build>
>
>
> Modified:
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/perf/PerformanceTest.java
> URL:
> http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/perf/PerformanceTest.java?rev=1397859&r1=1397858&r2=1397859&view=diff
>
> ==============================================================================
> ---
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/perf/PerformanceTest.java
> (original)
> +++
> commons/proper/csv/trunk/src/test/java/org/apache/commons/csv/perf/PerformanceTest.java
> Sat Oct 13 15:30:07 2012
> @@ -33,15 +33,13 @@ import org.apache.commons.csv.CSVFormat;
>  import org.apache.commons.csv.CSVRecord;
>  import org.apache.commons.io.IOUtils;
>  import org.junit.BeforeClass;
> -import org.junit.Ignore;
>  import org.junit.Test;
>
>  /**
>   * Tests performance.
>   *
> - * Only enable for your own development.
> + * To run this test, use: mvn test -Dtest=PeformanceTest
>   */
> -@Ignore
>  public class PerformanceTest {
>
>      private final int max = 10;
>
>
>


-- 
E-Mail: garydgregory@gmail.com | ggregory@apache.org
JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory