Skip to content

Commit 851b0b9

Browse files
Allows invoker tests to work with verify instead of install
Same as openzipkin/brave#1471 Signed-off-by: Adrian Cole <adrian@tetrate.io>
1 parent 94e01d9 commit 851b0b9

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

build-bin/test

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,4 @@
44
#
55
# See [README.md] for an explanation of this and how CI should use it.
66

7-
# We use install, not verify, because maven-invoker-tests need modules in this
8-
# project installed into the local repository before tests run.
9-
./mvnw -T1C install -nsu "$@"
7+
./mvnw -T1C verify -nsu "$@"

pom.xml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,9 @@
8585
<maven-help-plugin.version>3.5.1</maven-help-plugin.version>
8686
<maven-install-plugin.version>3.1.4</maven-install-plugin.version>
8787
<maven-invoker-plugin.version>3.9.1</maven-invoker-plugin.version>
88+
<!-- default is runtime, but test-scoped reactor deps like in
89+
spring_context need to be in the invoker local repo to resolve -->
90+
<invoker.install.scope>test</invoker.install.scope>
8891
<maven-javadoc-plugin.version>3.12.0</maven-javadoc-plugin.version>
8992
<maven-jar-plugin.version>3.5.0</maven-jar-plugin.version>
9093
<maven-release-plugin.version>3.3.1</maven-release-plugin.version>

src/it/settings.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,11 @@
1818
<url>@localRepositoryUrl@</url>
1919
<releases>
2020
<enabled>true</enabled>
21+
<updatePolicy>always</updatePolicy>
2122
</releases>
2223
<snapshots>
2324
<enabled>true</enabled>
25+
<updatePolicy>always</updatePolicy>
2426
</snapshots>
2527
</repository>
2628
</repositories>
@@ -30,9 +32,11 @@
3032
<url>@localRepositoryUrl@</url>
3133
<releases>
3234
<enabled>true</enabled>
35+
<updatePolicy>always</updatePolicy>
3336
</releases>
3437
<snapshots>
3538
<enabled>true</enabled>
39+
<updatePolicy>always</updatePolicy>
3640
</snapshots>
3741
</pluginRepository>
3842
</pluginRepositories>

0 commit comments

Comments
 (0)