make small fixes

README.md

@@ -1 +1,5 @@
-# test-naming-conventions
+# test-naming-conventions
+
+```shell
+mvn com.github.volodya-lombrozo:test-naming-conventions:check
+```

pom.xml

@@ -7,7 +7,7 @@
 
     <groupId>com.github.volodya-lombrozo</groupId>
     <artifactId>test-naming-conventions</artifactId>
-    <version>0.1.0</version>
+    <version>0.1.1</version>
     <packaging>maven-plugin</packaging>
     <description>Plugin for checking test naming rules</description>
 

src/main/java/com/github/lombrozo/testnames/JavaTestCode.java

@@ -8,7 +8,6 @@
 import java.nio.file.Path;
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.List;
 
 public final class JavaTestCode {
@@ -34,8 +33,7 @@ public Collection<String> names() {
             }
             return names;
         } catch (IOException ex) {
-            ex.printStackTrace();
+            throw new IllegalStateException(ex);
         }
-        return Collections.emptyList();
     }
 }

src/main/java/com/github/lombrozo/testnames/RuleForAllTests.java

@@ -1,8 +1,6 @@
 package com.github.lombrozo.testnames;
 
-import java.util.ArrayList;
 import java.util.Collection;
-import java.util.List;
 
 public final class RuleForAllTests implements Rule {
 
@@ -15,16 +13,8 @@ public RuleForAllTests(final JavaTestCode tests) {
     @Override
     public void validate() throws WrongTestName {
         final Collection<String> names = tests.names();
-        final List<WrongTestName> exceptions = new ArrayList<>(names.size());
         for (String test : names) {
-            try {
-                new PresentSimpleRule(test).validate();
-            } catch (WrongTestName ex){
-                exceptions.add(ex);
-            }
-        }
-        if(!exceptions.isEmpty()){
-            throw new WrongTestName(exceptions);
+            new PresentSimpleRule(test).validate();
         }
     }
-}
+}

src/main/java/com/github/lombrozo/testnames/ValidateMojo.java

@@ -3,6 +3,7 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
+import java.util.ArrayList;
 import java.util.List;
 import java.util.stream.Collectors;
 import org.apache.maven.plugin.AbstractMojo;
@@ -26,12 +27,20 @@ public void execute() throws MojoExecutionException, MojoFailureException {
                     Paths.get(project.getTestCompileSourceRoots().get(0)))
                 .filter(Files::exists)
                 .filter(Files::isRegularFile).collect(Collectors.toList());
+            final List<WrongTestName> exceptions = new ArrayList<>();
             for (final Path test : tests) {
-                new RuleForAllTests(new JavaTestCode(test)).validate();
+                try {
+                    new RuleForAllTests(new JavaTestCode(test)).validate();
+                } catch (final WrongTestName ex) {
+                    exceptions.add(ex);
+                }
+            }
+            if (!exceptions.isEmpty()) {
+                throw new WrongTestName(exceptions);
             }
         } catch (final WrongTestName ex) {
             throw new MojoFailureException(ex);
-        } catch (final Exception occasion){
+        } catch (final Exception occasion) {
             throw new MojoExecutionException(occasion);
         }
     }