Reduce code duplication down to 15 lines per piece

.github/workflows/simian.yml

@@ -24,4 +24,4 @@ jobs:
       - run: |
           wget --quiet http://public.yegor256.com/simian.jar -O /tmp/simian.jar
       - run: |
-          java -jar /tmp/simian.jar -threshold=19 -excludes=**/gen -excludes=**/it **/*.java
+          java -jar /tmp/simian.jar -threshold=14 -excludes=**/gen -excludes=**/it **/*.java

eo-maven-plugin/src/main/java/org/eolang/maven/util/HmBase.java

@@ -26,14 +26,11 @@
 import com.jcabi.log.Logger;
 import java.io.File;
 import java.io.IOException;
-import java.io.InputStream;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import org.cactoos.Bytes;
 import org.cactoos.Input;
-import org.cactoos.Text;
 import org.cactoos.bytes.BytesOf;
-import org.cactoos.io.InputOf;
 import org.cactoos.io.OutputTo;
 import org.cactoos.io.TeeInput;
 import org.cactoos.scalar.IoChecked;
@@ -45,7 +42,7 @@
  * @since 0.27
  */
 @SuppressWarnings("PMD.TooManyMethods")
-public final class HmBase implements Home {
+public final class HmBase extends Home {
     /**
      * Current working directory.
      */
@@ -69,26 +66,6 @@ public HmBase(final Path path) {
         this.cwd = path;
     }
 
-    @Override
-    public void save(final String str, final Path path) throws IOException {
-        this.save(new InputOf(str), path);
-    }
-
-    @Override
-    public void save(final Text txt, final Path path) throws IOException {
-        this.save(new InputOf(txt), path);
-    }
-
-    @Override
-    public void save(final InputStream stream, final Path path) throws IOException {
-        this.save(new InputOf(stream), path);
-    }
-
-    @Override
-    public void save(final byte[] bytes, final Path path) throws IOException {
-        this.save(new InputOf(bytes), path);
-    }
-
     @Override
     public void save(final Input input, final Path path) throws IOException {
         final Path target = this.absolute(this.onlyRelative(path));

eo-maven-plugin/src/main/java/org/eolang/maven/util/HmOptional.java

@@ -25,19 +25,16 @@
 
 import com.jcabi.log.Logger;
 import java.io.IOException;
-import java.io.InputStream;
 import java.nio.file.Path;
 import org.cactoos.Bytes;
 import org.cactoos.Input;
-import org.cactoos.Text;
-import org.cactoos.io.InputOf;
 
 /**
  * Location for files that saves optionally.
  * @since 0.32.0
  */
 @SuppressWarnings("PMD.TooManyMethods")
-public final class HmOptional implements Home {
+public final class HmOptional extends Home {
     /**
      * Original home.
      */
@@ -59,26 +56,6 @@ public HmOptional(final Home home, final boolean rwte) {
         this.rewrite = rwte;
     }
 
-    @Override
-    public void save(final String str, final Path path) throws IOException {
-        this.save(new InputOf(str), path);
-    }
-
-    @Override
-    public void save(final Text txt, final Path path) throws IOException {
-        this.save(new InputOf(txt), path);
-    }
-
-    @Override
-    public void save(final InputStream stream, final Path path) throws IOException {
-        this.save(new InputOf(stream), path);
-    }
-
-    @Override
-    public void save(final byte[] bytes, final Path path) throws IOException {
-        this.save(new InputOf(bytes), path);
-    }
-
     @Override
     public void save(final Input input, final Path path) throws IOException {
         final Path target = this.absolute(this.onlyRelative(path));

eo-maven-plugin/src/main/java/org/eolang/maven/util/Home.java

@@ -29,20 +29,23 @@
 import org.cactoos.Bytes;
 import org.cactoos.Input;
 import org.cactoos.Text;
+import org.cactoos.io.InputOf;
 
 /**
  * Location for the files.
  * @since 0.32.0
  */
-public interface Home {
+public abstract class Home {
     /**
      * Saving string.
      *
      * @param str String
      * @param path Cwd-relative path to file
      * @throws IOException If fails
      */
-    void save(String str, Path path) throws IOException;
+    public void save(final String str, final Path path) throws IOException {
+        this.save(new InputOf(str), path);
+    }
 
     /**
      * Saving text.
@@ -51,7 +54,9 @@ public interface Home {
      * @param path Cwd-relative path to file
      * @throws IOException If fails
      */
-    void save(Text txt, Path path) throws IOException;
+    public void save(final Text txt, final Path path) throws IOException {
+        this.save(new InputOf(txt), path);
+    }
 
     /**
      * Saving stream.
@@ -60,7 +65,9 @@ public interface Home {
      * @param path Cwd-relative path to file
      * @throws IOException If fails
      */
-    void save(InputStream stream, Path path) throws IOException;
+    public void save(final InputStream stream, final Path path) throws IOException  {
+        this.save(new InputOf(stream), path);
+    }
 
     /**
      * Saving bytes.
@@ -69,7 +76,9 @@ public interface Home {
      * @param path Cwd-relative path to file
      * @throws IOException If fails
      */
-    void save(byte[] bytes, Path path) throws IOException;
+    public void save(final byte[] bytes, final Path path) throws IOException  {
+        this.save(new InputOf(bytes), path);
+    }
 
     /**
      * Saving input.
@@ -79,7 +88,7 @@ public interface Home {
      * @throws IOException If fails
      * @throws IllegalArgumentException If given path is absolute
      */
-    void save(Input input, Path path) throws IOException;
+    public abstract void save(Input input, Path path) throws IOException;
 
     /**
      * Check if exists.
@@ -88,7 +97,7 @@ public interface Home {
      * @return True if exists
      * @throws IllegalArgumentException If given path is absolute
      */
-    boolean exists(Path path);
+    public abstract boolean exists(Path path);
 
     /**
      * Load bytes from file by path.
@@ -99,21 +108,21 @@ public interface Home {
      *  if some exception happens during reading the file
      * @throws IllegalArgumentException If given path is absolute
      */
-    Bytes load(Path path) throws IOException;
+    public abstract Bytes load(Path path) throws IOException;
 
     /**
      * Absolute path to a file.
      *
      * @param path Cwd-relative path to file
      * @return Absolute path
      */
-    Path absolute(Path path);
+    public abstract Path absolute(Path path);
 
     /**
      * Verifies that given path is relative and throws exception.
      * @param path Path to be verified
      * @return Given path if it's relative
      * @throws IllegalArgumentException If given path is Absolute
      */
-    Path onlyRelative(Path path);
+    public abstract Path onlyRelative(Path path);
 }

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

@@ -32,6 +32,7 @@
 import java.util.Map;
 import java.util.Set;
 import java.util.UUID;
+import java.util.stream.Stream;
 import org.cactoos.text.TextOf;
 import org.eolang.maven.tojos.PlacedTojo;
 import org.eolang.maven.tojos.PlacedTojos;
@@ -41,6 +42,9 @@
 import org.hamcrest.io.FileMatchers;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.Arguments;
+import org.junit.jupiter.params.provider.MethodSource;
 
 /**
  * Test case for {@link UnplaceMojo}.
@@ -129,74 +133,46 @@ void keepsJarBecauseItIsStillInUse(@TempDir final Path temp) throws IOException
         );
     }
 
-    @Test
-    void unplacesWithKeepAndRemoveBinariesParamTogether(@TempDir final Path temp) throws Exception {
-        final Path placed = UnplaceMojoTest.placeClass(temp, UnplaceMojoTest.clazz(temp));
-        final Map<String, Path> res = new FakeMaven(temp)
-            .with("placed", placed.toFile())
-            .with("keepBinaries", UnplaceMojoTest.GLOB_PATTERN)
-            .with("removeBinaries", UnplaceMojoTest.GLOB_PATTERN)
-            .execute(UnplaceMojo.class)
-            .result();
-        MatcherAssert.assertThat(
-            "TO ADD ASSERTION MESSAGE",
-            res.values().stream().noneMatch(UnplaceMojoTest::isClass),
-            Matchers.is(true)
-        );
-        MatcherAssert.assertThat(
-            "TO ADD ASSERTION MESSAGE",
-            new TextOf(res.get(placed.getFileName().toString())).asString(),
-            Matchers.allOf(
-                Matchers.not(Matchers.containsString("false")),
-                Matchers.containsString("true")
-            )
-        );
-    }
-
-    @Test
-    void unplacesWithRemoveBinariesParam(@TempDir final Path temp) throws Exception {
-        final Path placed = UnplaceMojoTest.placeClass(temp, UnplaceMojoTest.clazz(temp));
-        final Map<String, Path> res = new FakeMaven(temp)
-            .with("placed", placed.toFile())
-            .with("removeBinaries", UnplaceMojoTest.GLOB_PATTERN)
-            .execute(UnplaceMojo.class)
-            .result();
-        MatcherAssert.assertThat(
-            "TO ADD ASSERTION MESSAGE",
-            res.values().stream().noneMatch(UnplaceMojoTest::isClass),
-            Matchers.is(true)
-        );
-        MatcherAssert.assertThat(
-            "TO ADD ASSERTION MESSAGE",
-            new TextOf(res.get(placed.getFileName().toString())).asString(),
-            Matchers.allOf(
-                Matchers.not(Matchers.containsString("false")),
-                Matchers.containsString("true")
-            )
-        );
-    }
-
-    @Test
-    void unplacesWithKeepBinariesParam(@TempDir final Path temp) throws Exception {
+    @ParameterizedTest
+    @MethodSource("testArgsProvider")
+    void unplacesWithKeepOrRemoveBinariesParam(final String[] params, @TempDir final Path temp)
+        throws Exception {
         final Path placed = UnplaceMojoTest.placeClass(temp, UnplaceMojoTest.clazz(temp));
-        final Map<String, Path> res = new FakeMaven(temp)
-            .with("placed", placed.toFile())
-            .with("keepBinaries", UnplaceMojoTest.GLOB_PATTERN)
-            .execute(UnplaceMojo.class)
-            .result();
-        MatcherAssert.assertThat(
-            "TO ADD ASSERTION MESSAGE",
-            res.values().stream().anyMatch(UnplaceMojoTest::isClass),
-            Matchers.is(true)
-        );
-        MatcherAssert.assertThat(
-            "TO ADD ASSERTION MESSAGE",
-            new TextOf(res.get(placed.getFileName().toString())).asString(),
-            Matchers.allOf(
-                Matchers.containsString("false"),
-                Matchers.not(Matchers.containsString("true"))
-            )
-        );
+        final FakeMaven maven = new FakeMaven(temp)
+            .with("placed", placed.toFile());
+        for (final String param : params) {
+            maven.with(param, UnplaceMojoTest.GLOB_PATTERN);
+        }
+        final Map<String, Path> res = maven.execute(UnplaceMojo.class).result();
+        if (params.length == 1 && "keepBinaries".equals(params[0])) {
+            MatcherAssert.assertThat(
+                "TO ADD ASSERTION MESSAGE",
+                res.values().stream().anyMatch(UnplaceMojoTest::isClass),
+                Matchers.is(true)
+            );
+            MatcherAssert.assertThat(
+                "TO ADD ASSERTION MESSAGE",
+                new TextOf(res.get(placed.getFileName().toString())).asString(),
+                Matchers.allOf(
+                    Matchers.containsString("false"),
+                    Matchers.not(Matchers.containsString("true"))
+                )
+            );
+        } else {
+            MatcherAssert.assertThat(
+                "TO ADD ASSERTION MESSAGE",
+                res.values().stream().noneMatch(UnplaceMojoTest::isClass),
+                Matchers.is(true)
+            );
+            MatcherAssert.assertThat(
+                "TO ADD ASSERTION MESSAGE",
+                new TextOf(res.get(placed.getFileName().toString())).asString(),
+                Matchers.allOf(
+                    Matchers.not(Matchers.containsString("false")),
+                    Matchers.containsString("true")
+                )
+            );
+        }
     }
 
     @Test
@@ -307,5 +283,19 @@ private static Path clazz(final Path temp) throws IOException {
     private static boolean isClass(final Path path) {
         return path.toString().endsWith(".class");
     }
+
+    /**
+     * Input arguments for unit tests.
+     *
+     * @return Stream of arguments.
+     */
+    @SuppressWarnings("PMD.UnusedPrivateMethod")
+    private static Stream<Arguments> testArgsProvider() {
+        return Stream.of(
+            Arguments.of((Object) new String[]{"keepBinaries"}),
+            Arguments.of((Object) new String[]{"removeBinaries"}),
+            Arguments.of((Object) new String[]{"keepBinaries", "removeBinaries"})
+        );
+    }
 }