You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sj...@apache.org on 2022/05/23 20:15:35 UTC

[maven-plugin-tools] 01/01: Cleanup IT tests - remove skipTests

This is an automated email from the ASF dual-hosted git repository.

sjaranowski pushed a commit to branch rm-skip-test
in repository https://gitbox.apache.org/repos/asf/maven-plugin-tools.git

commit 4e8459e9169d628344ced8ef54b2c44690d9056c
Author: Slawomir Jaranowski <s....@gmail.com>
AuthorDate: Mon May 23 22:07:51 2022 +0200

    Cleanup IT tests - remove skipTests
    
    - skipTests is not needed, there are no unit test
    - minimum Maven version is 3.2.5 - no need restrict to 3.0+
---
 .../src/it/annotation-with-inheritance-reactor/invoker.properties | 2 +-
 .../src/it/annotation-with-inheritance/invoker.properties         | 4 ++--
 maven-plugin-plugin/src/it/ant-basic/invoker.properties           | 2 +-
 .../src/it/ant-reference-parameter/invoker.properties             | 2 +-
 maven-plugin-plugin/src/it/ant-simple-usage/invoker.properties    | 2 +-
 maven-plugin-plugin/src/it/beanshell-simple/invoker.properties    | 2 +-
 maven-plugin-plugin/src/it/help-basic-jdk11/invoker.properties    | 7 ++-----
 maven-plugin-plugin/src/it/help-basic/invoker.properties          | 8 ++------
 .../src/it/ignore-plugin-class-realm/invoker.properties           | 2 +-
 .../src/it/java-basic-annotations-jdk8/invoker.properties         | 5 ++---
 .../src/it/java-basic-annotations-jdkcurrent/invoker.properties   | 4 ++--
 .../src/it/java-basic-annotations/invoker.properties              | 4 ++--
 maven-plugin-plugin/src/it/java-basic/invoker.properties          | 2 +-
 maven-plugin-plugin/src/it/mplugin-187/invoker.properties         | 2 +-
 maven-plugin-plugin/src/it/mplugin-223/invoker.properties         | 2 +-
 .../src/it/mplugin-299_no-configuration/invoker.properties        | 3 +--
 .../src/it/plugin-report-annotations/invoker.properties           | 2 +-
 maven-plugin-plugin/src/it/plugin-report/invoker.properties       | 2 +-
 maven-plugin-plugin/src/it/skip/invoker.properties                | 4 ++--
 19 files changed, 26 insertions(+), 35 deletions(-)

diff --git a/maven-plugin-plugin/src/it/annotation-with-inheritance-reactor/invoker.properties b/maven-plugin-plugin/src/it/annotation-with-inheritance-reactor/invoker.properties
index 260443cd..24189f77 100644
--- a/maven-plugin-plugin/src/it/annotation-with-inheritance-reactor/invoker.properties
+++ b/maven-plugin-plugin/src/it/annotation-with-inheritance-reactor/invoker.properties
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.its.annotation-with-inheritance-reactor:module-mojo:1.0-SNAPSHOT:it0014
 invoker.goals.3 = org.apache.maven.its.annotation-with-inheritance-reactor:module-mojo:1.0-SNAPSHOT:first
diff --git a/maven-plugin-plugin/src/it/annotation-with-inheritance/invoker.properties b/maven-plugin-plugin/src/it/annotation-with-inheritance/invoker.properties
index 17e8ee4c..0ad41374 100644
--- a/maven-plugin-plugin/src/it/annotation-with-inheritance/invoker.properties
+++ b/maven-plugin-plugin/src/it/annotation-with-inheritance/invoker.properties
@@ -15,8 +15,8 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.its.annotation-with-inheritance:annotation-with-inheritance:1.0-SNAPSHOT:it0014
 invoker.goals.3 = org.apache.maven.its.annotation-with-inheritance:annotation-with-inheritance:1.0-SNAPSHOT:first
 invoker.goals.4 = org.apache.maven.its.annotation-with-inheritance:annotation-with-inheritance:1.0-SNAPSHOT:help
-invoker.goals.5 = org.apache.maven.its.annotation-with-inheritance:annotation-with-inheritance:1.0-SNAPSHOT:third
\ No newline at end of file
+invoker.goals.5 = org.apache.maven.its.annotation-with-inheritance:annotation-with-inheritance:1.0-SNAPSHOT:third
diff --git a/maven-plugin-plugin/src/it/ant-basic/invoker.properties b/maven-plugin-plugin/src/it/ant-basic/invoker.properties
index 0d94ede8..55875b64 100644
--- a/maven-plugin-plugin/src/it/ant-basic/invoker.properties
+++ b/maven-plugin-plugin/src/it/ant-basic/invoker.properties
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.ant.it:maven-ant-it-basic:1.0-SNAPSHOT:touch -Dname=touch.txt
 
