Change package structure

src/main/java/domain/Check.java

@@ -19,15 +19,15 @@ public abstract class Check {
 	/**
 	 * @param name The check's name, in camelCase, for identification in the configuration file
 	 */
-	Check(String name) {
+	protected Check(String name) {
 		this(name, true);
 	}
 
 	/**
 	 * @param name The check's name, in camelCase, for identification in the configuration file
 	 * @param isEnabledByDefault Whether the check is enabled by default (true for general checks, false for specialized tools and such)
 	 */
-	Check(String name, boolean isEnabledByDefault) {
+	protected Check(String name, boolean isEnabledByDefault) {
 		if (name == null) {throw new IllegalArgumentException("Check name cannot be null");}
 		this.name = name;
 		this.isEnabledByDefault = isEnabledByDefault;

src/main/java/domain/CheckRoster.java

@@ -1,5 +1,21 @@
 package domain;
 
+import domain.checks.AdapterPatternCheck;
+import domain.checks.ConstantInterfaceCheck;
+import domain.checks.ImmutableExceptionsCheck;
+import domain.checks.InformationHidingCheck;
+import domain.checks.LowCouplingCheck;
+import domain.checks.MethodLengthCheck;
+import domain.checks.NamingConventionsCheck;
+import domain.checks.NoGlobalVariablesCheck;
+import domain.checks.ObserverPatternCheck;
+import domain.checks.ParameterCountCheck;
+import domain.checks.PlantUMLGenerator;
+import domain.checks.ProgramToInterfaceNotImplementationCheck;
+import domain.checks.RequiredOverridesCheck;
+import domain.checks.StrategyPatternCheck;
+import domain.checks.UnusedAbstractionsCheck;
+
 public class CheckRoster {
 	/**
 	 * Array of all linter checks, in the order they should be displayed.

src/main/java/domain/AdapterPatternCheck.java → src/main/java/domain/checks/AdapterPatternCheck.java

@@ -1,10 +1,13 @@
-package domain;
+package domain.checks;
 
 import java.util.HashSet;
 import java.util.Set;
 import java.util.regex.Pattern;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 

src/main/java/domain/ConstantInterfaceCheck.java → src/main/java/domain/checks/ConstantInterfaceCheck.java

@@ -1,9 +1,12 @@
-package domain;
+package domain.checks;
 
 import java.util.HashSet;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.ClassType;

src/main/java/domain/CountCheckPropertyValidator.java → src/main/java/domain/checks/CountCheckPropertyValidator.java

@@ -1,6 +1,9 @@
-package domain;
+package domain.checks;
 
 import datasource.Configuration;
+import domain.Message;
+import domain.MessageLevel;
+
 import java.text.MessageFormat;
 
 /**

src/main/java/domain/GraphCheck.java → src/main/java/domain/checks/GraphCheck.java

@@ -1,8 +1,11 @@
-package domain;
+package domain.checks;
 
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.ClassGraph;
+import domain.Message;
 import domain.javadata.ClassDataCollection;
 
 public abstract class GraphCheck extends Check {

src/main/java/domain/ImmutableExceptionsCheck.java → src/main/java/domain/checks/ImmutableExceptionsCheck.java

@@ -1,11 +1,14 @@
-package domain;
+package domain.checks;
 
 import java.text.MessageFormat;
 import java.util.HashSet;
 import java.util.Set;
 import java.util.regex.Pattern;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.FieldData;

src/main/java/domain/InformationHidingCheck.java → src/main/java/domain/checks/InformationHidingCheck.java

@@ -1,4 +1,4 @@
-package domain;
+package domain.checks;
 
 import java.util.ArrayList;
 import java.util.HashMap;
@@ -7,6 +7,9 @@
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.AccessModifier;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;

src/main/java/domain/LowCouplingCheck.java → src/main/java/domain/checks/LowCouplingCheck.java

@@ -1,9 +1,12 @@
-package domain;
+package domain.checks;
 
 import java.text.MessageFormat;
 import java.util.*;
 
 import datasource.Configuration;
+import domain.ClassGraphIterator;
+import domain.Message;
+import domain.MessageLevel;
 
 public class LowCouplingCheck extends GraphCheck {
     private int maxInDegree;
@@ -145,4 +148,4 @@ public int compareTo(IntegerAndDegree o) {
             return Integer.compare(this.inDegree, o.inDegree);
         }
     }
-}
+}

src/main/java/domain/MethodLengthCheck.java → src/main/java/domain/checks/MethodLengthCheck.java

@@ -1,11 +1,14 @@
-package domain;
+package domain.checks;
 
 import java.text.MessageFormat;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.InstrData;

src/main/java/domain/NamingConventions.java → src/main/java/domain/checks/NamingConventions.java

@@ -1,4 +1,4 @@
-package domain;
+package domain.checks;
 
 import java.util.function.Function;
 

src/main/java/domain/NamingConventionsCheck.java → src/main/java/domain/checks/NamingConventionsCheck.java

@@ -1,10 +1,13 @@
-package domain;
+package domain.checks;
 
 import java.text.MessageFormat;
 import java.util.HashSet;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.ClassType;

src/main/java/domain/NoGlobalVariablesCheck.java → src/main/java/domain/checks/NoGlobalVariablesCheck.java

@@ -1,10 +1,13 @@
-package domain;
+package domain.checks;
 
 import java.text.MessageFormat;
 import java.util.HashSet;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.FieldData;

src/main/java/domain/ObserverPatternCheck.java → src/main/java/domain/checks/ObserverPatternCheck.java

@@ -1,9 +1,13 @@
-package domain;
+package domain.checks;
 
 import java.util.HashSet;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.ClassGraph;
+import domain.ClassGraphIterator;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassType;
 
@@ -182,4 +186,4 @@ private boolean isConcreteClass(ClassGraphIterator it, ClassGraphIterator it2, i
         return j != it2.getCurrent() && ClassGraph.checkHasA(graph.getWeight(j, it.getCurrent())) && (ClassGraph.checkImplement(graph.getWeight(j, it2.getCurrent())) || ClassGraph.checkExtend(graph.getWeight(j, it2.getCurrent())));
     }
 
-}
+}

src/main/java/domain/ParameterCountCheck.java → src/main/java/domain/checks/ParameterCountCheck.java

@@ -1,11 +1,14 @@
-package domain;
+package domain.checks;
 
 import java.text.MessageFormat;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.MethodData;

src/main/java/domain/PlantUMLGenerator.java → src/main/java/domain/checks/PlantUMLGenerator.java

@@ -1,4 +1,4 @@
-package domain;
+package domain.checks;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
 import java.nio.charset.Charset;
@@ -8,6 +8,10 @@
 import datasource.Configuration;
 import datasource.DataPrinter;
 import datasource.FullFilePrinter;
+import domain.ClassGraph;
+import domain.Message;
+import domain.MessageLevel;
+import domain.PackageStructure;
 import domain.javadata.AccessModifier;
 import domain.javadata.ClassData;
 import domain.javadata.ClassType;

src/main/java/domain/ProgramToInterfaceNotImplementationCheck.java → src/main/java/domain/checks/ProgramToInterfaceNotImplementationCheck.java

@@ -1,11 +1,14 @@
-package domain;
+package domain.checks;
 
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import java.util.regex.Pattern;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.ClassType;

src/main/java/domain/RequiredOverridesCheck.java → src/main/java/domain/checks/RequiredOverridesCheck.java

@@ -1,10 +1,13 @@
-package domain;
+package domain.checks;
 
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.MethodData;

src/main/java/domain/StrategyPatternCheck.java → src/main/java/domain/checks/StrategyPatternCheck.java

@@ -1,9 +1,12 @@
-package domain;
+package domain.checks;
 
 import java.util.ArrayList;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.ClassType;

src/main/java/domain/TypeValidator.java → src/main/java/domain/checks/TypeValidator.java

@@ -1,10 +1,12 @@
-package domain;
+package domain.checks;
 
 import java.text.MessageFormat;
 import java.util.Collection;
 import java.util.Set;
 import java.util.function.Function;
 
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.FieldData;
 import domain.javadata.MethodData;

src/main/java/domain/UnusedAbstractionsCheck.java → src/main/java/domain/checks/UnusedAbstractionsCheck.java

@@ -1,10 +1,13 @@
-package domain;
+package domain.checks;
 
 import java.util.ArrayList;
 import java.util.HashSet;
 import java.util.Set;
 
 import datasource.Configuration;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
 import domain.javadata.ClassData;
 import domain.javadata.ClassDataCollection;
 import domain.javadata.ClassType;

src/test/java/cli/CliSystemTest.java

@@ -16,10 +16,10 @@
 import datasource.Configuration;
 import datasource.DirLoader;
 import domain.Check;
-import domain.MethodLengthCheck;
-import domain.NamingConventionsCheck;
-import domain.NoGlobalVariablesCheck;
-import domain.ParameterCountCheck;
+import domain.checks.MethodLengthCheck;
+import domain.checks.NamingConventionsCheck;
+import domain.checks.NoGlobalVariablesCheck;
+import domain.checks.ParameterCountCheck;
 
 public class CliSystemTest {
 	private static final String TEST_RESOURCES_DIR_PATH = "src/test/resources/cli-system-test";

src/test/java/domain/AdapterPatternCheckTest.java → src/test/java/domain/checks/AdapterPatternCheckTest.java

@@ -1,4 +1,4 @@
-package domain;
+package domain.checks;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
@@ -12,6 +12,10 @@
 import datasource.Configuration;
 import datasource.DirLoader;
 import datasource.FilesLoader;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
+import domain.TestUtility;
 import domain.javadata.ClassDataCollection;
 
 public class AdapterPatternCheckTest {

src/test/java/domain/ConstantInterfaceCheckTest.java → src/test/java/domain/checks/ConstantInterfaceCheckTest.java

@@ -1,4 +1,4 @@
-package domain;
+package domain.checks;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
@@ -12,6 +12,10 @@
 import datasource.Configuration;
 import datasource.DirLoader;
 import datasource.FilesLoader;
+import domain.Check;
+import domain.Message;
+import domain.MessageLevel;
+import domain.TestUtility;
 import domain.javadata.ClassDataCollection;
 
 public class ConstantInterfaceCheckTest {

src/test/java/domain/CountCheckPropertyValidatorTest.java → src/test/java/domain/checks/CountCheckPropertyValidatorTest.java

@@ -1,4 +1,4 @@
-package domain;
+package domain.checks;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertThrows;
@@ -8,6 +8,8 @@
 import org.junit.jupiter.api.Test;
 
 import datasource.Configuration;
+import domain.Message;
+import domain.MessageLevel;
 
 public class CountCheckPropertyValidatorTest {
 	private static final String KEY = "key";