You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by el...@apache.org on 2026/01/23 18:12:30 UTC

(maven-build-cache-extension) branch master updated: Remove undeclared NotNull annotations (#440)

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

elharo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-build-cache-extension.git


The following commit(s) were added to refs/heads/master by this push:
     new 31d3513  Remove undeclared NotNull annotations (#440)
31d3513 is described below

commit 31d3513a1ebb540104d3ce463c7090426e45b3c2
Author: Elliotte Rusty Harold <el...@users.noreply.github.com>
AuthorDate: Fri Jan 23 18:12:25 2026 +0000

    Remove undeclared NotNull annotations (#440)
---
 .../apache/maven/buildcache/BuildCacheMojosExecutionStrategyTest.java   | 2 --
 .../java/org/apache/maven/buildcache/LifecyclePhasesHelperTest.java     | 2 --
 src/test/java/org/apache/maven/buildcache/its/RemoteCacheDavTest.java   | 2 --
 3 files changed, 6 deletions(-)

diff --git a/src/test/java/org/apache/maven/buildcache/BuildCacheMojosExecutionStrategyTest.java b/src/test/java/org/apache/maven/buildcache/BuildCacheMojosExecutionStrategyTest.java
index 976a3f2..53f410d 100644
--- a/src/test/java/org/apache/maven/buildcache/BuildCacheMojosExecutionStrategyTest.java
+++ b/src/test/java/org/apache/maven/buildcache/BuildCacheMojosExecutionStrategyTest.java
@@ -34,7 +34,6 @@
 import org.apache.maven.plugin.MavenPluginManager;
 import org.apache.maven.plugin.MojoExecution;
 import org.apache.maven.project.MavenProject;
-import org.jetbrains.annotations.NotNull;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
@@ -191,7 +190,6 @@ void testMismatch() {
             assertFalse(strategy.isParamsMatched(projectMock, executionMock, testMojo, cacheRecordMock));
         }
 
-        @NotNull
         private Pair<TrackedProperty, PropertyValue> setupProperty(String propertyName, String value) {
             TrackedProperty config = new TrackedProperty();
             config.setPropertyName(propertyName);
diff --git a/src/test/java/org/apache/maven/buildcache/LifecyclePhasesHelperTest.java b/src/test/java/org/apache/maven/buildcache/LifecyclePhasesHelperTest.java
index 79f7fed..0b6a860 100644
--- a/src/test/java/org/apache/maven/buildcache/LifecyclePhasesHelperTest.java
+++ b/src/test/java/org/apache/maven/buildcache/LifecyclePhasesHelperTest.java
@@ -30,7 +30,6 @@
 import org.apache.maven.lifecycle.internal.stub.LifecyclesTestUtils;
 import org.apache.maven.plugin.MojoExecution;
 import org.apache.maven.project.MavenProject;
-import org.jetbrains.annotations.NotNull;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
@@ -362,7 +361,6 @@ private void publishForkedProjectEvent(MojoExecution origin) {
         lifecyclePhasesHelper.forkedProjectStarted(eventMock);
     }
 
-    @NotNull
     private static MojoExecution mockedMojoExecution(String phase) {
         MojoExecution mojoExecution = mock(MojoExecution.class);
         when(mojoExecution.getLifecyclePhase()).thenReturn(phase);
diff --git a/src/test/java/org/apache/maven/buildcache/its/RemoteCacheDavTest.java b/src/test/java/org/apache/maven/buildcache/its/RemoteCacheDavTest.java
index baf0984..209d126 100644
--- a/src/test/java/org/apache/maven/buildcache/its/RemoteCacheDavTest.java
+++ b/src/test/java/org/apache/maven/buildcache/its/RemoteCacheDavTest.java
@@ -36,7 +36,6 @@
 import org.apache.maven.buildcache.its.junit.IntegrationTestExtension;
 import org.apache.maven.it.VerificationException;
 import org.apache.maven.it.Verifier;
-import org.jetbrains.annotations.NotNull;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
@@ -222,7 +221,6 @@ private boolean hasBuildInfoXml(Path cache) throws IOException {
         return Files.walk(cache).anyMatch(isBuildInfoXml());
     }
 
-    @NotNull
     private Predicate<Path> isBuildInfoXml() {
         return p -> p.getFileName().toString().equals("buildinfo.xml");
     }