diff --git a/src/it/it-decoloring/src/main/java/com/jcabi/log/MulticolorLayoutIntegration.java b/src/it/it-decoloring/src/main/java/com/jcabi/log/MulticolorLayoutIntegration.java index 1f053b75..ecd01704 100644 --- a/src/it/it-decoloring/src/main/java/com/jcabi/log/MulticolorLayoutIntegration.java +++ b/src/it/it-decoloring/src/main/java/com/jcabi/log/MulticolorLayoutIntegration.java @@ -34,8 +34,6 @@ * be run with maven-invoker-plugin because they set the system property * com.jcabi.log.coloring, which interferes with other tests (tests are run * by multiple threads at once) - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ * @since 0.1 */ public final class MulticolorLayoutIntegration { diff --git a/src/it/it-decoloring/src/main/java/com/jcabi/log/package-info.java b/src/it/it-decoloring/src/main/java/com/jcabi/log/package-info.java index 5987230e..e02cfea3 100644 --- a/src/it/it-decoloring/src/main/java/com/jcabi/log/package-info.java +++ b/src/it/it-decoloring/src/main/java/com/jcabi/log/package-info.java @@ -30,7 +30,5 @@ /** * Integration tests. - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ */ package com.jcabi.log; diff --git a/src/it/it-decoloring/src/test/java/com/jcabi/log/MulticolorLayoutIntegrationTest.java b/src/it/it-decoloring/src/test/java/com/jcabi/log/MulticolorLayoutIntegrationTest.java index 06172b00..d87fb548 100644 --- a/src/it/it-decoloring/src/test/java/com/jcabi/log/MulticolorLayoutIntegrationTest.java +++ b/src/it/it-decoloring/src/test/java/com/jcabi/log/MulticolorLayoutIntegrationTest.java @@ -42,8 +42,6 @@ * be run with maven-invoker-plugin because they set the system property * com.jcabi.log.coloring, which interferes with other tests (tests are run * by multiple threads at once) - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ * @since 0.1 */ public final class MulticolorLayoutIntegrationTest { @@ -51,6 +49,7 @@ public final class MulticolorLayoutIntegrationTest { * Conversation pattern for test case. */ private static final String CONV_PATTERN = "[%color{%p}] %color{%m}"; + /** * Property that dictates wheter the text should be coloured or not. */ diff --git a/src/it/it-decoloring/src/test/java/com/jcabi/log/package-info.java b/src/it/it-decoloring/src/test/java/com/jcabi/log/package-info.java index 313bf599..e02cfea3 100644 --- a/src/it/it-decoloring/src/test/java/com/jcabi/log/package-info.java +++ b/src/it/it-decoloring/src/test/java/com/jcabi/log/package-info.java @@ -30,7 +30,5 @@ /** * Integration tests. - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ */ -package com.jcabi.log.test; +package com.jcabi.log; diff --git a/src/main/java/com/jcabi/log/ColorfullyFormatted.java b/src/main/java/com/jcabi/log/ColorfullyFormatted.java index 3e8a4e1c..80cf6b57 100644 --- a/src/main/java/com/jcabi/log/ColorfullyFormatted.java +++ b/src/main/java/com/jcabi/log/ColorfullyFormatted.java @@ -31,8 +31,6 @@ /** * Formats a log event using ANSI color codes. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ class ColorfullyFormatted implements Formatted { diff --git a/src/main/java/com/jcabi/log/Colors.java b/src/main/java/com/jcabi/log/Colors.java index 02da07a5..f44fc99e 100644 --- a/src/main/java/com/jcabi/log/Colors.java +++ b/src/main/java/com/jcabi/log/Colors.java @@ -34,8 +34,6 @@ /** * Store human readable color data. - * @author Igor Piddubnyi (igor.piddubnyi@gmail.com) - * @version $Id$ * @since 0.18 */ public class Colors { diff --git a/src/main/java/com/jcabi/log/ControlSequenceIndicatorFormatted.java b/src/main/java/com/jcabi/log/ControlSequenceIndicatorFormatted.java index 5c84b740..5d9998cc 100644 --- a/src/main/java/com/jcabi/log/ControlSequenceIndicatorFormatted.java +++ b/src/main/java/com/jcabi/log/ControlSequenceIndicatorFormatted.java @@ -31,8 +31,6 @@ /** * Replaces string format with a Control Sequence Indicator. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ public class ControlSequenceIndicatorFormatted implements Formatted { diff --git a/src/main/java/com/jcabi/log/ConversionPattern.java b/src/main/java/com/jcabi/log/ConversionPattern.java index 8d89a8d6..0b1af6f7 100644 --- a/src/main/java/com/jcabi/log/ConversionPattern.java +++ b/src/main/java/com/jcabi/log/ConversionPattern.java @@ -34,8 +34,6 @@ /** * Generates the conversion pattern. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ class ConversionPattern { diff --git a/src/main/java/com/jcabi/log/DecorException.java b/src/main/java/com/jcabi/log/DecorException.java index 465c076b..f92935a1 100644 --- a/src/main/java/com/jcabi/log/DecorException.java +++ b/src/main/java/com/jcabi/log/DecorException.java @@ -35,8 +35,6 @@ /** * Exception if some problem with decor. * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/DecorsManager.java b/src/main/java/com/jcabi/log/DecorsManager.java index 4a6fd756..5f00b33b 100644 --- a/src/main/java/com/jcabi/log/DecorsManager.java +++ b/src/main/java/com/jcabi/log/DecorsManager.java @@ -40,9 +40,6 @@ /** * Manager of all decors. * - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/DomDecor.java b/src/main/java/com/jcabi/log/DomDecor.java index cb8cac3c..0ee158ef 100644 --- a/src/main/java/com/jcabi/log/DomDecor.java +++ b/src/main/java/com/jcabi/log/DomDecor.java @@ -46,8 +46,6 @@ /** * Decorates XML Document. * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/DullyFormatted.java b/src/main/java/com/jcabi/log/DullyFormatted.java index d6bad35c..28f6892e 100644 --- a/src/main/java/com/jcabi/log/DullyFormatted.java +++ b/src/main/java/com/jcabi/log/DullyFormatted.java @@ -31,8 +31,6 @@ /** * Formats a log event without using ANSI color codes. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ class DullyFormatted implements Formatted { diff --git a/src/main/java/com/jcabi/log/ExceptionDecor.java b/src/main/java/com/jcabi/log/ExceptionDecor.java index 0ea61a92..9a9b528d 100644 --- a/src/main/java/com/jcabi/log/ExceptionDecor.java +++ b/src/main/java/com/jcabi/log/ExceptionDecor.java @@ -51,8 +51,6 @@ * } * * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/Formatted.java b/src/main/java/com/jcabi/log/Formatted.java index df241ee1..8499a5cc 100644 --- a/src/main/java/com/jcabi/log/Formatted.java +++ b/src/main/java/com/jcabi/log/Formatted.java @@ -31,8 +31,6 @@ /** * Contract for a class that know how to format something. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ interface Formatted { diff --git a/src/main/java/com/jcabi/log/ListDecor.java b/src/main/java/com/jcabi/log/ListDecor.java index 3a18efc6..3d581866 100644 --- a/src/main/java/com/jcabi/log/ListDecor.java +++ b/src/main/java/com/jcabi/log/ListDecor.java @@ -38,8 +38,6 @@ /** * Format list. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/Logger.java b/src/main/java/com/jcabi/log/Logger.java index d7f624f1..5463437c 100644 --- a/src/main/java/com/jcabi/log/Logger.java +++ b/src/main/java/com/jcabi/log/Logger.java @@ -83,8 +83,6 @@ * preparing of the arguments. By default, such a call is made inside every * method of {@link Logger} class. * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/MsDecor.java b/src/main/java/com/jcabi/log/MsDecor.java index 037a8419..8c908547 100644 --- a/src/main/java/com/jcabi/log/MsDecor.java +++ b/src/main/java/com/jcabi/log/MsDecor.java @@ -46,8 +46,6 @@ * Logger.debug("completed in %[ms]s", System.currentTimeMillis() - start); * * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/MulticolorLayout.java b/src/main/java/com/jcabi/log/MulticolorLayout.java index aa1506d0..3669ca3c 100644 --- a/src/main/java/com/jcabi/log/MulticolorLayout.java +++ b/src/main/java/com/jcabi/log/MulticolorLayout.java @@ -74,8 +74,6 @@ * <groupId>com.jcabi</groupId> * <artifactId>jcabi-log</artifactId> * </dependency> - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1.10 * @see ANSI escape code * @see PatternLayout from LOG4J diff --git a/src/main/java/com/jcabi/log/NanoDecor.java b/src/main/java/com/jcabi/log/NanoDecor.java index 94f68984..352456a3 100644 --- a/src/main/java/com/jcabi/log/NanoDecor.java +++ b/src/main/java/com/jcabi/log/NanoDecor.java @@ -46,9 +46,6 @@ * Logger.debug("completed in %[nano]s", System.nanoTime() - start); * * - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/ObjectDecor.java b/src/main/java/com/jcabi/log/ObjectDecor.java index 7d6587d3..63225a48 100644 --- a/src/main/java/com/jcabi/log/ObjectDecor.java +++ b/src/main/java/com/jcabi/log/ObjectDecor.java @@ -40,9 +40,6 @@ /** * Format internal structure of an object. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString @@ -85,8 +82,8 @@ public void formatTo(final Formatter formatter, final int flags, /** * {@link PrivilegedAction} for obtaining array contents. - * @author Aleksey Popov (alopen@yandex.ru) - * @version $Id$ + * + * @since 0.1 */ private static final class ArrayFormatAction implements PrivilegedAction { @@ -120,9 +117,8 @@ public String run() { /** * {@link PrivilegedAction} for obtaining object contents. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ + * + * @since 0.1 */ private static final class ObjectContentsFormatAction implements PrivilegedAction { @@ -161,6 +157,6 @@ public String run() { } builder.replace(builder.length() - 2, builder.length(), "}"); return builder.toString(); - }; + } } } diff --git a/src/main/java/com/jcabi/log/ParseableInformation.java b/src/main/java/com/jcabi/log/ParseableInformation.java index 1a9449d8..45a0ddd3 100644 --- a/src/main/java/com/jcabi/log/ParseableInformation.java +++ b/src/main/java/com/jcabi/log/ParseableInformation.java @@ -35,8 +35,6 @@ /** * Converts items inside a string like K1:V1,K2:V2 - where K is for key and V * is for value - to a {@code Map} of string key and string value. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ class ParseableInformation { diff --git a/src/main/java/com/jcabi/log/ParseableLevelInformation.java b/src/main/java/com/jcabi/log/ParseableLevelInformation.java index 9b8a4a7e..4f4e3360 100644 --- a/src/main/java/com/jcabi/log/ParseableLevelInformation.java +++ b/src/main/java/com/jcabi/log/ParseableLevelInformation.java @@ -38,8 +38,6 @@ /** * Parse information like {@code ParseInformation} does, but increments with * some extra checks for {@code Level}s. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ class ParseableLevelInformation { diff --git a/src/main/java/com/jcabi/log/PreFormatter.java b/src/main/java/com/jcabi/log/PreFormatter.java index 6e784613..97b32145 100644 --- a/src/main/java/com/jcabi/log/PreFormatter.java +++ b/src/main/java/com/jcabi/log/PreFormatter.java @@ -41,9 +41,6 @@ * Processor of formatting string and arguments, before sending it to * {@link String#format(String,Object[])}. * - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/SecretDecor.java b/src/main/java/com/jcabi/log/SecretDecor.java index 34c9981e..27fad88a 100644 --- a/src/main/java/com/jcabi/log/SecretDecor.java +++ b/src/main/java/com/jcabi/log/SecretDecor.java @@ -37,9 +37,6 @@ /** * Decorator of a secret text. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/SizeDecor.java b/src/main/java/com/jcabi/log/SizeDecor.java index a546a6a5..10ec86f8 100644 --- a/src/main/java/com/jcabi/log/SizeDecor.java +++ b/src/main/java/com/jcabi/log/SizeDecor.java @@ -39,10 +39,6 @@ /** * Size decorator. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @author Carlos Miranda (miranda.cma@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/Supplier.java b/src/main/java/com/jcabi/log/Supplier.java index 0766aa92..2d2c45bd 100644 --- a/src/main/java/com/jcabi/log/Supplier.java +++ b/src/main/java/com/jcabi/log/Supplier.java @@ -34,8 +34,6 @@ * or method references. Can be used for method referencing when the method * signature respects the following: returns something and takes no arguments. * - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ * @param The type of results supplied by this supplier * @since 0.18 */ diff --git a/src/main/java/com/jcabi/log/SupplierLogger.java b/src/main/java/com/jcabi/log/SupplierLogger.java index df4a1022..be744d88 100644 --- a/src/main/java/com/jcabi/log/SupplierLogger.java +++ b/src/main/java/com/jcabi/log/SupplierLogger.java @@ -32,8 +32,6 @@ /** * Logging methods which take {@link Supplier} arguments. * Used with Java 8 method referencing. - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ * @since 0.18 * @checkstyle HideUtilityClassConstructorCheck (500 lines) */ diff --git a/src/main/java/com/jcabi/log/TextDecor.java b/src/main/java/com/jcabi/log/TextDecor.java index 60202f52..2619e293 100644 --- a/src/main/java/com/jcabi/log/TextDecor.java +++ b/src/main/java/com/jcabi/log/TextDecor.java @@ -43,8 +43,6 @@ * Logger.debug("Long input '%[text]s' provided", input); * } * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1.5 */ @ToString diff --git a/src/main/java/com/jcabi/log/TypeDecor.java b/src/main/java/com/jcabi/log/TypeDecor.java index 6806dc9b..c9bc07a5 100644 --- a/src/main/java/com/jcabi/log/TypeDecor.java +++ b/src/main/java/com/jcabi/log/TypeDecor.java @@ -45,8 +45,6 @@ * } * * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @ToString diff --git a/src/main/java/com/jcabi/log/VerboseCallable.java b/src/main/java/com/jcabi/log/VerboseCallable.java index 7d339905..b3689c11 100644 --- a/src/main/java/com/jcabi/log/VerboseCallable.java +++ b/src/main/java/com/jcabi/log/VerboseCallable.java @@ -52,8 +52,6 @@ * *

This class is thread-safe. * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.16 * @param Type of result * @see VerboseThreads @@ -122,6 +120,7 @@ public T call() throws Exception { runnable.run(); return null; } + @Override public String toString() { return runnable.toString(); diff --git a/src/main/java/com/jcabi/log/VerboseProcess.java b/src/main/java/com/jcabi/log/VerboseProcess.java index c297c79b..ae3d3ad8 100644 --- a/src/main/java/com/jcabi/log/VerboseProcess.java +++ b/src/main/java/com/jcabi/log/VerboseProcess.java @@ -62,8 +62,6 @@ * *

The class is thread-safe. * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.5 */ @ToString @@ -405,24 +403,30 @@ private static void close(final Closeable res) { /** * Stream monitor. + * + * @since 0.1 */ private static final class Monitor implements Callable { /** * Stream to read. */ private final transient InputStream input; + /** * Latch to count down when done. */ private final transient CountDownLatch done; + /** * Buffer to save output. */ private final transient OutputStream output; + /** * Log level. */ private final transient Level level; + /** * Ctor. * @param inp Stream to monitor @@ -438,6 +442,7 @@ private static final class Monitor implements Callable { this.output = out; this.level = lvl; } + @Override public Void call() throws Exception { final BufferedReader reader = new BufferedReader( @@ -495,6 +500,8 @@ public Void call() throws Exception { /** * Class representing the result of a process. + * + * @since 0.1 */ public static final class Result { diff --git a/src/main/java/com/jcabi/log/VerboseRunnable.java b/src/main/java/com/jcabi/log/VerboseRunnable.java index d96e9e09..ec9dd483 100644 --- a/src/main/java/com/jcabi/log/VerboseRunnable.java +++ b/src/main/java/com/jcabi/log/VerboseRunnable.java @@ -51,8 +51,6 @@ * *

This class is thread-safe. * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1.3 * @see VerboseThreads * @link Java theory and practice: Dealing with InterruptedException @@ -138,6 +136,7 @@ public void run() { throw new IllegalStateException(ex); } } + @Override public String toString() { return callable.toString(); diff --git a/src/main/java/com/jcabi/log/VerboseThreads.java b/src/main/java/com/jcabi/log/VerboseThreads.java index 542b1855..15e2959c 100644 --- a/src/main/java/com/jcabi/log/VerboseThreads.java +++ b/src/main/java/com/jcabi/log/VerboseThreads.java @@ -78,8 +78,6 @@ * *

This class is thread-safe. * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1.2 * @see VerboseRunnable */ @@ -183,6 +181,8 @@ public Thread newThread(final Runnable runnable) { /** * Group to use. + * + * @since 0.1 */ private static final class Group extends ThreadGroup { /** @@ -192,6 +192,7 @@ private static final class Group extends ThreadGroup { Group(final String name) { super(name); } + @Override public void uncaughtException(final Thread thread, final Throwable throwable) { @@ -201,12 +202,15 @@ public void uncaughtException(final Thread thread, /** * Runnable decorator. + * + * @since 0.1 */ private static final class Wrap implements Runnable { /** * Origin runnable. */ private final transient Runnable origin; + /** * Ctor. * @param runnable Origin runnable @@ -214,6 +218,7 @@ private static final class Wrap implements Runnable { Wrap(final Runnable runnable) { this.origin = runnable; } + @Override @SuppressWarnings("PMD.AvoidCatchingGenericException") public void run() { diff --git a/src/main/java/com/jcabi/log/package-info.java b/src/main/java/com/jcabi/log/package-info.java index da3798bb..6b1965af 100644 --- a/src/main/java/com/jcabi/log/package-info.java +++ b/src/main/java/com/jcabi/log/package-info.java @@ -39,8 +39,6 @@ * <artifactId>jcabi-log</artifactId> * </dependency> * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @see project website */ package com.jcabi.log; diff --git a/src/test/java/com/jcabi/log/ConversionPatternTest.java b/src/test/java/com/jcabi/log/ConversionPatternTest.java index 88753a55..09a496f7 100644 --- a/src/test/java/com/jcabi/log/ConversionPatternTest.java +++ b/src/test/java/com/jcabi/log/ConversionPatternTest.java @@ -35,8 +35,6 @@ /** * Test case for {@link ConversionPattern}. - * @author Csaba Kos (csaba@indeed.com) - * @version $Id$ * @since 0.19 */ @SuppressWarnings("PMD.AvoidDuplicateLiterals") diff --git a/src/test/java/com/jcabi/log/DecorMocker.java b/src/test/java/com/jcabi/log/DecorMocker.java index 40f1b12d..b6e3ce3d 100644 --- a/src/test/java/com/jcabi/log/DecorMocker.java +++ b/src/test/java/com/jcabi/log/DecorMocker.java @@ -34,9 +34,6 @@ /** * Primitive decor, for testing only. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ public final class DecorMocker implements Formattable { diff --git a/src/test/java/com/jcabi/log/DecorsManagerTest.java b/src/test/java/com/jcabi/log/DecorsManagerTest.java index cfddaf1a..92e3bfb9 100644 --- a/src/test/java/com/jcabi/log/DecorsManagerTest.java +++ b/src/test/java/com/jcabi/log/DecorsManagerTest.java @@ -36,9 +36,6 @@ /** * Test case for {@link DecorsManager}. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ public final class DecorsManagerTest { @@ -55,11 +52,8 @@ public void hasBuiltInDecors() throws Exception { public void throwsExceptionForAbsentDecor() { Assertions.assertThrows( DecorException.class, - () -> { - DecorsManager.decor("non-existing-formatter", null); - } + () -> DecorsManager.decor("non-existing-formatter", null) ); - } } diff --git a/src/test/java/com/jcabi/log/DomDecorTest.java b/src/test/java/com/jcabi/log/DomDecorTest.java index 767b1a23..e2f41d9b 100644 --- a/src/test/java/com/jcabi/log/DomDecorTest.java +++ b/src/test/java/com/jcabi/log/DomDecorTest.java @@ -40,8 +40,7 @@ /** * Test case for {@link DomDecor}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ + * * @since 0.1 */ public final class DomDecorTest { diff --git a/src/test/java/com/jcabi/log/ExceptionDecorTest.java b/src/test/java/com/jcabi/log/ExceptionDecorTest.java index cc9f4541..56818d11 100644 --- a/src/test/java/com/jcabi/log/ExceptionDecorTest.java +++ b/src/test/java/com/jcabi/log/ExceptionDecorTest.java @@ -39,8 +39,6 @@ /** * Test case for {@link ExceptionDecor}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ public final class ExceptionDecorTest { diff --git a/src/test/java/com/jcabi/log/LineNumberTest.java b/src/test/java/com/jcabi/log/LineNumberTest.java index 601e30ed..4b4b118d 100644 --- a/src/test/java/com/jcabi/log/LineNumberTest.java +++ b/src/test/java/com/jcabi/log/LineNumberTest.java @@ -44,8 +44,6 @@ * Test case for %L pattern. * If you change this class, you have to care about line number * in "com.jcabi.log.LineNumberTest:72" - * @author Haris Peco (snpe60@gmail.com) - * @version $Id$ * @since 1.18 */ public final class LineNumberTest { @@ -71,7 +69,7 @@ public void testLineNumber() throws Exception { MatcherAssert.assertThat( writer.toString(), Matchers.containsString( - "com.jcabi.log.LineNumberTest:247" + "com.jcabi.log.LineNumberTest:245" ) ); } finally { diff --git a/src/test/java/com/jcabi/log/ListDecorTest.java b/src/test/java/com/jcabi/log/ListDecorTest.java index 287f0f8d..e4ce837b 100644 --- a/src/test/java/com/jcabi/log/ListDecorTest.java +++ b/src/test/java/com/jcabi/log/ListDecorTest.java @@ -40,9 +40,9 @@ /** * Test case for {@link ListDecor}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ + * * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ public final class ListDecorTest { @@ -72,6 +72,7 @@ public void testLogsRight(final Object list, final String text, * Params for this parametrized test. * @return Array of arrays of params for ctor */ + @SuppressWarnings("PMD.UnusedPrivateMethod") private static Collection params() { return Arrays.asList( new Object[][] { diff --git a/src/test/java/com/jcabi/log/LoggerTest.java b/src/test/java/com/jcabi/log/LoggerTest.java index ea1beff1..d777b17b 100644 --- a/src/test/java/com/jcabi/log/LoggerTest.java +++ b/src/test/java/com/jcabi/log/LoggerTest.java @@ -42,8 +42,6 @@ /** * Test case for {@link Logger}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ public final class LoggerTest { diff --git a/src/test/java/com/jcabi/log/MsDecorTest.java b/src/test/java/com/jcabi/log/MsDecorTest.java index 248a4fba..7ebcb256 100644 --- a/src/test/java/com/jcabi/log/MsDecorTest.java +++ b/src/test/java/com/jcabi/log/MsDecorTest.java @@ -43,6 +43,7 @@ * Test case for {@link MsDecor}. * * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ public final class MsDecorTest { @@ -80,6 +81,7 @@ public void testPrintsNullRight() { * Params for this parametrized test. * @return Array of arrays of params for ctor */ + @SuppressWarnings("PMD.UnusedPrivateMethod") private static Collection params() { return Arrays.asList( new Object[][] { @@ -89,10 +91,10 @@ private static Collection params() { {13L, "13.0ms", 0, 0, 1}, {1024L, "1s", 0, 0, 0}, {6001L, "6.0010s", 0, 0, 4}, - {122001L, " 2MIN", FormattableFlags.UPPERCASE, 6, 0}, - {3789003L, "1hr", 0, 0, 0}, - {86400000L, "1days", 0, 0, 0}, - {864000000L, "10days", 0, 0, 0}, + {122_001L, " 2MIN", FormattableFlags.UPPERCASE, 6, 0}, + {3_789_003L, "1hr", 0, 0, 0}, + {86_400_000L, "1days", 0, 0, 0}, + {864_000_000L, "10days", 0, 0, 0}, } ); } diff --git a/src/test/java/com/jcabi/log/NanoDecorTest.java b/src/test/java/com/jcabi/log/NanoDecorTest.java index ed25fef6..97269daf 100644 --- a/src/test/java/com/jcabi/log/NanoDecorTest.java +++ b/src/test/java/com/jcabi/log/NanoDecorTest.java @@ -41,10 +41,9 @@ /** * Test case for {@link NanoDecor}. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ + * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ public final class NanoDecorTest { @@ -82,6 +81,7 @@ public void testPrintsNullRight() { * Params for this parametrized test. * @return Array of arrays of params for ctor */ + @SuppressWarnings("PMD.UnusedPrivateMethod") private static Collection params() { return Arrays.asList( new Object[][] { @@ -94,17 +94,17 @@ private static Collection params() { {1024L, "1µs", 0, 0, 0}, {1056L, "1.056µs", 0, 0, 3}, {9022L, "9.02µs", 0, 0, 2}, - {53111L, "53.11µs ", FormattableFlags.LEFT_JUSTIFY, 10, 2}, - {53156L, " 53µs", 0, 7, 0}, - {87090432L, " 87ms", 0, 6, 0}, - {87090543L, "87.09ms", 0, 0, 2}, - {87090548L, "87.0905ms", 0, 0, 4}, - {6001001001L, "6.0010s", 0, 0, 4}, - {122001001001L, " 2MIN", FormattableFlags.UPPERCASE, 6, 0}, - {3789001001001L, "63.15002min", 0, 0, 5}, - {3789002002002L, "63.2min", 0, 0, 1}, - {3789003003003L, "63min", 0, 0, 0}, - {342000004004004L, "5700min", 0, 0, 0}, + {53_111L, "53.11µs ", FormattableFlags.LEFT_JUSTIFY, 10, 2}, + {53_156L, " 53µs", 0, 7, 0}, + {87_090_432L, " 87ms", 0, 6, 0}, + {87_090_543L, "87.09ms", 0, 0, 2}, + {87_090_548L, "87.0905ms", 0, 0, 4}, + {6_001_001_001L, "6.0010s", 0, 0, 4}, + {122_001_001_001L, " 2MIN", FormattableFlags.UPPERCASE, 6, 0}, + {3_789_001_001_001L, "63.15002min", 0, 0, 5}, + {3_789_002_002_002L, "63.2min", 0, 0, 1}, + {3_789_003_003_003L, "63min", 0, 0, 0}, + {342_000_004_004_004L, "5700min", 0, 0, 0}, } ); } diff --git a/src/test/java/com/jcabi/log/ObjectDecorProbe.java b/src/test/java/com/jcabi/log/ObjectDecorTest.java similarity index 96% rename from src/test/java/com/jcabi/log/ObjectDecorProbe.java rename to src/test/java/com/jcabi/log/ObjectDecorTest.java index 28fa9ee0..f5f98d75 100644 --- a/src/test/java/com/jcabi/log/ObjectDecorProbe.java +++ b/src/test/java/com/jcabi/log/ObjectDecorTest.java @@ -41,8 +41,9 @@ * Test case for {@link ObjectDecor}. * * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ -public final class ObjectDecorProbe { +public final class ObjectDecorTest { @ParameterizedTest @MethodSource("params") @@ -70,6 +71,7 @@ public void testLogsRight(final Object obj, final String text, * Params for this parametrized test. * @return Array of arrays of params for ctor */ + @SuppressWarnings("PMD.UnusedPrivateMethod") private static Collection params() { return Arrays.asList( new Object[][] { @@ -92,6 +94,8 @@ private static Collection params() { /** * Test class for displaying object contents. + * + * @since 0.1 */ private static final class Foo { /** @@ -99,11 +103,13 @@ private static final class Foo { */ @SuppressWarnings("unused") private final transient int num; + /** * The name. */ @SuppressWarnings("unused") private final transient String name; + /** * Ctor. * @param number The number diff --git a/src/test/java/com/jcabi/log/ParseableInformationTest.java b/src/test/java/com/jcabi/log/ParseableInformationTest.java index c47a38ed..20c5f421 100644 --- a/src/test/java/com/jcabi/log/ParseableInformationTest.java +++ b/src/test/java/com/jcabi/log/ParseableInformationTest.java @@ -38,8 +38,6 @@ /** * ParseableInformation test case. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ public class ParseableInformationTest { diff --git a/src/test/java/com/jcabi/log/ParseableLevelInformationTest.java b/src/test/java/com/jcabi/log/ParseableLevelInformationTest.java index 28357fec..198ff875 100644 --- a/src/test/java/com/jcabi/log/ParseableLevelInformationTest.java +++ b/src/test/java/com/jcabi/log/ParseableLevelInformationTest.java @@ -38,8 +38,6 @@ /** * ParseableLevelInformation test case. - * @author Jose V. Dal Pra Junior (jrdalpra@gmail.com) - * @version $Id$ * @since 0.18 */ public class ParseableLevelInformationTest { diff --git a/src/test/java/com/jcabi/log/PreFormatterTest.java b/src/test/java/com/jcabi/log/PreFormatterTest.java index 63f11011..967eafed 100644 --- a/src/test/java/com/jcabi/log/PreFormatterTest.java +++ b/src/test/java/com/jcabi/log/PreFormatterTest.java @@ -35,9 +35,6 @@ /** * Test case for {@link PreFormatter}. - * @author Marina Kosenko (marina.kosenko@gmail.com) - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ public final class PreFormatterTest { diff --git a/src/test/java/com/jcabi/log/SecretDecorTest.java b/src/test/java/com/jcabi/log/SecretDecorTest.java index e4da52c8..f0e9ef7f 100644 --- a/src/test/java/com/jcabi/log/SecretDecorTest.java +++ b/src/test/java/com/jcabi/log/SecretDecorTest.java @@ -42,13 +42,14 @@ * Test case for {@link SecretDecor}. * * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ public final class SecretDecorTest { @ParameterizedTest @MethodSource("params") public void testPrintsRight(final Object list, final String text, - final int flags, final int width, final int precision) throws DecorException { + final int flags, final int width, final int precision) { Locale.setDefault(Locale.US); MatcherAssert.assertThat( new Printed(new SecretDecor(list), flags, width, precision), @@ -59,7 +60,7 @@ public void testPrintsRight(final Object list, final String text, @ParameterizedTest @MethodSource("params") public void testLogsRight(final Object list, final String text, - final int flags, final int width, final int precision) throws DecorException { + final int flags, final int width, final int precision) { Locale.setDefault(Locale.US); MatcherAssert.assertThat( new Logged(new SecretDecor(list), flags, width, precision), @@ -71,6 +72,7 @@ public void testLogsRight(final Object list, final String text, * Params for this parametrized test. * @return Array of arrays of params for ctor */ + @SuppressWarnings("PMD.UnusedPrivateMethod") private static Collection params() { return Arrays.asList( new Object[][] { diff --git a/src/test/java/com/jcabi/log/SizeDecorTest.java b/src/test/java/com/jcabi/log/SizeDecorTest.java index f87ecc16..94a2cede 100644 --- a/src/test/java/com/jcabi/log/SizeDecorTest.java +++ b/src/test/java/com/jcabi/log/SizeDecorTest.java @@ -43,6 +43,7 @@ * Test case for {@link SizeDecor}. * * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ public final class SizeDecorTest { @@ -80,6 +81,7 @@ public void testPrintsNullRight() { * Params for this parametrized test. * @return Array of arrays of params for ctor */ + @SuppressWarnings("PMD.UnusedPrivateMethod") private static Collection params() { return Arrays.asList( new Object[][] { @@ -88,10 +90,10 @@ private static Collection params() { {123L, " 123b", 0, 6, 0}, {1024L, "1.000Kb", 0, 0, 3}, {5120L, "5Kb", 0, 0, 0}, - {12345L, "12.056Kb", 0, 0, 3}, - {12345L, "12.1Kb ", FormattableFlags.LEFT_JUSTIFY, 8, 1}, - {98765432L, "94.190MB", FormattableFlags.UPPERCASE, 0, 3}, - {98765432L, "94.190Mb", 0, 0, 3}, + {12_345L, "12.056Kb", 0, 0, 3}, + {12_345L, "12.1Kb ", FormattableFlags.LEFT_JUSTIFY, 8, 1}, + {98_765_432L, "94.190MB", FormattableFlags.UPPERCASE, 0, 3}, + {98_765_432L, "94.190Mb", 0, 0, 3}, {90L * 1024 * 1024 * 1024, "90Gb", 0, 0, 0}, {13L * 1024 * 1024 * 1024 * 1024, "13Tb", 0, 0, 0}, {33L * 1024 * 1024 * 1024 * 1024 * 1024, "33Pb", 0, 0, 0}, diff --git a/src/test/java/com/jcabi/log/SupplierLoggerTest.java b/src/test/java/com/jcabi/log/SupplierLoggerTest.java index 3be22e72..c8bf708a 100644 --- a/src/test/java/com/jcabi/log/SupplierLoggerTest.java +++ b/src/test/java/com/jcabi/log/SupplierLoggerTest.java @@ -37,8 +37,6 @@ /** * Test case for {@link SupplierLogger}. - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ * @since 0.18 * @todo #100:30min Some tests here are ignored since they conflict * in multi-threading run. I don't know exactly how to fix them, diff --git a/src/test/java/com/jcabi/log/TextDecorTest.java b/src/test/java/com/jcabi/log/TextDecorTest.java index 132ed4f1..64ae74f5 100644 --- a/src/test/java/com/jcabi/log/TextDecorTest.java +++ b/src/test/java/com/jcabi/log/TextDecorTest.java @@ -45,6 +45,7 @@ * Test case for {@link TextDecor}. * * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ public final class TextDecorTest { @@ -70,23 +71,6 @@ public void testLogsRight(final String obj, final String text, ); } - /** - * Params for this parametrized test. - * @return Array of arrays of params for ctor - */ - @SuppressWarnings("PMD.AvoidDuplicateLiterals") - private static Collection params() { - return Arrays.asList( - new Object[][] { - // @checkstyle MultipleStringLiterals (1 line) - {"simple text", "simple text", 0, 0, 0}, - {null, "NULL", 0, 0, 0}, - // @checkstyle MultipleStringLiteralsCheck (1 line) - {"\u0433!", "\u0433!", 0, 0, 0}, - } - ); - } - /** * Test for a long text. */ @@ -105,4 +89,26 @@ public void compressesLongText() { ) ); } + + /** + * Params for this parametrized test. + * @return Array of arrays of params for ctor + */ + @SuppressWarnings( + { + "PMD.AvoidDuplicateLiterals", + "PMD.UnusedPrivateMethod" + } + ) + private static Collection params() { + return Arrays.asList( + new Object[][] { + // @checkstyle MultipleStringLiterals (1 line) + {"simple text", "simple text", 0, 0, 0}, + {null, "NULL", 0, 0, 0}, + // @checkstyle MultipleStringLiteralsCheck (1 line) + {"\u0433!", "\u0433!", 0, 0, 0}, + } + ); + } } diff --git a/src/test/java/com/jcabi/log/TypeDecorTest.java b/src/test/java/com/jcabi/log/TypeDecorTest.java index 36526b65..d08ac08b 100644 --- a/src/test/java/com/jcabi/log/TypeDecorTest.java +++ b/src/test/java/com/jcabi/log/TypeDecorTest.java @@ -41,6 +41,7 @@ * Test case for {@link TypeDecor}. * * @since 0.1 + * @checkstyle ParameterNumberCheck (500 lines) */ public final class TypeDecorTest { @@ -70,6 +71,7 @@ public void testLogsRight(final Object list, final String text, * Params for this parametrized test. * @return Array of arrays of params for ctor */ + @SuppressWarnings("PMD.UnusedPrivateMethod") private static Collection params() { return Arrays.asList( new Object[][] { diff --git a/src/test/java/com/jcabi/log/UnitTestAppender.java b/src/test/java/com/jcabi/log/UnitTestAppender.java index 539f5218..3a77f3f4 100644 --- a/src/test/java/com/jcabi/log/UnitTestAppender.java +++ b/src/test/java/com/jcabi/log/UnitTestAppender.java @@ -39,8 +39,6 @@ * Log4j appender for unit tests. Normally, we could use * slf4j-test, but we * have log4j in the classpath anyway, for {@link MulticolorLayout}. - * @author Mihai Andronache (amihaiemil@gmail.com) - * @version $Id$ * @since 0.18 */ public final class UnitTestAppender extends WriterAppender { diff --git a/src/test/java/com/jcabi/log/VerboseCallableTest.java b/src/test/java/com/jcabi/log/VerboseCallableTest.java index 0ce81564..8c773b58 100644 --- a/src/test/java/com/jcabi/log/VerboseCallableTest.java +++ b/src/test/java/com/jcabi/log/VerboseCallableTest.java @@ -34,8 +34,6 @@ /** * Test case for {@link VerboseCallable}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.16 */ @SuppressWarnings({ "PMD.DoNotUseThreads", "PMD.TooManyMethods" }) diff --git a/src/test/java/com/jcabi/log/VerboseProcessTest.java b/src/test/java/com/jcabi/log/VerboseProcessTest.java index 69a89a34..a1c196ef 100644 --- a/src/test/java/com/jcabi/log/VerboseProcessTest.java +++ b/src/test/java/com/jcabi/log/VerboseProcessTest.java @@ -30,6 +30,7 @@ package com.jcabi.log; import java.io.ByteArrayInputStream; +import java.io.Closeable; import java.io.File; import java.io.FileInputStream; import java.io.IOException; @@ -56,8 +57,7 @@ /** * Test case for {@link VerboseProcess}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ + * * @checkstyle MultipleStringLiterals (500 lines) * @checkstyle ClassDataAbstractionCoupling (200 lines) * @todo #18 Locale/encoding problem in two test methods here. I'm not @@ -247,13 +247,14 @@ public void stdoutQuietlyLogsErrors() { } @Test + @SuppressWarnings("PMD.AvoidFileStream") public void logsErrorWhenUnderlyingStreamIsClosed() throws Exception { final StringWriter writer = new StringWriter(); org.apache.log4j.Logger.getRootLogger().addAppender( new WriterAppender(new SimpleLayout(), writer) ); final Process prc = Mockito.mock(Process.class); - final InputStream stdout = new FileInputStream( + final Closeable stdout = new FileInputStream( File.createTempFile("temp", "test") ); stdout.close(); @@ -368,6 +369,8 @@ public void run() { /** * {@link InputStream} returning endless flow of characters. + * + * @since 0.1 */ private final class InfiniteInputStream extends InputStream { /** @@ -379,10 +382,12 @@ private final class InfiniteInputStream extends InputStream { * Character, endlessly repeated in the stream. */ private final transient char chr; + /** * Whether the next char in the stream should be EOL. */ private transient boolean feed; + /** * Whether this stream is closed. */ @@ -425,6 +430,8 @@ public void close() { * *

It filters out messages of monitor threads, that doesn't belong to * specific {@link VerboseProcess}. + * + * @since 0.1 */ private final class VrbPrcMonitorFilter extends Filter { /** diff --git a/src/test/java/com/jcabi/log/VerboseRunnableTest.java b/src/test/java/com/jcabi/log/VerboseRunnableTest.java index 154dd25c..bb417d23 100644 --- a/src/test/java/com/jcabi/log/VerboseRunnableTest.java +++ b/src/test/java/com/jcabi/log/VerboseRunnableTest.java @@ -42,8 +42,6 @@ /** * Test case for {@link VerboseRunnable}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @SuppressWarnings({ "PMD.DoNotUseThreads", "PMD.TooManyMethods" }) @@ -59,7 +57,6 @@ public void logsExceptionsInRunnable() { } ).run() ); - } @Test @@ -91,6 +88,7 @@ public void translatesToStringFromUnderlyingRunnable() throws Exception { public void run() { assert true; } + @Override public String toString() { return text; @@ -112,6 +110,7 @@ public void translatesToStringFromUnderlyingCallable() throws Exception { public Void call() { return null; } + @Override public String toString() { return text; diff --git a/src/test/java/com/jcabi/log/VerboseThreadsTest.java b/src/test/java/com/jcabi/log/VerboseThreadsTest.java index 6d135ed6..d815e576 100644 --- a/src/test/java/com/jcabi/log/VerboseThreadsTest.java +++ b/src/test/java/com/jcabi/log/VerboseThreadsTest.java @@ -37,8 +37,6 @@ /** * Test case for {@link VerboseThreads}. - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ * @since 0.1 */ @SuppressWarnings("PMD.DoNotUseThreads") diff --git a/src/test/java/com/jcabi/log/package-info.java b/src/test/java/com/jcabi/log/package-info.java index 6e324d02..e30c88c8 100644 --- a/src/test/java/com/jcabi/log/package-info.java +++ b/src/test/java/com/jcabi/log/package-info.java @@ -30,8 +30,5 @@ /** * Convenient utils, tests. - * - * @author Yegor Bugayenko (yegor256@gmail.com) - * @version $Id$ */ package com.jcabi.log;