diff --git a/maven-plugin-plugin/src/it/ant-reference-parameter/invoker.properties b/maven-plugin-plugin/src/it/ant-reference-parameter/invoker.properties
index adcfaf53..96636d0b 100644
--- a/maven-plugin-plugin/src/it/ant-reference-parameter/invoker.properties
+++ b/maven-plugin-plugin/src/it/ant-reference-parameter/invoker.properties
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.ant.it:maven-ant-it-referenceParameter:1.0-SNAPSHOT:test
 
diff --git a/maven-plugin-plugin/src/it/ant-simple-usage/invoker.properties b/maven-plugin-plugin/src/it/ant-simple-usage/invoker.properties
index 853271b3..3df4a4d6 100644
--- a/maven-plugin-plugin/src/it/ant-simple-usage/invoker.properties
+++ b/maven-plugin-plugin/src/it/ant-simple-usage/invoker.properties
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 #invoker.goals.2 = org.apache.maven.ant.it:maven-ant-it-referenceParameter:1.0-SNAPSHOT:test
 
diff --git a/maven-plugin-plugin/src/it/beanshell-simple/invoker.properties b/maven-plugin-plugin/src/it/beanshell-simple/invoker.properties
index 8ec6452a..88773cf4 100644
--- a/maven-plugin-plugin/src/it/beanshell-simple/invoker.properties
+++ b/maven-plugin-plugin/src/it/beanshell-simple/invoker.properties
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.beanshell.it:maven-beanshell-it-basic:1.0-SNAPSHOT:touch -Dname=touch.txt
 
diff --git a/maven-plugin-plugin/src/it/help-basic-jdk11/invoker.properties b/maven-plugin-plugin/src/it/help-basic-jdk11/invoker.properties
index e1086cb5..9cdacab6 100644
--- a/maven-plugin-plugin/src/it/help-basic-jdk11/invoker.properties
+++ b/maven-plugin-plugin/src/it/help-basic-jdk11/invoker.properties
@@ -16,8 +16,5 @@
 # under the License.
 
 invoker.java.version = 11+
-invoker.maven.version = 3.0+
-invoker.goals.1 = clean install -DskipTests
-invoker.goals.2 = org.apache.maven.its.plugin:help-jdk11:1.0-SNAPSHOT:help
-invoker.goals.3 = --log-file help.log org.apache.maven.its.plugin:help-jdk11:1.0-SNAPSHOT:help
-# --log-file option is only supported by Maven 3
+invoker.goals.1 = clean install
+invoker.goals.2 = --log-file help.log org.apache.maven.its.plugin:help-jdk11:1.0-SNAPSHOT:help
diff --git a/maven-plugin-plugin/src/it/help-basic/invoker.properties b/maven-plugin-plugin/src/it/help-basic/invoker.properties
index 544361b7..e7b0b4d6 100644
--- a/maven-plugin-plugin/src/it/help-basic/invoker.properties
+++ b/maven-plugin-plugin/src/it/help-basic/invoker.properties
@@ -15,9 +15,5 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.maven.version = 3.0+
-invoker.goals.1 = clean install -DskipTests
-invoker.goals.2 = org.apache.maven.its.plugin:help:1.0-SNAPSHOT:help
-invoker.goals.3 = --log-file help.log org.apache.maven.its.plugin:help:1.0-SNAPSHOT:help
-# --log-file option is only supported by Maven 3
-invoker.maven.version = 3.0+
+invoker.goals.1 = clean install
+invoker.goals.2 = --log-file help.log org.apache.maven.its.plugin:help:1.0-SNAPSHOT:help
diff --git a/maven-plugin-plugin/src/it/ignore-plugin-class-realm/invoker.properties b/maven-plugin-plugin/src/it/ignore-plugin-class-realm/invoker.properties
index 5493444d..9c6bf146 100644
--- a/maven-plugin-plugin/src/it/ignore-plugin-class-realm/invoker.properties
+++ b/maven-plugin-plugin/src/it/ignore-plugin-class-realm/invoker.properties
@@ -15,7 +15,7 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.profiles.1 = setup
 
 invoker.goals.2 = clean process-classes
