#128 clean

jcabi · Dec 26, 2023 · 2b827a1 · 2b827a1
1 parent ef0626c
commit 2b827a1
Show file tree

Hide file tree

Showing 16 changed files with 33 additions and 33 deletions.
diff --git a/.rultor.yml b/.rultor.yml
@@ -1,5 +1,5 @@
 docker:
-  image: yegor256/rultor-image:1.19.0
+  image: yegor256/rultor-image:1.22.0
 assets:
   settings.xml: yegor256/home#assets/jcabi/settings.xml
   secring.gpg: yegor256/home#assets/secring.gpg

diff --git a/src/test/java/com/jcabi/log/FileDecorTest.java b/src/test/java/com/jcabi/log/FileDecorTest.java
@@ -58,7 +58,7 @@ void simplyWorks() {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final Object path, final String text,
+    void testPrintsRight(final Object path, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -69,7 +69,7 @@ public void testPrintsRight(final Object path, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final Object path, final String text,
+    void testLogsRight(final Object path, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/ListDecorTest.java b/src/test/java/com/jcabi/log/ListDecorTest.java
@@ -48,7 +48,7 @@ final class ListDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final Object list, final String text,
+    void testPrintsRight(final Object list, final String text,
         final int flags, final int width, final int precision) throws DecorException {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -59,7 +59,7 @@ public void testPrintsRight(final Object list, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final Object list, final String text,
+    void testLogsRight(final Object list, final String text,
         final int flags, final int width, final int precision) throws DecorException {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/LoggerTest.java b/src/test/java/com/jcabi/log/LoggerTest.java
@@ -125,7 +125,7 @@ void usesStringAsLoggerName() {
     }
 
     @Test
-void findsArgsByPositions() {
+    void findsArgsByPositions() {
         final String first = "xyz";
         final String second = "ddd";
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/MsDecorTest.java b/src/test/java/com/jcabi/log/MsDecorTest.java
@@ -49,7 +49,7 @@ final class MsDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final long value, final String text,
+    void testPrintsRight(final long value, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -60,7 +60,7 @@ public void testPrintsRight(final long value, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final long value, final String text,
+    void testLogsRight(final long value, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/NanoDecorTest.java b/src/test/java/com/jcabi/log/NanoDecorTest.java
@@ -49,7 +49,7 @@ final class NanoDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final long nano, final String text,
+    void testPrintsRight(final long nano, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -60,7 +60,7 @@ public void testPrintsRight(final long nano, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final long nano, final String text,
+    void testLogsRight(final long nano, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/ObjectDecorTest.java b/src/test/java/com/jcabi/log/ObjectDecorTest.java
@@ -47,7 +47,7 @@ final class ObjectDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final Object obj, final String text,
+    void testPrintsRight(final Object obj, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -58,7 +58,7 @@ public void testPrintsRight(final Object obj, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final Object obj, final String text,
+    void testLogsRight(final Object obj, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/ParseableInformationTest.java b/src/test/java/com/jcabi/log/ParseableInformationTest.java
@@ -40,15 +40,15 @@
  * ParseableInformation test case.
  * @since 0.18
  */
-public class ParseableInformationTest {
+class ParseableInformationTest {
 
     /**
      * ParseableInformation can parse if the information correctly if is using
      * the right pattern.
      */
     @Test
     @SuppressWarnings("PMD.UseConcurrentHashMap")
-    public final void parsesTheInformationCorrectly() {
+    final void parsesTheInformationCorrectly() {
         final Map<String, String> parsed = new ParseableInformation(
             "red:10,black:20"
         ).information();

diff --git a/src/test/java/com/jcabi/log/ParseableLevelInformationTest.java b/src/test/java/com/jcabi/log/ParseableLevelInformationTest.java
@@ -40,15 +40,15 @@
  * ParseableLevelInformation test case.
  * @since 0.18
  */
-public class ParseableLevelInformationTest {
+class ParseableLevelInformationTest {
 
     /**
      * ParseableLevelInformation can parse the information correctly when it's
      * with the right pattern.
      */
     @Test
     @SuppressWarnings("PMD.UseConcurrentHashMap")
-    public final void parsesCorrectlyTheInformation() {
+    final void parsesCorrectlyTheInformation() {
         final Map<String, String> parsed = new ParseableLevelInformation(
             "INFO:2;10,WARN:2;32"
         ).information();

diff --git a/src/test/java/com/jcabi/log/SecretDecorTest.java b/src/test/java/com/jcabi/log/SecretDecorTest.java
@@ -48,7 +48,7 @@ final class SecretDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final Object list, final String text,
+    void testPrintsRight(final Object list, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -59,7 +59,7 @@ public void testPrintsRight(final Object list, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final Object list, final String text,
+    void testLogsRight(final Object list, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/SizeDecorTest.java b/src/test/java/com/jcabi/log/SizeDecorTest.java
@@ -49,7 +49,7 @@ final class SizeDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final long size, final String text,
+    void testPrintsRight(final long size, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -60,7 +60,7 @@ public void testPrintsRight(final long size, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final long size, final String text,
+    void testLogsRight(final long size, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/SupplierLoggerTest.java b/src/test/java/com/jcabi/log/SupplierLoggerTest.java
@@ -64,7 +64,7 @@ void debugIsDisabled() {
 
     @Test
     @Disabled
-    public void debugIsEnabled() {
+    void debugIsEnabled() {
         final String name = "debugen";
         final String appender = "debugapp";
         final org.apache.log4j.Logger logger = this.loggerForTest(
@@ -135,7 +135,7 @@ void warnIsDisabled() {
 
     @Test
     @Disabled
-    public void warnIsEnabled() {
+    void warnIsEnabled() {
         final String name = "enwarn";
         final String appender = "warnapp";
         final org.apache.log4j.Logger logger = this.loggerForTest(
@@ -171,7 +171,7 @@ void infoIsDisabled() {
 
     @Test
     @Disabled
-    public void infoIsEnabled() {
+    void infoIsEnabled() {
         final String name = "withinfo";
         final String appender = "infoapp";
         final org.apache.log4j.Logger logger = this.loggerForTest(
@@ -195,7 +195,7 @@ public void infoIsEnabled() {
      * @param level Logging level
      * @return Logger for test
      */
-    public org.apache.log4j.Logger loggerForTest(
+    private org.apache.log4j.Logger loggerForTest(
         final String name, final String appender, final Level level) {
         final org.apache.log4j.Logger logger = org.apache.log4j.Logger
             .getLogger(name);

diff --git a/src/test/java/com/jcabi/log/TextDecorTest.java b/src/test/java/com/jcabi/log/TextDecorTest.java
@@ -53,7 +53,7 @@ final class TextDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final String obj, final String text,
+    void testPrintsRight(final String obj, final String text,
         final int flags, final int width, final int precision) {
         Assumptions.assumeTrue("UTF-8".equals(Charset.defaultCharset().name()));
         Locale.setDefault(Locale.US);
@@ -65,7 +65,7 @@ public void testPrintsRight(final String obj, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final String obj, final String text,
+    void testLogsRight(final String obj, final String text,
         final int flags, final int width, final int precision) {
         Assumptions.assumeTrue("UTF-8".equals(Charset.defaultCharset().name()));
         Locale.setDefault(Locale.US);

diff --git a/src/test/java/com/jcabi/log/TypeDecorTest.java b/src/test/java/com/jcabi/log/TypeDecorTest.java
@@ -47,7 +47,7 @@ final class TypeDecorTest {
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testPrintsRight(final Object list, final String text,
+    void testPrintsRight(final Object list, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(
@@ -58,7 +58,7 @@ public void testPrintsRight(final Object list, final String text,
 
     @ParameterizedTest
     @MethodSource("params")
-    public void testLogsRight(final Object list, final String text,
+    void testLogsRight(final Object list, final String text,
         final int flags, final int width, final int precision) {
         Locale.setDefault(Locale.US);
         MatcherAssert.assertThat(

diff --git a/src/test/java/com/jcabi/log/UnitTestAppender.java b/src/test/java/com/jcabi/log/UnitTestAppender.java
@@ -53,7 +53,7 @@ final class UnitTestAppender extends WriterAppender {
      * @param name The appender's name
      */
     @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors")
-    public UnitTestAppender(final String name) {
+    UnitTestAppender(final String name) {
         super();
         this.setName(name);
         this.logs = new ByteArrayOutputStream();

diff --git a/src/test/java/com/jcabi/log/VerboseProcessTest.java b/src/test/java/com/jcabi/log/VerboseProcessTest.java
@@ -71,7 +71,7 @@ final class VerboseProcessTest {
 
     @Test
     @Disabled
-    public void runsACommandLineScript() {
+    void runsACommandLineScript() {
         Assumptions.assumeFalse(SystemUtils.IS_OS_WINDOWS, "");
         final VerboseProcess process = new VerboseProcess(
             new ProcessBuilder("echo", "hey \u20ac!").redirectErrorStream(true)
@@ -84,7 +84,7 @@ public void runsACommandLineScript() {
 
     @Test
     @Disabled
-    public void echosUnicodeCorrectly() {
+    void echosUnicodeCorrectly() {
         Assumptions.assumeFalse(SystemUtils.IS_OS_WINDOWS, "");
         MatcherAssert.assertThat(
             new VerboseProcess(
@@ -197,7 +197,7 @@ void rejectsStderrWithLevelAll() {
 
     @Test
     @SuppressWarnings("PMD.DoNotUseThreads")
-    public void quietlyTerminatesLongRunningProcess() throws Exception {
+    void quietlyTerminatesLongRunningProcess() throws Exception {
         Assumptions.assumeFalse(SystemUtils.IS_OS_WINDOWS, "");
         final Process proc = new ProcessBuilder("sleep", "10000").start();
         final VerboseProcess process = new VerboseProcess(proc);
@@ -248,7 +248,7 @@ void stdoutQuietlyLogsErrors() {
 
     @Test
     @SuppressWarnings("PMD.AvoidFileStream")
-    public void logsErrorWhenUnderlyingStreamIsClosed() throws Exception {
+    void logsErrorWhenUnderlyingStreamIsClosed() throws Exception {
         final StringWriter writer = new StringWriter();
         org.apache.log4j.Logger.getRootLogger().addAppender(
             new WriterAppender(new SimpleLayout(), writer)