#128 all tests are private

pom.xml

@@ -134,7 +134,7 @@ OF THE POSSIBILITY OF SUCH DAMAGE.
           <plugin>
             <groupId>com.qulice</groupId>
             <artifactId>qulice-maven-plugin</artifactId>
-            <version>0.21.1</version>
+            <version>0.22.0</version>
             <configuration>
               <excludes combine.children="append">
                 <exclude>findbugs:.*</exclude>

src/it/it-decoloring/src/test/java/com/jcabi/log/MulticolorLayoutIntegrationTest.java

@@ -60,7 +60,7 @@ public final class MulticolorLayoutIntegrationTest {
      * when -Dcom.jcabi.log.coloring=false.
      */
     @Test
-    public void disablesDefaultColor() {
+    void disablesDefaultColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern(
             MulticolorLayoutIntegrationTest.CONV_PATTERN
@@ -97,7 +97,7 @@ public void disablesDefaultColor() {
      * MulticolorLayout disables the color that overwrites the default one.
      */
     @Test
-    public void disablesOverridenDefaultColor() {
+    void disablesOverridenDefaultColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern("[%color{%p}] %m");
         layout.setLevels("DEBUG:2;10");
@@ -134,7 +134,7 @@ public void disablesOverridenDefaultColor() {
      * when -Dcom.jcabi.log.coloring=false.
      */
     @Test
-    public void disablesConstantColor() {
+    void disablesConstantColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern("[%color-blue{%p}] %color-blue{%m}");
         final LoggingEvent event = Mockito.mock(LoggingEvent.class);
@@ -169,7 +169,7 @@ public void disablesConstantColor() {
      * MulticolorLayout disables the color that overwrites the constant one.
      */
     @Test
-    public void disablesOverridenConstantColor() {
+    void disablesOverridenConstantColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern("[%color-red{%p}] %color-red{%m}");
         layout.setColors("red:12");

src/test/java/com/jcabi/log/ConversionPatternTest.java

@@ -50,7 +50,7 @@ public final class ConversionPatternTest {
     private static final Colors COLORS = new Colors();
 
     @Test
-    public void testGenerateNoReplacement() {
+    void testGenerateNoReplacement() {
         MatcherAssert.assertThat(
             convert(""),
             Matchers.equalTo("")
@@ -74,15 +74,15 @@ public void testGenerateNoReplacement() {
     }
 
     @Test
-    public void testGenerateEmpty() {
+    void testGenerateEmpty() {
         MatcherAssert.assertThat(
             convert("%color{}"),
             Matchers.equalTo(colorWrap(""))
         );
     }
 
     @Test
-    public void testGenerateSimple() {
+    void testGenerateSimple() {
         MatcherAssert.assertThat(
             convert("%color{Hello World}"),
             Matchers.equalTo(colorWrap("Hello World"))
@@ -100,7 +100,7 @@ public void testGenerateSimple() {
     }
 
     @Test
-    public void testGenerateCurlyBraces() {
+    void testGenerateCurlyBraces() {
         MatcherAssert.assertThat(
             ConversionPatternTest.convert("%color{%c{1}}"),
             Matchers.equalTo(ConversionPatternTest.colorWrap("%c{1}"))

src/test/java/com/jcabi/log/DecorsManagerTest.java

@@ -41,15 +41,15 @@
 public final class DecorsManagerTest {
 
     @Test
-    public void hasBuiltInDecors() throws Exception {
+    void hasBuiltInDecors() throws Exception {
         MatcherAssert.assertThat(
             DecorsManager.decor("nano", 1L),
             Matchers.instanceOf(NanoDecor.class)
         );
     }
 
     @Test
-    public void throwsExceptionForAbsentDecor() {
+    void throwsExceptionForAbsentDecor() {
         Assertions.assertThrows(
             DecorException.class,
             () -> DecorsManager.decor("non-existing-formatter", null)

src/test/java/com/jcabi/log/DomDecorTest.java

@@ -46,7 +46,7 @@
 public final class DomDecorTest {
 
     @Test
-    public void convertsDocumentToText() throws Exception {
+    void convertsDocumentToText() throws Exception {
         final Document doc = DocumentBuilderFactory.newInstance()
             .newDocumentBuilder().newDocument();
         doc.appendChild(doc.createElement("root"));
@@ -60,7 +60,7 @@ public void convertsDocumentToText() throws Exception {
     }
 
     @Test
-    public void convertsNullToText() throws Exception {
+    void convertsNullToText() throws Exception {
         final Formattable decor = new DomDecor(null);
         final Appendable dest = Mockito.mock(Appendable.class);
         final Formatter fmt = new Formatter(dest);

src/test/java/com/jcabi/log/ExceptionDecorTest.java

@@ -44,7 +44,7 @@
 public final class ExceptionDecorTest {
 
     @Test
-    public void convertsExceptionToText() throws Exception {
+    void convertsExceptionToText() throws Exception {
         final Formattable decor = new ExceptionDecor(new IOException("ouch!"));
         final Appendable dest = Mockito.mock(Appendable.class);
         final Formatter fmt = new Formatter(dest);
@@ -64,7 +64,7 @@ public void convertsExceptionToText() throws Exception {
     }
 
     @Test
-    public void convertsNullToText() throws Exception {
+    void convertsNullToText() throws Exception {
         final Formattable decor = new ExceptionDecor(null);
         final Appendable dest = Mockito.mock(Appendable.class);
         final Formatter fmt = new Formatter(dest);

src/test/java/com/jcabi/log/FileDecorTest.java

@@ -49,7 +49,7 @@
 public final class FileDecorTest {
 
     @Test
-    public void simplyWorks() {
+    void simplyWorks() {
         MatcherAssert.assertThat(
             new Printed(new FileDecor("/tmp/test-me.txt"), 0, 0, 0).toString(),
             Matchers.endsWith("test-me.txt")

src/test/java/com/jcabi/log/LineNumberTest.java

@@ -54,7 +54,7 @@ public final class LineNumberTest {
     private static final String CONV_PATTERN = "%c:%L";
 
     @Test
-    public void testLineNumber() throws Exception {
+    void testLineNumber() throws Exception {
         final PatternLayout layout = new PatternLayout();
         layout.setConversionPattern(LineNumberTest.CONV_PATTERN);
         final org.apache.log4j.Logger root = LogManager.getRootLogger();

src/test/java/com/jcabi/log/LoggerTest.java

@@ -47,38 +47,38 @@
 public final class LoggerTest {
 
     @Test
-    public void detectsLoggerNameCorrectly() {
+    void detectsLoggerNameCorrectly() {
         // not implemented yet
     }
 
     @Test
-    public void detectsNameOfStaticSource() {
+    void detectsNameOfStaticSource() {
         // not implemented yet
     }
 
     @Test
-    public void setsLoggingLevel() {
+    void setsLoggingLevel() {
         // not implemented yet
     }
 
     @Test
-    public void doesntFormatArraysSinceTheyAreVarArgs() {
+    void doesntFormatArraysSinceTheyAreVarArgs() {
         Assertions.assertThrows(
             IllegalArgumentException.class,
             () -> Logger.format("array: %[list]s", new Object[] {"hi", 1})
         );
     }
 
     @Test
-    public void interpretsArraysAsVarArgs() {
+    void interpretsArraysAsVarArgs() {
         MatcherAssert.assertThat(
             Logger.format("array: %s : %d", new Object[] {"hello", 2}),
             Matchers.is("array: hello : 2")
         );
     }
 
     @Test
-    public void providesOutputStream() throws Exception {
+    void providesOutputStream() throws Exception {
         final OutputStream stream = Logger.stream(Level.INFO, this);
         final PrintWriter writer = new PrintWriter(
             new OutputStreamWriter(stream, "UTF-8")
@@ -90,23 +90,23 @@ public void providesOutputStream() throws Exception {
     }
 
     @Test
-    public void throwsWhenParamsLessThanFormatArgs() {
+    void throwsWhenParamsLessThanFormatArgs() {
         Assertions.assertThrows(
             ArrayIndexOutOfBoundsException.class,
             () -> Logger.format("String %s Char %c Number %d", "howdy", 'x')
         );
     }
 
     @Test
-    public void throwsWhenParamsMoreThanFormatArgs() {
+    void throwsWhenParamsMoreThanFormatArgs() {
         Assertions.assertThrows(
             IllegalArgumentException.class,
             () -> Logger.format("String %s Number %d Char %c", "hey", 1, 'x', 2)
         );
     }
 
     @Test
-    public void checksLogLevel() throws Exception {
+    void checksLogLevel() throws Exception {
         LogManager.getRootLogger().setLevel(org.apache.log4j.Level.INFO);
         TimeUnit.MILLISECONDS.sleep(1L);
         MatcherAssert.assertThat(
@@ -120,12 +120,12 @@ public void checksLogLevel() throws Exception {
     }
 
     @Test
-    public void usesStringAsLoggerName() {
+    void usesStringAsLoggerName() {
         Logger.info("com.jcabi.log...why.not", "hello, %s!", "world!");
     }
 
     @Test
-    public void findsArgsByPositions() {
+void findsArgsByPositions() {
         final String first = "xyz";
         final String second = "ddd";
         MatcherAssert.assertThat(

src/test/java/com/jcabi/log/MsDecorTest.java

@@ -70,7 +70,7 @@ public void testLogsRight(final long value, final String text,
     }
 
     @Test
-    public void testPrintsNullRight() {
+    void testPrintsNullRight() {
         MatcherAssert.assertThat(
             new Logged(new MsDecor(null), 0, 0, 0),
             Matchers.hasToString("NULL")

src/test/java/com/jcabi/log/MulticolorLayoutTest.java

@@ -51,7 +51,7 @@ public final class MulticolorLayoutTest {
     private static final String CONV_PATTERN = "[%color{%p}] %color{%m}";
 
     @Test
-    public void transformsLoggingEventToText() {
+    void transformsLoggingEventToText() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern(MulticolorLayoutTest.CONV_PATTERN);
         final LoggingEvent event = Mockito.mock(LoggingEvent.class);
@@ -66,7 +66,7 @@ public void transformsLoggingEventToText() {
     }
 
     @Test
-    public void overwriteDefaultColor() {
+    void overwriteDefaultColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern(MulticolorLayoutTest.CONV_PATTERN);
         layout.setLevels("INFO:2;10");
@@ -82,7 +82,7 @@ public void overwriteDefaultColor() {
     }
 
     @Test
-    public void rendersCustomConstantColor() {
+    void rendersCustomConstantColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern("%color-red{%p} %m");
         final LoggingEvent event = Mockito.mock(LoggingEvent.class);
@@ -95,7 +95,7 @@ public void rendersCustomConstantColor() {
     }
 
     @Test
-    public void overwriteCustomConstantColor() {
+    void overwriteCustomConstantColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern("%color-white{%p} %m");
         layout.setColors("white:10");
@@ -109,7 +109,7 @@ public void overwriteCustomConstantColor() {
     }
 
     @Test
-    public void rendersAnsiConstantColor() {
+    void rendersAnsiConstantColor() {
         final MulticolorLayout layout = new MulticolorLayout();
         layout.setConversionPattern("%color-0;0;31{%p} %m");
         final LoggingEvent event = Mockito.mock(LoggingEvent.class);
@@ -122,7 +122,7 @@ public void rendersAnsiConstantColor() {
     }
 
     @Test
-    public void throwsOnIllegalColorName() {
+    void throwsOnIllegalColorName() {
         Assertions.assertThrows(
             IllegalArgumentException.class,
             () -> {

src/test/java/com/jcabi/log/NanoDecorTest.java

@@ -70,7 +70,7 @@ public void testLogsRight(final long nano, final String text,
     }
 
     @Test
-    public void testPrintsNullRight() {
+    void testPrintsNullRight() {
         MatcherAssert.assertThat(
             new Logged(new NanoDecor(null), 0, 0, 0),
             Matchers.hasToString("NULL")

src/test/java/com/jcabi/log/ParseableInformationTest.java

@@ -60,7 +60,7 @@ public final void parsesTheInformationCorrectly() {
      * ParseableInformation can throw an an exception when parsing wrong info.
      */
     @Test
-    public final void throwsAnExceptionWhenParsingSomethingWrong() {
+    final void throwsAnExceptionWhenParsingSomethingWrong() {
         final String white = "white";
         try {
             new ParseableInformation(white).information();

src/test/java/com/jcabi/log/ParseableLevelInformationTest.java

@@ -61,7 +61,7 @@ public final void parsesCorrectlyTheInformation() {
      * not with the right pattern.
      */
     @Test
-    public final void throwsAnExceptionWhenParsingIncorrectInformation() {
+    final void throwsAnExceptionWhenParsingIncorrectInformation() {
         final String wrong = "INFO;10,WARN;32";
         try {
             new ParseableLevelInformation(wrong).information();
@@ -85,7 +85,7 @@ public final void throwsAnExceptionWhenParsingIncorrectInformation() {
      * with a wrong type of level.
      */
     @Test
-    public final void throwsAnExceptionWhenParsingWrongLevelType() {
+    final void throwsAnExceptionWhenParsingWrongLevelType() {
         try {
             new ParseableLevelInformation(
                 "INFO:2;10,EXTREME:2;32"

src/test/java/com/jcabi/log/PreFormatterTest.java

@@ -43,7 +43,7 @@ public final class PreFormatterTest {
      * PreFormatter can format simple texts.
      */
     @Test
-    public void decoratesArguments() {
+    void decoratesArguments() {
         final PreFormatter pre = new PreFormatter(
             "%[com.jcabi.log.DecorMocker]-5.2f and %1$+.6f",
             1.0d
@@ -62,7 +62,7 @@ public void decoratesArguments() {
      * PreFormatter can handle missed decors.
      */
     @Test
-    public void formatsEvenWithMissedDecors() {
+    void formatsEvenWithMissedDecors() {
         final PreFormatter pre =
             new PreFormatter("ouch: %[missed]s", "test");
         MatcherAssert.assertThat(
@@ -79,7 +79,7 @@ public void formatsEvenWithMissedDecors() {
      * PreFormatter can handle directly provided decors.
      */
     @Test
-    public void formatsWithDirectlyProvidedDecors() {
+    void formatsWithDirectlyProvidedDecors() {
         final DecorMocker decor = new DecorMocker("a");
         final PreFormatter pre = new PreFormatter("test: %s", decor);
         MatcherAssert.assertThat(
@@ -92,7 +92,7 @@ public void formatsWithDirectlyProvidedDecors() {
      * PreFormatter can handle new line specifier.
      */
     @Test
-    public void handleNewLineSpecifier() {
+    void handleNewLineSpecifier() {
         final String fmt = "%s%n%s";
         final Object[] args = {"new", "line"};
         final PreFormatter pre = new PreFormatter(fmt, args);
@@ -110,7 +110,7 @@ public void handleNewLineSpecifier() {
      * PreFormatter can handle percent specifier.
      */
     @Test
-    public void handlePercentSpecifier() {
+    void handlePercentSpecifier() {
         final String fmt = "%s%%";
         final Object[] args = {"percent: "};
         final PreFormatter pre = new PreFormatter(fmt, args);

src/test/java/com/jcabi/log/SizeDecorTest.java

@@ -70,7 +70,7 @@ public void testLogsRight(final long size, final String text,
     }
 
     @Test
-    public void testPrintsNullRight() {
+    void testPrintsNullRight() {
         MatcherAssert.assertThat(
             new Logged(new SizeDecor(null), 0, 0, 0),
             Matchers.hasToString("NULL")