diff --git a/maven-plugin-plugin/src/it/java-basic-annotations-jdk8/invoker.properties b/maven-plugin-plugin/src/it/java-basic-annotations-jdk8/invoker.properties
index a67d07c8..d0a9e140 100644
--- a/maven-plugin-plugin/src/it/java-basic-annotations-jdk8/invoker.properties
+++ b/maven-plugin-plugin/src/it/java-basic-annotations-jdk8/invoker.properties
@@ -15,7 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.java.version = 1.8+
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:it0014
-invoker.goals.3 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:help
\ No newline at end of file
+invoker.goals.3 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:help
diff --git a/maven-plugin-plugin/src/it/java-basic-annotations-jdkcurrent/invoker.properties b/maven-plugin-plugin/src/it/java-basic-annotations-jdkcurrent/invoker.properties
index 64ad8107..c955e5b7 100644
--- a/maven-plugin-plugin/src/it/java-basic-annotations-jdkcurrent/invoker.properties
+++ b/maven-plugin-plugin/src/it/java-basic-annotations-jdkcurrent/invoker.properties
@@ -16,6 +16,6 @@
 # under the License.
 
 invoker.java.version = 9+
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:it0014
-invoker.goals.3 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:help
\ No newline at end of file
+invoker.goals.3 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:help
diff --git a/maven-plugin-plugin/src/it/java-basic-annotations/invoker.properties b/maven-plugin-plugin/src/it/java-basic-annotations/invoker.properties
index f3a9963d..d0a9e140 100644
--- a/maven-plugin-plugin/src/it/java-basic-annotations/invoker.properties
+++ b/maven-plugin-plugin/src/it/java-basic-annotations/invoker.properties
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:it0014
-invoker.goals.3 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:help
\ No newline at end of file
+invoker.goals.3 = org.apache.maven.its.basic-java-annotations:maven-it-basic-java-annotations:1.0-SNAPSHOT:help
diff --git a/maven-plugin-plugin/src/it/java-basic/invoker.properties b/maven-plugin-plugin/src/it/java-basic/invoker.properties
index 6672f025..da4a708a 100644
--- a/maven-plugin-plugin/src/it/java-basic/invoker.properties
+++ b/maven-plugin-plugin/src/it/java-basic/invoker.properties
@@ -15,6 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = org.apache.maven.its.it0013:maven-it-it0013:1.0-SNAPSHOT:it0013
 
diff --git a/maven-plugin-plugin/src/it/mplugin-187/invoker.properties b/maven-plugin-plugin/src/it/mplugin-187/invoker.properties
index 5659577b..a16bae84 100644
--- a/maven-plugin-plugin/src/it/mplugin-187/invoker.properties
+++ b/maven-plugin-plugin/src/it/mplugin-187/invoker.properties
@@ -15,4 +15,4 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals = clean plugin:report -DskipTests
+invoker.goals = clean plugin:report
diff --git a/maven-plugin-plugin/src/it/mplugin-223/invoker.properties b/maven-plugin-plugin/src/it/mplugin-223/invoker.properties
index adce2cd6..bbd66b86 100644
--- a/maven-plugin-plugin/src/it/mplugin-223/invoker.properties
+++ b/maven-plugin-plugin/src/it/mplugin-223/invoker.properties
@@ -12,4 +12,4 @@
 # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and
 # limitations under the License.
-invoker.goals.1 = clean verify -DskipTests
+invoker.goals.1 = clean verify
diff --git a/maven-plugin-plugin/src/it/mplugin-299_no-configuration/invoker.properties b/maven-plugin-plugin/src/it/mplugin-299_no-configuration/invoker.properties
index a9041f9b..51bd1d3f 100644
--- a/maven-plugin-plugin/src/it/mplugin-299_no-configuration/invoker.properties
+++ b/maven-plugin-plugin/src/it/mplugin-299_no-configuration/invoker.properties
@@ -15,6 +15,5 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.maven.version = 3.0+
-invoker.goals.1 = clean install -DskipTests
+invoker.goals.1 = clean install
 invoker.goals.2 = --log-file help.log org.apache.maven.its.plugin:help:1.0-SNAPSHOT:help
diff --git a/maven-plugin-plugin/src/it/plugin-report-annotations/invoker.properties b/maven-plugin-plugin/src/it/plugin-report-annotations/invoker.properties
index abdde0d4..5cc536a2 100644
--- a/maven-plugin-plugin/src/it/plugin-report-annotations/invoker.properties
+++ b/maven-plugin-plugin/src/it/plugin-report-annotations/invoker.properties
@@ -15,4 +15,4 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals = clean site -DskipTests
+invoker.goals = clean site
diff --git a/maven-plugin-plugin/src/it/plugin-report/invoker.properties b/maven-plugin-plugin/src/it/plugin-report/invoker.properties
index abdde0d4..5cc536a2 100644
--- a/maven-plugin-plugin/src/it/plugin-report/invoker.properties
+++ b/maven-plugin-plugin/src/it/plugin-report/invoker.properties
@@ -15,4 +15,4 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals = clean site -DskipTests
+invoker.goals = clean site
diff --git a/maven-plugin-plugin/src/it/skip/invoker.properties b/maven-plugin-plugin/src/it/skip/invoker.properties
index 8e51f719..b1922736 100644
--- a/maven-plugin-plugin/src/it/skip/invoker.properties
+++ b/maven-plugin-plugin/src/it/skip/invoker.properties
@@ -15,5 +15,5 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals = install -DskipTests
-invoker.mavenOpts = -Dmaven.plugin.skip=true
\ No newline at end of file
+invoker.goals = install
+invoker.mavenOpts = -Dmaven.plugin.skip=true