Skip to content

Commit 29d1c0d

Browse files
slachiewiczslawekjaranowski
authored andcommitted
[MENFORCER-497] Require Maven 3.6.3+
1 parent 80e6626 commit 29d1c0d

File tree

14 files changed

+30
-146
lines changed

14 files changed

+30
-146
lines changed

enforcer-rules/pom.xml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,12 @@
7171
<scope>provided</scope>
7272
</dependency>
7373
<dependency>
74-
<groupId>org.eclipse.aether</groupId>
75-
<artifactId>aether-api</artifactId>
74+
<groupId>org.apache.maven.resolver</groupId>
75+
<artifactId>maven-resolver-api</artifactId>
7676
</dependency>
7777
<dependency>
78-
<groupId>org.eclipse.aether</groupId>
79-
<artifactId>aether-util</artifactId>
78+
<groupId>org.apache.maven.resolver</groupId>
79+
<artifactId>maven-resolver-util</artifactId>
8080
</dependency>
8181
<dependency>
8282
<groupId>org.eclipse.sisu</groupId>

maven-enforcer-extension/src/it/projects/invoker.properties

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
# KIND, either express or implied. See the License for the
1515
# specific language governing permissions and limitations
1616
# under the License.
17-
invoker.maven.version = 3.3.1+
1817
invoker.goals.1 = validate
1918
invoker.goals.2 = verify
2019
invoker.buildResult.2 = failure

maven-enforcer-plugin/src/it/projects/dependencies_converge_exclude_wildcard/invoker.properties

Lines changed: 0 additions & 18 deletions
This file was deleted.

maven-enforcer-plugin/src/it/projects/require-no-repositories_mm_ci/invoker.properties

Lines changed: 0 additions & 18 deletions
This file was deleted.

maven-enforcer-plugin/src/it/projects/require-plugin-versions-ci/invoker.properties

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,4 @@
1515
# specific language governing permissions and limitations
1616
# under the License.
1717

18-
invoker.maven.version = 3.5.0+
19-
invoker.buildResult=failure
18+
invoker.buildResult=failure

maven-enforcer-plugin/src/it/projects/require-plugin-versions-expressions/invoker.properties

Lines changed: 0 additions & 25 deletions
This file was deleted.

maven-enforcer-plugin/src/it/projects/require-plugin-versions-mm-ci-friendly/invoker.properties

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,4 @@
1414
# KIND, either express or implied. See the License for the
1515
# specific language governing permissions and limitations
1616
# under the License.
17-
invoker.maven.version = 3.5.0+
18-
invoker.goals = install -Drevision=0.10.0-SNAPSHOT
17+
invoker.goals = install -Drevision=0.10.0-SNAPSHOT

maven-enforcer-plugin/src/it/projects/require-plugin-versions-plugin-with-integration-test-lifecycle/invoker.properties

Lines changed: 0 additions & 18 deletions
This file was deleted.

maven-enforcer-plugin/src/it/projects/require-snapshot-version_failure_parent-ci-friendly/invoker.properties

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,4 @@
1515
# specific language governing permissions and limitations
1616
# under the License.
1717

18-
invoker.maven.version = 3.5.0+
1918
invoker.buildResult = failure

maven-enforcer-plugin/src/it/projects/require-snapshot-version_success_parent-ci-friendly/invoker.properties

Lines changed: 0 additions & 18 deletions
This file was deleted.

maven-enforcer-plugin/src/it/projects/require-upper-bound-exclude_wildcard/invoker.properties

Lines changed: 0 additions & 18 deletions
This file was deleted.

maven-enforcer-plugin/src/test/java/org/apache/maven/plugins/enforcer/TestEnforceMojo.java

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import java.util.Arrays;
2222
import java.util.Collections;
2323
import java.util.List;
24+
import java.util.Properties;
2425

2526
import org.apache.maven.enforcer.rule.api.EnforcerLevel;
2627
import org.apache.maven.enforcer.rule.api.EnforcerRule;
@@ -33,21 +34,21 @@
3334
import org.apache.maven.plugins.enforcer.internal.EnforcerRuleDesc;
3435
import org.apache.maven.plugins.enforcer.internal.EnforcerRuleManager;
3536
import org.assertj.core.api.Assertions;
37+
import org.junit.jupiter.api.BeforeEach;
3638
import org.junit.jupiter.api.Test;
3739
import org.junit.jupiter.api.extension.ExtendWith;
3840
import org.mockito.InjectMocks;
3941
import org.mockito.Mock;
4042
import org.mockito.Mockito;
4143
import org.mockito.junit.jupiter.MockitoExtension;
42-
import org.mockito.junit.jupiter.MockitoSettings;
43-
import org.mockito.quality.Strictness;
4444

