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:34 UTC

[maven-plugin-tools] branch rm-skip-test created (now 4e8459e9)

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

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


      at 4e8459e9 Cleanup IT tests - remove skipTests

This branch includes the following new commits:

     new 4e8459e9 Cleanup IT tests - remove skipTests

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



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

Posted by sj...@apache.org.
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