Skip to content
This repository was archived by the owner on Oct 7, 2019. It is now read-only.

Commit 81ea9ab

Browse files
author
Garvin LeClaire
committed
Merge branch 'spotbugs-master'
fix tag to pull off released findbugs tests
2 parents 329bf15 + a8fc375 commit 81ea9ab

File tree

3 files changed

+21
-14
lines changed

3 files changed

+21
-14
lines changed

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@ Skip tests
2424
```
2525
mvn -DskipTests=true clean install
2626
```
27-
Run tests on findbugs test source code that is local instead of from FindBugs SVN repository
27+
Run tests on findbugs test source code that is local instead of from FindBugs github repository
2828
```
29-
mvn -DtestSrc=local -DlocalTestSrc=/opt/findBugs/findbugsTestCases/src -Prun-its clean install
29+
mvn -DtestSrc=local -DlocalTestSrc=/opt/findBugs -Prun-its clean install
3030
```
3131

3232
Run selected tests

pom.xml

Lines changed: 18 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -129,12 +129,12 @@
129129
<junitVersion>4.12</junitVersion>
130130
<mavenSurefireVersion>2.20</mavenSurefireVersion>
131131
<InfoReportsVersion>2.9</InfoReportsVersion>
132+
132133
<findbugsTestDebug>false</findbugsTestDebug>
133134
<integrationTestSrc>${project.build.directory}/it-src-findbugs</integrationTestSrc>
134-
<localTestSrc>${user.dir}/FindBugs/findbugsTestCases/src</localTestSrc>
135-
<remoteTestSrc>scm:svn:http://findbugs.googlecode.com/svn/branches/1.3.9/findbugsTestCases/src</remoteTestSrc>
136-
<!-- <remoteTestSrc>scm:svn:http://findbugs.googlecode.com/svn/trunk/findbugsTestCases/src/java/</remoteTestSrc> -->
137-
<includesTestSrcPattern>**/A*.java, **/Use*.java</includesTestSrcPattern>
135+
<localTestSrc>${user.dir}/FindBugs</localTestSrc>
136+
<remoteTestSrc>scm:git:https://github.com/findbugsproject/findbugs/</remoteTestSrc>
137+
<includesTestSrcPattern>**findbugsTestCases/src/java/A*.java, **findbugsTestCases/src/java/Use*.java, **findbugsTestCases/src/java/annotations/*.java</includesTestSrcPattern>
138138
<testSrc>remote</testSrc>
139139

140140
<pmd.skip>true</pmd.skip>
@@ -583,8 +583,9 @@
583583
</plugins>
584584
</build>
585585
</profile>
586+
586587
<profile>
587-
<id>find-it-src-export</id>
588+
<id>find-it-src-checkout</id>
588589
<activation>
589590
<!-- <file> -->
590591
<!-- <missing>${integrationTestSrc}/A.java</missing> -->
@@ -605,23 +606,25 @@
605606
<id>prepare-integration-test-remote-findbugs-src</id>
606607
<phase>pre-integration-test</phase>
607608
<goals>
608-
<goal>export</goal>
609+
<goal>checkout</goal>
609610
</goals>
610611
<configuration>
611612
<connectionUrl>${remoteTestSrc}</connectionUrl>
612-
<exportDirectory>${integrationTestSrc}</exportDirectory>
613+
<checkoutDirectory>${integrationTestSrc}</checkoutDirectory>
613614
<providerImplementations>
614-
<svn>javasvn</svn>
615+
<git>jgit</git>
615616
</providerImplementations>
616617
<includes>${includesTestSrcPattern}</includes>
618+
<scmVersion>release-${findbugsVersion}</scmVersion>
619+
<scmVersionType>branch</scmVersionType>
617620
</configuration>
618621
</execution>
619622
</executions>
620623
<dependencies>
621624
<dependency>
622-
<groupId>com.google.code.maven-scm-provider-svnjava</groupId>
623-
<artifactId>maven-scm-provider-svnjava</artifactId>
624-
<version>2.1.2</version>
625+
<groupId>org.apache.maven.scm</groupId>
626+
<artifactId>maven-scm-provider-jgit</artifactId>
627+
<version>1.9.5</version>
625628
</dependency>
626629
</dependencies>
627630
</plugin>
@@ -657,7 +660,11 @@
657660
<target>
658661

659662
<echo>Copying Source for Tests......</echo>
663+
<echo>Making ${integrationTestSrc}</echo>
660664
<mkdir dir="${integrationTestSrc}" />
665+
<echo>Copying to ${integrationTestSrc}</echo>
666+
<echo>from ${localTestSrc}</echo>
667+
<echo>for ${includesTestSrcPattern}</echo>
661668
<copy todir="${integrationTestSrc}">
662669
<fileset dir="${localTestSrc}" includes="${includesTestSrcPattern}" />
663670
</copy>

src/it/common.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
</dependencies>
5454

5555
<build>
56-
<sourceDirectory>@project.build.directory@/it-src-findbugs/java</sourceDirectory>
56+
<sourceDirectory>@project.build.directory@/it-src-findbugs/findbugsTestCases/src/java</sourceDirectory>
5757
<testSourceDirectory>@basedir@/src/it-src/test/java</testSourceDirectory>
5858
<pluginManagement>
5959
<plugins>

0 commit comments

Comments
 (0)