Fix the code quality errors in the tests

eo-maven-plugin/src/test/java/org/eolang/maven/ChTextTest.java

@@ -28,18 +28,12 @@
 @ExtendWith(MktmpResolver.class)
 final class ChTextTest {
 
-    /**
-     * Test file path in temp dir.
-     */
-    private static Path file;
-
     @BeforeAll
     @ExtendWith(MktmpResolver.class)
     static void setUp(@Mktmp final Path dir) throws IOException {
-        ChTextTest.file = dir.resolve("tags.txt");
         new Saved(
             new ResourceOf("org/eolang/maven/commits/tags.txt"),
-            ChTextTest.file
+            dir.resolve("tags.txt")
         ).value();
     }
 
@@ -67,12 +61,13 @@ static void setUp(@Mktmp final Path dir) throws IOException {
         "be83d9adda4b7c9e670e625fe951c80f3ead4177, 0.28.9"
     })
     void readsCorrectHashByTagFromFile(
+        @Mktmp final Path dir,
         final String hash,
         final String tag
     ) {
         MatcherAssert.assertThat(
             "ChText should read the correct hash by tag from the file, but it didn't",
-            new ChText(ChTextTest.file, tag).value(),
+            new ChText(dir.resolve("tags.txt"), tag).value(),
             Matchers.equalTo(hash)
         );
     }

eo-maven-plugin/src/test/java/org/eolang/maven/FpDefaultTest.java