4545
import static org.assertj.core.api.Assertions.assertThatCode;
4646
import static org.assertj.core.api.Assertions.assertThatThrownBy;
4747
import static org.junit.jupiter.api.Assertions.assertFalse;
4848
import static org.junit.jupiter.api.Assertions.assertTrue;
4949
import static org.junit.jupiter.api.Assertions.fail;
5050
import static org.mockito.ArgumentMatchers.any;
51+
import static org.mockito.Mockito.lenient;
5152
import static org.mockito.Mockito.verify;
5253
import static org.mockito.Mockito.when;
5354

@@ -57,7 +58,6 @@
5758
* @author <a href="mailto:brianf@apache.org">Brian Fox</a>
5859
*/
5960
@ExtendWith(MockitoExtension.class)
60-
@MockitoSettings(strictness = Strictness.LENIENT)
6161
class TestEnforceMojo {
6262

6363
@Mock
@@ -69,6 +69,12 @@ class TestEnforceMojo {
6969
@InjectMocks
7070
private EnforceMojo mojo;
7171

72+
@BeforeEach
73+
void setup() {
74+
lenient().when(session.getSystemProperties()).thenReturn(new Properties());
75+
lenient().when(session.getUserProperties()).thenReturn(new Properties());
76+
}
77+
7278
@Test
7379
void emptyRuleListShouldThrowException() {
7480
mojo.setFail(false);

maven-enforcer-plugin/src/test/java/org/apache/maven/plugins/enforcer/internal/EnforcerRuleManagerTest.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import javax.inject.Provider;
2222

2323
import java.util.List;
24+
import java.util.Properties;
2425

2526
import org.apache.maven.enforcer.rule.api.EnforcerLevel;
2627
import org.apache.maven.enforcer.rule.api.EnforcerRuleBase;
@@ -94,7 +95,10 @@ void setupMocks(Boolean hasComponent) {
9495

9596
when(mojoDescriptor.getPluginDescriptor()).thenReturn(mock(PluginDescriptor.class));
9697

97-
when(sessionProvider.get()).thenReturn(mock(MavenSession.class));
98+
MavenSession mavenSession = mock(MavenSession.class);
99+
when(mavenSession.getSystemProperties()).thenReturn(new Properties());
100+
when(mavenSession.getUserProperties()).thenReturn(new Properties());
101+
when(sessionProvider.get()).thenReturn(mavenSession);
98102

99103
when(plexusContainer.hasComponent(any(Class.class), anyString())).thenReturn(hasComponent);
100104
}

pom.xml

Lines changed: 10 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -77,13 +77,13 @@
7777
</distributionManagement>
7878

7979
<properties>
80-
<mavenVersion>3.2.5</mavenVersion>
80+
<mavenVersion>3.6.3</mavenVersion>
8181
<maven.site.path>enforcer-archives/enforcer-LATEST</maven.site.path>
8282
<javaVersion>8</javaVersion>
8383
<mockito.version>4.11.0</mockito.version>
84-
<project.build.outputTimestamp>2024-05-26T08:52:43Z</project.build.outputTimestamp>
85-
<!-- the same as Maven 3.2.5 -->
86-
<aether.version>1.0.0.v20140518</aether.version>
84+
<project.build.outputTimestamp>2023-09-07T17:27:52Z</project.build.outputTimestamp>
85+
<!-- the same as Maven 3.6.3 -->
86+
<resolver.version>1.4.1</resolver.version>
8787

8888
<!-- plugins used in IT, not defined in parent -->
8989
<version.maven-pmd-plugin>3.21.0</version.maven-pmd-plugin>
@@ -149,15 +149,15 @@
149149
<scope>provided</scope>
150150
</dependency>
151151
<dependency>
152-
<groupId>org.eclipse.aether</groupId>
153-
<artifactId>aether-api</artifactId>
154-
<version>${aether.version}</version>
152+
<groupId>org.apache.maven.resolver</groupId>
153+
<artifactId>maven-resolver-api</artifactId>
154+
<version>${resolver.version}</version>
155155
<scope>provided</scope>
156156
</dependency>
157157
<dependency>
158-
<groupId>org.eclipse.aether</groupId>
159-
<artifactId>aether-util</artifactId>
160-
<version>${aether.version}</version>
158+
<groupId>org.apache.maven.resolver</groupId>
159+
<artifactId>maven-resolver-util</artifactId>
160+
<version>${resolver.version}</version>
161161
</dependency>
162162
<dependency>
163163
<groupId>org.eclipse.sisu</groupId>
@@ -202,13 +202,6 @@
202202
</dependency>
203203

204204
<!-- test dependencies -->
205-
<dependency>
206-
<groupId>org.junit</groupId>
207-
<artifactId>junit-bom</artifactId>
208-
<version>5.10.2</version>
209-
<type>pom</type>
210-
<scope>import</scope>
211-
</dependency>
212205
<dependency>
213206
<groupId>org.mockito</groupId>
214207
<artifactId>mockito-core</artifactId>

0 commit comments

Comments
 (0)