@@ -25,32 +25,13 @@
 @SuppressWarnings("PMD.TooManyMethods")
 @ExtendWith(MktmpResolver.class)
 final class FpDefaultTest {
-    /**
-     * Lambda content.
-     */
-    private static final String LAMBDA_CONTENT = "Footprint content";
-
-    /**
-     * Target content.
-     */
-    private static final String TARGET_CONTENT = "Target content";
-
-    /**
-     * Cache content.
-     */
-    private static final String CACHE_CONTENT = "Cache content";
-
-    /**
-     * Snapshot.
-     */
-    private static final String SNAPSHOT = "SNAPSHOT";
 
     @Test
     void failsIfSourcePathNotExists() {
         Assertions.assertThrows(
             IllegalStateException.class,
             () -> new FpDefault(
-                src -> FpDefaultTest.LAMBDA_CONTENT,
+                src -> "Footprint content",
                 Paths.get(""),
                 "",
                 "",
@@ -66,7 +47,7 @@ void doesNothingWhenTargetIsOlderThanSource(@Mktmp final Path temp) throws Excep
         final Path target = FpDefaultTest.existedTarget(temp);
         FpDefaultTest.makeOlder(target);
         final Path result = new FpDefault(
-            src -> FpDefaultTest.LAMBDA_CONTENT,
+            src -> "Footprint content",
             temp,
             "1.2.3",
             "abcdef",
@@ -80,7 +61,7 @@ void doesNothingWhenTargetIsOlderThanSource(@Mktmp final Path temp) throws Excep
         MatcherAssert.assertThat(
             "The content of target file has not to be changed",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.TARGET_CONTENT)
+            Matchers.equalTo("Target content")
         );
     }
 
@@ -92,20 +73,20 @@ void updatesOnlyTargetFromSourceIfNoTargetAndCacheIsNotCacheable(
         final Path target = FpDefaultTest.notExistedTarget(temp);
         assert Files.notExists(target);
         new FpDefault(
-            src -> FpDefaultTest.LAMBDA_CONTENT,
+            src -> "Footprint content",
             temp,
-            FpDefaultTest.SNAPSHOT,
+            "SNAPSHOT",
             "",
             Paths.get("cache1.txt")
         ).apply(source, target);
         MatcherAssert.assertThat(
             "Target file must be updated from content function, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
         MatcherAssert.assertThat(
             "Cache file has not to be updated",
-            temp.resolve(FpDefaultTest.SNAPSHOT).resolve("cache.txt").toFile().exists(),
+            temp.resolve("SNAPSHOT").resolve("cache.txt").toFile().exists(),
             Matchers.equalTo(false)
         );
     }
@@ -118,20 +99,20 @@ void updatesOnlyTargetFromSourceIfYoungerTargetAndCacheIsNotCacheable(
         final Path target = FpDefaultTest.existedTarget(temp);
         FpDefaultTest.makeOlder(source);
         new FpDefault(
-            src -> FpDefaultTest.LAMBDA_CONTENT,
+            src -> "Footprint content",
             temp,
-            FpDefaultTest.SNAPSHOT,
+            "SNAPSHOT",
             "",
             Paths.get("cache2.txt")
         ).apply(source, target);
         MatcherAssert.assertThat(
             "Target file must be updated from content function, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
         MatcherAssert.assertThat(
             "Cache file has not to be updated",
-            temp.resolve(FpDefaultTest.SNAPSHOT).resolve("cache.txt").toFile().exists(),
+            temp.resolve("SNAPSHOT").resolve("cache.txt").toFile().exists(),
             Matchers.equalTo(false)
         );
     }
@@ -149,12 +130,12 @@ void updatesBothIfTargetYoungerAndNotExistedCacheableCache(
         MatcherAssert.assertThat(
             "Target content must be updated from lambda, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
         MatcherAssert.assertThat(
             "Cache content must be updated from lambda, but it didn't",
             new TextOf(cache.path()).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
     }
 
@@ -168,12 +149,12 @@ void updatesBothNoTargetAndNotExistedCacheableCache(@Mktmp final Path temp) thro
         MatcherAssert.assertThat(
             "Target content must be updated from lambda, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
         MatcherAssert.assertThat(
             "Cache content must be updated from lambda, but it didn't",
             new TextOf(cache.path()).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
     }
 
@@ -190,12 +171,12 @@ void updatesBothIfTargetYoungerAndExistedCacheableCacheIsYounger(
         MatcherAssert.assertThat(
             "Target content must be updated from lambda, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
         MatcherAssert.assertThat(
             "Cache content must be updated from lambda, but it didn't",
             new TextOf(cache.path()).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
     }
 
@@ -212,12 +193,12 @@ void updatesBothIfNoTargetAndExistedCacheableCacheIsYounger(
         MatcherAssert.assertThat(
             "Target content must be updated from lambda, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
         MatcherAssert.assertThat(
             "Cache content must be updated from lambda, but it didn't",
             new TextOf(cache.path()).asString(),
-            Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT)
+            Matchers.equalTo("Footprint content")
         );
     }
 
@@ -234,12 +215,12 @@ void copiesFromCacheIfTargetYoungerAndExistedCacheableCacheOlder(
         MatcherAssert.assertThat(
             "Target content must be updated from cache, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.CACHE_CONTENT)
+            Matchers.equalTo("Cache content")
         );
         MatcherAssert.assertThat(
             "Cache content must not be changed, but it did",
             new TextOf(cache.path()).asString(),
-            Matchers.equalTo(FpDefaultTest.CACHE_CONTENT)
+            Matchers.equalTo("Cache content")
         );
     }
 
@@ -257,12 +238,12 @@ void copiesFromCacheIfNoTaretAndExistedCacheableCacheOlder(
         MatcherAssert.assertThat(
             "Target content must be updated from cache, but it didn't",
             new TextOf(target).asString(),
-            Matchers.equalTo(FpDefaultTest.CACHE_CONTENT)
+            Matchers.equalTo("Cache content")
         );
         MatcherAssert.assertThat(
             "Cache content must not be changed, but it did",
             new TextOf(cache.path()).asString(),
-            Matchers.equalTo(FpDefaultTest.CACHE_CONTENT)
+            Matchers.equalTo("Cache content")
         );
     }
 
@@ -275,7 +256,7 @@ void skipsCacheIfItIsNotEnabled(@Mktmp final Path temp) throws Exception {
         FpDefaultTest.makeOlder(source);
         FpDefaultTest.makeOlder(cache.path(), 100_000);
         new FpDefault(
-            src -> FpDefaultTest.LAMBDA_CONTENT,
+            src -> "Footprint content",
             cache.base,
             cache.semver,
             () -> cache.hash,
@@ -286,8 +267,8 @@ void skipsCacheIfItIsNotEnabled(@Mktmp final Path temp) throws Exception {
             "Global cache must be skipped because of the boolean flag",
             new TextOf(target).asString(),
             Matchers.allOf(
-                Matchers.equalTo(FpDefaultTest.LAMBDA_CONTENT),
-                Matchers.not(Matchers.equalTo(FpDefaultTest.CACHE_CONTENT))
+                Matchers.equalTo("Footprint content"),
+                Matchers.not(Matchers.equalTo("Cache content"))
             )
         );
     }
@@ -300,7 +281,7 @@ void throwsNpeIfHashIsNull(@Mktmp final Path temp) throws IOException {
         Assertions.assertThrows(
             NullPointerException.class,
             () -> new FpDefault(
-                src1 -> FpDefaultTest.LAMBDA_CONTENT,
+                src1 -> "Footprint content",
                 cache.base,
                 cache.semver,
                 null,
@@ -321,7 +302,7 @@ private static void defaultFootprint(
         final Cache cache, final Path source, final Path target
     ) throws Exception {
         new FpDefault(
-            src -> FpDefaultTest.LAMBDA_CONTENT,
+            src -> "Footprint content",
             cache.base,
             cache.semver,
             cache.hash,
@@ -368,7 +349,7 @@ private static Path notExistedSource(final Path temp) {
      */
     private static Path existedTarget(final Path temp) throws IOException {
         final Path source = FpDefaultTest.notExistedTarget(temp);
-        return FpDefaultTest.existedFile(source, FpDefaultTest.TARGET_CONTENT);
+        return FpDefaultTest.existedFile(source, "Target content");
     }
 
     /**
@@ -410,7 +391,7 @@ private static void makeOlder(final Path file, final long time) throws IOExcepti
      */
     private static Cache existedCache(final Path temp) throws IOException {
         final Cache cache = FpDefaultTest.notExistedCache(temp);
-        FpDefaultTest.existedFile(cache.path(), FpDefaultTest.CACHE_CONTENT);
+        FpDefaultTest.existedFile(cache.path(), "Cache content");
         return cache;
     }