diff --git a/eo-runtime/pom.xml b/eo-runtime/pom.xml index aa6744a1e3..19d5461f5f 100644 --- a/eo-runtime/pom.xml +++ b/eo-runtime/pom.xml @@ -336,7 +336,14 @@ SOFTWARE. qulice-maven-plugin - pmd:.* + + pmd:/src/main/java/EOorg/.* + pmd:/src/test/java/EOorg/.* dependencies:org.eolang:eo-maven-plugin diff --git a/eo-runtime/src/main/java/org/eolang/Bytes.java b/eo-runtime/src/main/java/org/eolang/Bytes.java index c3cdc12869..ea6cbaa511 100644 --- a/eo-runtime/src/main/java/org/eolang/Bytes.java +++ b/eo-runtime/src/main/java/org/eolang/Bytes.java @@ -51,8 +51,9 @@ public interface Bytes { * OR operation. * @param other Bytes. * @return Bytes. - * @checkstyle MethodNameCheck (2 lines) + * @checkstyle MethodNameCheck (3 lines) */ + @SuppressWarnings("PMD.ShortMethodName") Bytes or(Bytes other); /** diff --git a/eo-runtime/src/main/java/org/eolang/BytesOf.java b/eo-runtime/src/main/java/org/eolang/BytesOf.java index cefb3a5822..af6260bd6d 100644 --- a/eo-runtime/src/main/java/org/eolang/BytesOf.java +++ b/eo-runtime/src/main/java/org/eolang/BytesOf.java @@ -31,8 +31,14 @@ * Bytes. * * @since 1.0 + * @todo #3160:90min This class requires refactoring. As a result of refactoring you should remove + * {@code @SuppressWarnings("PMD.GodClass")} from this class and + * {@code @SuppressWarnings("PMD.CognitiveComplexity")} from {@link BytesOf#shift} method. You can + * check description of this rules here + * pmd.github.io */ @Versionized +@SuppressWarnings({"PMD.TooManyMethods", "PMD.GodClass"}) public final class BytesOf implements Bytes { /** @@ -45,7 +51,7 @@ public final class BytesOf implements Bytes { * @param data Data. */ public BytesOf(final byte[] data) { - this.data = data; + this.data = Arrays.copyOf(data, data.length); } /** @@ -108,6 +114,7 @@ public Bytes and(final Bytes other) { } @Override + @SuppressWarnings("PMD.ShortMethodName") public Bytes or(final Bytes other) { final byte[] first = this.take(); final byte[] second = other.take(); @@ -128,6 +135,7 @@ public Bytes xor(final Bytes other) { } @Override + @SuppressWarnings("PMD.CognitiveComplexity") public Bytes shift(final int bits) { // @checkstyle MethodBodyCommentsCheck (3 lines) // @checkstyle NestedIfDepthCheck (40 lines) diff --git a/eo-runtime/src/main/java/org/eolang/Data.java b/eo-runtime/src/main/java/org/eolang/Data.java index e295e00896..877abcb13b 100644 --- a/eo-runtime/src/main/java/org/eolang/Data.java +++ b/eo-runtime/src/main/java/org/eolang/Data.java @@ -32,6 +32,7 @@ * @since 0.1 */ @Versionized +@SuppressWarnings("PMD.TooManyMethods") public interface Data { /** * Attach data to the object. @@ -192,11 +193,20 @@ private static Phi toPhi(final Object obj) { * * @param str A string optionally containing standard java escape sequences. * @return The translated string + * @todo #3160:90min This method should be refactored because it has high cognitive + * complexity and other problems. All {@code @checkstyle} warnings suppression and + * {@code SuppressWarnings("PMD.WarningName")} annotations for this method should be + * removed as a result of refactoring. * @checkstyle CyclomaticComplexityCheck (100 lines) * @checkstyle JavaNCSSCheck (100 lines) * @checkstyle NestedIfDepthCheck (100 lines) * @checkstyle ModifiedControlVariableCheck (100 lines) */ + @SuppressWarnings({ + "PMD.AvoidReassigningLoopVariables", + "PMD.CognitiveComplexity", + "PMD.NPathComplexity" + }) private static String unescapeJavaString(final String str) { final StringBuilder unescaped = new StringBuilder(str.length()); for (int idx = 0; idx < str.length(); ++idx) { @@ -209,19 +219,19 @@ private static String unescapeJavaString(final String str) { next = str.charAt(idx + 1); } if (next >= '0' && next <= '7') { - String code = String.valueOf(next); + final StringBuilder code = new StringBuilder(String.valueOf(next)); ++idx; if (idx < str.length() - 1 && str.charAt(idx + 1) >= '0' && str.charAt(idx + 1) <= '7') { - code += str.charAt(idx + 1); + code.append(str.charAt(idx + 1)); ++idx; if (idx < str.length() - 1 && str.charAt(idx + 1) >= '0' && str.charAt(idx + 1) <= '7') { - code += str.charAt(idx + 1); + code.append(str.charAt(idx + 1)); ++idx; } } - unescaped.append((char) Integer.parseInt(code, 8)); + unescaped.append((char) Integer.parseInt(code.toString(), 8)); continue; } switch (next) { diff --git a/eo-runtime/src/main/java/org/eolang/Dataized.java b/eo-runtime/src/main/java/org/eolang/Dataized.java index 75c1e6b479..d63d11dce0 100644 --- a/eo-runtime/src/main/java/org/eolang/Dataized.java +++ b/eo-runtime/src/main/java/org/eolang/Dataized.java @@ -150,14 +150,4 @@ public T take(final Class type) { } return type.cast(strong); } - - /** - * Clean up resources. - * This includes call of {@link ThreadLocal#remove()} method to prevent - * memory leaks. - */ - public static void cleanUp() { - Dataized.LEVEL.remove(); - Dataized.MAX_LEVEL.remove(); - } } diff --git a/eo-runtime/src/main/java/org/eolang/ExAbstract.java b/eo-runtime/src/main/java/org/eolang/ExAbstract.java index 2229c390fa..f92e0d971c 100644 --- a/eo-runtime/src/main/java/org/eolang/ExAbstract.java +++ b/eo-runtime/src/main/java/org/eolang/ExAbstract.java @@ -42,25 +42,26 @@ public abstract class ExAbstract extends RuntimeException { /** * Ctor. + * @param cause Exception cause */ - public ExAbstract() { - this(null); + public ExAbstract(final String cause) { + super(cause); } /** * Ctor. * @param cause Exception cause + * @param root Root cause exception */ - public ExAbstract(final String cause) { - super(cause); + public ExAbstract(final String cause, final Throwable root) { + super(cause, root); } /** * Ctor. - * @param cause Exception cause * @param root Root cause exception */ - ExAbstract(final String cause, final Throwable root) { - super(cause, root); + public ExAbstract(final Throwable root) { + super(root); } } diff --git a/eo-runtime/src/main/java/org/eolang/ExInterrupted.java b/eo-runtime/src/main/java/org/eolang/ExInterrupted.java index cc5da73fbc..50e3ad9675 100644 --- a/eo-runtime/src/main/java/org/eolang/ExInterrupted.java +++ b/eo-runtime/src/main/java/org/eolang/ExInterrupted.java @@ -38,8 +38,9 @@ public class ExInterrupted extends ExAbstract { /** * Ctor. + * @param root Root cause exception */ - public ExInterrupted() { - super(null); + public ExInterrupted(final InterruptedException root) { + super(root); } } diff --git a/eo-runtime/src/main/java/org/eolang/Main.java b/eo-runtime/src/main/java/org/eolang/Main.java index 27c36f223d..5cfad3dce5 100644 --- a/eo-runtime/src/main/java/org/eolang/Main.java +++ b/eo-runtime/src/main/java/org/eolang/Main.java @@ -49,6 +49,7 @@ * @since 0.1 */ @Versionized +@SuppressWarnings("PMD.MoreThanOneLogger") public final class Main { /** @@ -182,7 +183,8 @@ private static void run(final List opts) throws Exception { .newInstance(); } catch (final ClassNotFoundException ex) { throw new ExUnset( - String.format("Can not find '%s' object", opts.get(0)) + String.format("Can not find '%s' object", opts.get(0)), + ex ); } if (opts.size() > 1) { diff --git a/eo-runtime/src/main/java/org/eolang/PhDefault.java b/eo-runtime/src/main/java/org/eolang/PhDefault.java index d356ff0020..3c1ed00624 100644 --- a/eo-runtime/src/main/java/org/eolang/PhDefault.java +++ b/eo-runtime/src/main/java/org/eolang/PhDefault.java @@ -42,7 +42,7 @@ * @checkstyle DesignForExtensionCheck (500 lines) */ @Versionized -@SuppressWarnings({"PMD.TooManyMethods", "PMD.ConstructorShouldDoInitialization"}) +@SuppressWarnings({"PMD.TooManyMethods", "PMD.GodClass"}) public class PhDefault implements Phi, Cloneable { /** * Vertices. @@ -74,7 +74,7 @@ public class PhDefault implements Phi, Cloneable { * Data. * @checkstyle VisibilityModifierCheck (2 lines) */ - private AtomicReference data = new AtomicReference<>(null); + private AtomicReference data; /** * Forma of it. @@ -96,6 +96,7 @@ public class PhDefault implements Phi, Cloneable { */ @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") public PhDefault() { + this.data = new AtomicReference<>(null); this.vertex = PhDefault.VTX.next(); this.form = this.getClass().getName(); this.attrs = new HashMap<>(0); @@ -300,15 +301,6 @@ public String forma() { return this.form; } - /** - * Clean up resources. - * This includes call of {@link ThreadLocal#remove()} method to prevent - * memory leaks. - */ - public static void cleanUp() { - PhDefault.NESTING.remove(); - } - /** * Add new attribute. * diff --git a/eo-runtime/src/main/java/org/eolang/PhFake.java b/eo-runtime/src/main/java/org/eolang/PhFake.java index d723824b5f..959b3dca40 100644 --- a/eo-runtime/src/main/java/org/eolang/PhFake.java +++ b/eo-runtime/src/main/java/org/eolang/PhFake.java @@ -47,6 +47,7 @@ public PhFake() { * Ctor. * @param sup The function to return the real object */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") public PhFake(final Supplier sup) { this.add("args", new AtVoid("args")); this.add("φ", new AtComposite(this, rho -> sup.get())); diff --git a/eo-runtime/src/main/java/org/eolang/PhLocated.java b/eo-runtime/src/main/java/org/eolang/PhLocated.java index 33458ae922..ded8ce8da7 100644 --- a/eo-runtime/src/main/java/org/eolang/PhLocated.java +++ b/eo-runtime/src/main/java/org/eolang/PhLocated.java @@ -29,6 +29,7 @@ * * @since 0.21 */ +@SuppressWarnings("PMD.TooManyMethods") @Versionized public final class PhLocated implements Phi { diff --git a/eo-runtime/src/main/java/org/eolang/PhLogged.java b/eo-runtime/src/main/java/org/eolang/PhLogged.java index 27356f0f29..5692a1d1b4 100644 --- a/eo-runtime/src/main/java/org/eolang/PhLogged.java +++ b/eo-runtime/src/main/java/org/eolang/PhLogged.java @@ -34,6 +34,7 @@ * @since 0.24 */ @Versionized +@SuppressWarnings({"PMD.TooManyMethods", "PMD.SystemPrintln"}) public final class PhLogged implements Phi { /** diff --git a/eo-runtime/src/main/java/org/eolang/PhNamed.java b/eo-runtime/src/main/java/org/eolang/PhNamed.java index d5b0157b76..5de26bb4f7 100644 --- a/eo-runtime/src/main/java/org/eolang/PhNamed.java +++ b/eo-runtime/src/main/java/org/eolang/PhNamed.java @@ -30,6 +30,7 @@ * @since 0.17 */ @Versionized +@SuppressWarnings("PMD.TooManyMethods") final class PhNamed implements Phi { /** diff --git a/eo-runtime/src/main/java/org/eolang/PhOnce.java b/eo-runtime/src/main/java/org/eolang/PhOnce.java index 466d103e02..a5ce5f6214 100644 --- a/eo-runtime/src/main/java/org/eolang/PhOnce.java +++ b/eo-runtime/src/main/java/org/eolang/PhOnce.java @@ -33,6 +33,7 @@ * @since 0.1 */ @Versionized +@SuppressWarnings("PMD.TooManyMethods") class PhOnce implements Phi { /** diff --git a/eo-runtime/src/main/java/org/eolang/PhPackage.java b/eo-runtime/src/main/java/org/eolang/PhPackage.java index 6177f82c07..7badbf4942 100644 --- a/eo-runtime/src/main/java/org/eolang/PhPackage.java +++ b/eo-runtime/src/main/java/org/eolang/PhPackage.java @@ -35,6 +35,7 @@ * @since 0.22 */ @Versionized +@SuppressWarnings("PMD.TooManyMethods") final class PhPackage implements Phi { /** @@ -45,9 +46,7 @@ final class PhPackage implements Phi { /** * All of them. */ - private final ThreadLocal> objects = ThreadLocal.withInitial( - () -> new ConcurrentHashMap<>(0) - ); + private final ThreadLocal> objects; /** * Ctor. @@ -55,6 +54,9 @@ final class PhPackage implements Phi { */ PhPackage(final String name) { this.pkg = name; + this.objects = ThreadLocal.withInitial( + () -> new ConcurrentHashMap<>(0) + ); } @Override diff --git a/eo-runtime/src/main/java/org/eolang/PhSafe.java b/eo-runtime/src/main/java/org/eolang/PhSafe.java index 3d3b716e5b..38e7dfafce 100644 --- a/eo-runtime/src/main/java/org/eolang/PhSafe.java +++ b/eo-runtime/src/main/java/org/eolang/PhSafe.java @@ -33,6 +33,7 @@ * @since 0.26 */ @Versionized +@SuppressWarnings("PMD.TooManyMethods") public final class PhSafe implements Phi { /** diff --git a/eo-runtime/src/main/java/org/eolang/PhTraced.java b/eo-runtime/src/main/java/org/eolang/PhTraced.java index e8d59f70af..5f906510d3 100644 --- a/eo-runtime/src/main/java/org/eolang/PhTraced.java +++ b/eo-runtime/src/main/java/org/eolang/PhTraced.java @@ -35,13 +35,13 @@ * @since 0.36 */ @Versionized +@SuppressWarnings("PMD.TooManyMethods") public final class PhTraced implements Phi { /** * Name of property that responsible for keeping max depth. */ - public static final String - MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME = "EO_MAX_CAGE_RECURSION_DEPTH"; + public static final String RECURSION_LIMIT = "EO_MAX_CAGE_RECURSION_DEPTH"; /** * Cages that are currently being dataized. If one cage is being datazed, and @@ -58,7 +58,7 @@ public final class PhTraced implements Phi { /** * Locator of encaged object. */ - private final Integer locator; + private final Integer locatr; /** * Max depth of cage recursion. @@ -75,7 +75,7 @@ public PhTraced(final Phi object, final Integer locator) { object, locator, Integer.parseInt( - System.getProperty(PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME, "100") + System.getProperty(PhTraced.RECURSION_LIMIT, "100") ) ); } @@ -88,13 +88,13 @@ public PhTraced(final Phi object, final Integer locator) { */ public PhTraced(final Phi object, final Integer locator, final int depth) { this.object = object; - this.locator = locator; + this.locatr = locator; this.depth = depth; } @Override public Phi copy() { - return new PhTraced(this.object.copy(), this.locator); + return new PhTraced(this.object.copy(), this.locatr); } @Override @@ -188,13 +188,13 @@ public T get() { */ private Integer incrementCageCounter() { return PhTraced.DATAIZING_CAGES.get().compute( - PhTraced.this.locator, (key, counter) -> { + PhTraced.this.locatr, (key, counter) -> { final int ret = this.incremented(counter); if (ret > PhTraced.this.depth) { throw new ExFailure( "The cage %s with locator %d has reached the maximum nesting depth = %d", PhTraced.this.object, - PhTraced.this.locator, + PhTraced.this.locatr, PhTraced.this.depth ); } @@ -229,11 +229,11 @@ private void decrementCageCounter(final int incremented) { final int decremented = incremented - 1; if (decremented == 0) { PhTraced.DATAIZING_CAGES.get().remove( - PhTraced.this.locator + PhTraced.this.locatr ); } else { PhTraced.DATAIZING_CAGES.get().put( - PhTraced.this.locator, decremented + PhTraced.this.locatr, decremented ); } } diff --git a/eo-runtime/src/main/java/org/eolang/PhWrite.java b/eo-runtime/src/main/java/org/eolang/PhWrite.java deleted file mode 100644 index 7ce0cf81c6..0000000000 --- a/eo-runtime/src/main/java/org/eolang/PhWrite.java +++ /dev/null @@ -1,68 +0,0 @@ -/* - * The MIT License (MIT) - * - * Copyright (c) 2016-2024 Objectionary.com - * - * Permission is hereby granted, free of charge, to any person obtaining a copy - * of this software and associated documentation files (the "Software"), to deal - * in the Software without restriction, including without limitation the rights - * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell - * copies of the Software, and to permit persons to whom the Software is - * furnished to do so, subject to the following conditions: - * - * The above copyright notice and this permission notice shall be included - * in all copies or substantial portions of the Software. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR - * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, - * FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL THE - * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER - * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, - * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE - * SOFTWARE. - */ - -/* - * @checkstyle PackageNameCheck (4 lines) - */ - -package org.eolang; - -import java.util.function.Function; - -/** - * Object that writes other object to own \rho. - * @since 0.36.0 - */ -public final class PhWrite extends PhDefault implements Atom { - /** - * Attribute name. - */ - private final String attribute; - - /** - * Return value. - */ - private final Function value; - - /** - * Ctor. - * @param attr Attribute name - * @param ret Return value function - */ - public PhWrite( - final String attr, - final Function ret - ) { - this.attribute = attr; - this.add(this.attribute, new AtVoid(this.attribute)); - this.value = ret; - } - - @Override - public Phi lambda() { - final Phi rho = this.take(Attr.RHO); - rho.put(this.attribute, this.take(this.attribute)); - return this.value.apply(rho); - } -} diff --git a/eo-runtime/src/main/java/org/eolang/Phi.java b/eo-runtime/src/main/java/org/eolang/Phi.java index 3f4df95cd5..d1f676fde9 100644 --- a/eo-runtime/src/main/java/org/eolang/Phi.java +++ b/eo-runtime/src/main/java/org/eolang/Phi.java @@ -44,6 +44,7 @@ public interface Phi extends Term, Data { * @checkstyle ConstantNameCheck (5 lines) * @checkstyle AnonInnerLengthCheck (30 lines) */ + @SuppressWarnings("PMD.FieldNamingConventions") Phi Φ = new Phi() { /** * Default package. diff --git a/eo-runtime/src/main/java/org/eolang/SafeFunc.java b/eo-runtime/src/main/java/org/eolang/SafeFunc.java index ca0ebc9f7e..ff8feb14a7 100644 --- a/eo-runtime/src/main/java/org/eolang/SafeFunc.java +++ b/eo-runtime/src/main/java/org/eolang/SafeFunc.java @@ -47,18 +47,17 @@ class SafeFunc implements Supplier { } @Override + @SuppressWarnings({"PMD.AvoidCatchingGenericException", "PMD.AvoidRethrowingException"}) public T get() { try { return this.origin.call(); } catch (final InterruptedException ex) { Thread.currentThread().interrupt(); - throw new ExInterrupted(); - } catch (final ExAbstract ex) { - throw ex; + throw new ExInterrupted(ex); // @checkstyle IllegalCatchCheck (3 line) } catch (final RuntimeException ex) { throw ex; - } catch (final Throwable ex) { + } catch (final Exception ex) { throw new ExFailure( String.format( "Unexpected error '%s' of type %s", diff --git a/eo-runtime/src/main/java/org/eolang/Term.java b/eo-runtime/src/main/java/org/eolang/Term.java index d7273d89d9..03710277ac 100644 --- a/eo-runtime/src/main/java/org/eolang/Term.java +++ b/eo-runtime/src/main/java/org/eolang/Term.java @@ -42,6 +42,7 @@ public interface Term { * @return The expression in φ-calculus * @checkstyle MethodNameCheck (5 lines) */ + @SuppressWarnings("PMD.MethodNamingConventions") String φTerm(); } diff --git a/eo-runtime/src/main/java/org/eolang/UniverseSafe.java b/eo-runtime/src/main/java/org/eolang/UniverseSafe.java index aeaf9002c0..1c5bfaed18 100644 --- a/eo-runtime/src/main/java/org/eolang/UniverseSafe.java +++ b/eo-runtime/src/main/java/org/eolang/UniverseSafe.java @@ -40,6 +40,7 @@ * @checkstyle IllegalCatchCheck (200 lines) */ @Versionized +@SuppressWarnings("PMD.AvoidCatchingThrowable") public final class UniverseSafe implements Universe { /** diff --git a/eo-runtime/src/main/java/org/eolang/VerboseBytesAsString.java b/eo-runtime/src/main/java/org/eolang/VerboseBytesAsString.java index e4c4a3d66b..c7cb05ebeb 100644 --- a/eo-runtime/src/main/java/org/eolang/VerboseBytesAsString.java +++ b/eo-runtime/src/main/java/org/eolang/VerboseBytesAsString.java @@ -45,7 +45,7 @@ public final class VerboseBytesAsString implements Supplier { * @param data Data. */ public VerboseBytesAsString(final byte[] data) { - this.data = data; + this.data = Arrays.copyOf(data, data.length); } @Override @@ -81,6 +81,7 @@ public String get() { Arrays.toString(this.data), new String(this.data, StandardCharsets.UTF_8) ); + break; } return result; } diff --git a/eo-runtime/src/test/java/EOorg/EOeolang/EOcageTest.java b/eo-runtime/src/test/java/EOorg/EOeolang/EOcageTest.java index fb9684ceb8..e8d8ac4ae6 100644 --- a/eo-runtime/src/test/java/EOorg/EOeolang/EOcageTest.java +++ b/eo-runtime/src/test/java/EOorg/EOeolang/EOcageTest.java @@ -212,13 +212,13 @@ class RecursionTests { @BeforeEach void setDepth() { System.setProperty( - PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME, String.valueOf(MAX_DEPTH) + PhTraced.RECURSION_LIMIT, String.valueOf(MAX_DEPTH) ); } @AfterEach void clearDepth() { - System.clearProperty(PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME); + System.clearProperty(PhTraced.RECURSION_LIMIT); } @Test @@ -257,7 +257,7 @@ void doesNotThrowIfDataizesConcurrently() { @Test void rewritesItselfToItselfViaDummy() { System.setProperty( - PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME, "2" + PhTraced.RECURSION_LIMIT, "2" ); final Phi cage = EOcageTest.encaged( new PhWith(new EOcageTest.Dummy(), 0, new Data.ToPhi(1L)) @@ -295,8 +295,8 @@ void doesNotThrowExceptionIfSmallDepth() { () -> new Dataized(cage).take(), String.format( "We expect that dataizing of nested cage which recursion depth is less than property %s = %s does not throw %s", - PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME, - System.getProperty(PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME), + PhTraced.RECURSION_LIMIT, + System.getProperty(PhTraced.RECURSION_LIMIT), ExAbstract.class ) ); @@ -316,8 +316,8 @@ void doesNotThrowExceptionIfMaxDepth() { () -> new Dataized(cage).take(), String.format( "We expect that dataizing of nested cage which recursion depth is equal to property %s = %s does not throw %s", - PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME, - System.getProperty(PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME), + PhTraced.RECURSION_LIMIT, + System.getProperty(PhTraced.RECURSION_LIMIT), ExAbstract.class ) ); @@ -335,8 +335,8 @@ void throwsExceptionIfBigDepth() { () -> new Dataized(cage).take(), String.format( "We expect that dataizing of nested cage which recursion depth is more than property %s = %s does not throw %s", - PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME, - System.getProperty(PhTraced.MAX_CAGE_RECURSION_DEPTH_PROPERTY_NAME), + PhTraced.RECURSION_LIMIT, + System.getProperty(PhTraced.RECURSION_LIMIT), ExAbstract.class ) ); diff --git a/eo-runtime/src/test/java/org/eolang/AtCompositeTest.java b/eo-runtime/src/test/java/org/eolang/AtCompositeTest.java index df6df237b4..de3017f998 100644 --- a/eo-runtime/src/test/java/org/eolang/AtCompositeTest.java +++ b/eo-runtime/src/test/java/org/eolang/AtCompositeTest.java @@ -34,6 +34,7 @@ * * @since 0.16 */ +@SuppressWarnings("PMD.JUnit5TestShouldBePackagePrivate") public final class AtCompositeTest { /** @@ -95,6 +96,7 @@ private static class Rnd extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Rnd() { super(); this.add( diff --git a/eo-runtime/src/test/java/org/eolang/AtLoggedTest.java b/eo-runtime/src/test/java/org/eolang/AtLoggedTest.java index 396b72cb9a..8e48279f18 100644 --- a/eo-runtime/src/test/java/org/eolang/AtLoggedTest.java +++ b/eo-runtime/src/test/java/org/eolang/AtLoggedTest.java @@ -39,7 +39,7 @@ * * @since 0.29 */ -class AtLoggedTest { +final class AtLoggedTest { /** * Testable object. diff --git a/eo-runtime/src/test/java/org/eolang/AtNamedTest.java b/eo-runtime/src/test/java/org/eolang/AtNamedTest.java index 2c25d1c743..4ce46c0153 100644 --- a/eo-runtime/src/test/java/org/eolang/AtNamedTest.java +++ b/eo-runtime/src/test/java/org/eolang/AtNamedTest.java @@ -64,6 +64,7 @@ private static class Dummy extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Dummy() { super(); this.add( diff --git a/eo-runtime/src/test/java/org/eolang/BytesOfTest.java b/eo-runtime/src/test/java/org/eolang/BytesOfTest.java index 42a36e24d7..122e534dd3 100644 --- a/eo-runtime/src/test/java/org/eolang/BytesOfTest.java +++ b/eo-runtime/src/test/java/org/eolang/BytesOfTest.java @@ -35,6 +35,7 @@ * * @since 1.0 */ +@SuppressWarnings("PMD.TooManyMethods") final class BytesOfTest { @Test diff --git a/eo-runtime/src/test/java/org/eolang/DataTest.java b/eo-runtime/src/test/java/org/eolang/DataTest.java index c9d5ffa10f..5e31fcb248 100644 --- a/eo-runtime/src/test/java/org/eolang/DataTest.java +++ b/eo-runtime/src/test/java/org/eolang/DataTest.java @@ -24,7 +24,6 @@ package org.eolang; import java.nio.charset.StandardCharsets; -import java.util.regex.Pattern; import org.hamcrest.MatcherAssert; import org.hamcrest.Matchers; import org.junit.jupiter.api.Test; diff --git a/eo-runtime/src/test/java/org/eolang/DataizedTest.java b/eo-runtime/src/test/java/org/eolang/DataizedTest.java index e1fbceba96..eb267bda19 100644 --- a/eo-runtime/src/test/java/org/eolang/DataizedTest.java +++ b/eo-runtime/src/test/java/org/eolang/DataizedTest.java @@ -41,6 +41,10 @@ * @since 0.22 */ final class DataizedTest { + /** + * System property for maximum dataization log level. + */ + private static final String MAX_DATAIZATION = "max.dataization.log"; @Test void logsCorrectly() { @@ -100,24 +104,13 @@ void printsShortLogs() throws InterruptedException { final List logs = new LinkedList<>(); final Handler hnd = new Hnd(logs); log.addHandler(hnd); - final Thread thread = new Thread( - () -> { - final String property = System.getProperty("max.dataization.log"); - System.getProperties().setProperty("max.dataization.log", String.valueOf(1)); - final Phi phi = new PhiDec(); - new Dataized(phi, log).take(); - if (property != null) { - System.getProperties().setProperty("max.dataization.log", property); - } else { - System.clearProperty("max.dataization.log"); - } - }); + final Thread thread = new DataizedLoggerThread(log, 1); thread.start(); thread.join(); log.setLevel(before); log.removeHandler(hnd); MatcherAssert.assertThat( - AtCompositeTest.TO_ADD_MESSAGE, + "Number of log records should be equal to 1 in the case of short logs", logs.size(), Matchers.equalTo(1) ); @@ -131,29 +124,54 @@ void printsLongLogs() throws InterruptedException { final List logs = new LinkedList<>(); final Handler hnd = new Hnd(logs); log.addHandler(hnd); - final Thread thread = new Thread( - () -> { - final String property = System.getProperty("max.dataization.log"); - System.getProperties().setProperty("max.dataization.log", String.valueOf(2)); - final Phi phi = new PhiDec(); - new Dataized(phi, log).take(); - if (property != null) { - System.getProperties().setProperty("max.dataization.log", property); - } else { - System.clearProperty("max.dataization.log"); - } - }); + final Thread thread = new DataizedLoggerThread(log, 2); thread.start(); thread.join(); log.setLevel(before); log.removeHandler(hnd); MatcherAssert.assertThat( - AtCompositeTest.TO_ADD_MESSAGE, + "Number of log records should be greater than 1 the in case of long logs", logs.size(), Matchers.greaterThan(1) ); } + /** + * Thread that logs dataization process with specified log level. + * @since 1.0 + */ + private static class DataizedLoggerThread extends Thread { + /** + * Ctor. + * + * @param logger Logger. + * @param level Log level. + */ + DataizedLoggerThread(final Logger logger, final int level) { + super( + () -> { + final String property = System.getProperty( + DataizedTest.MAX_DATAIZATION + ); + System.getProperties().setProperty( + DataizedTest.MAX_DATAIZATION, + String.valueOf(level) + ); + final Phi phi = new PhiDec(); + new Dataized(phi, logger).take(); + if (property != null) { + System.getProperties().setProperty( + DataizedTest.MAX_DATAIZATION, + property + ); + } else { + System.clearProperty(DataizedTest.MAX_DATAIZATION); + } + } + ); + } + } + /** * Fake Phi failing when dataized. * @since 1.0 @@ -163,6 +181,7 @@ private static class PhIncorrect extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") PhIncorrect() { this.add( "Δ", @@ -184,6 +203,7 @@ public static class PhiDec extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") PhiDec() { this.add( "φ", @@ -232,7 +252,7 @@ public void flush() { } @Override - public void close() throws SecurityException { + public void close() { throw new UnsupportedOperationException("#close()"); } diff --git a/eo-runtime/src/test/java/org/eolang/ExInterruptedTest.java b/eo-runtime/src/test/java/org/eolang/ExInterruptedTest.java index 9866ac5444..56a9a92973 100644 --- a/eo-runtime/src/test/java/org/eolang/ExInterruptedTest.java +++ b/eo-runtime/src/test/java/org/eolang/ExInterruptedTest.java @@ -31,7 +31,7 @@ * * @since 0.28.3 */ -public class ExInterruptedTest { +final class ExInterruptedTest { @Test void throwsRightException() { @@ -52,6 +52,7 @@ private static class EOthrow extends PhDefault { /** * CTor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") EOthrow() { this.add( "φ", diff --git a/eo-runtime/src/test/java/org/eolang/MainTest.java b/eo-runtime/src/test/java/org/eolang/MainTest.java index 540070c882..5ac67c0f90 100644 --- a/eo-runtime/src/test/java/org/eolang/MainTest.java +++ b/eo-runtime/src/test/java/org/eolang/MainTest.java @@ -30,10 +30,8 @@ import java.io.IOException; import java.nio.channels.Channels; import java.nio.charset.StandardCharsets; -import java.util.Arrays; import org.hamcrest.MatcherAssert; import org.hamcrest.Matchers; -import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; /** @@ -41,6 +39,7 @@ * * @since 0.1 */ +@SuppressWarnings("PMD.TooManyMethods") final class MainTest { @Test diff --git a/eo-runtime/src/test/java/org/eolang/PhDataTest.java b/eo-runtime/src/test/java/org/eolang/PhDataTest.java index f2c1ad94fe..65523c4fe8 100644 --- a/eo-runtime/src/test/java/org/eolang/PhDataTest.java +++ b/eo-runtime/src/test/java/org/eolang/PhDataTest.java @@ -32,7 +32,7 @@ * Test case for {@link PhData}. * @since 0.36.0 */ -public class PhDataTest { +final class PhDataTest { @Test void addsApplicationWithDelta() { MatcherAssert.assertThat( @@ -44,7 +44,7 @@ void addsApplicationWithDelta() { @Test void returnsData() { - final byte[] data = new byte[] {0x2A, 0x3B}; + final byte[] data = {0x2A, 0x3B}; MatcherAssert.assertThat( AtCompositeTest.TO_ADD_MESSAGE, new PhData(new PhDefault(), data).delta(), diff --git a/eo-runtime/src/test/java/org/eolang/PhDefaultTest.java b/eo-runtime/src/test/java/org/eolang/PhDefaultTest.java index 20f49471f0..33d7e8a23c 100644 --- a/eo-runtime/src/test/java/org/eolang/PhDefaultTest.java +++ b/eo-runtime/src/test/java/org/eolang/PhDefaultTest.java @@ -41,7 +41,17 @@ * Test case for {@link PhDefault}. * @since 0.1 */ +@SuppressWarnings("PMD.TooManyMethods") final class PhDefaultTest { + /** + * Name of attribute. + */ + private static final String PLUS_ATT = "plus"; + + /** + * Name of attribute. + */ + private static final String VOID_ATT = "void"; @Test void comparesTwoObjects() { @@ -82,7 +92,7 @@ void doesNotHaveRhoWhenFormed() { @Test void setsRhoAfterDispatch() { - final Phi kid = new PhDefaultTest.Int().take("plus"); + final Phi kid = new PhDefaultTest.Int().take(PhDefaultTest.PLUS_ATT); Assertions.assertDoesNotThrow( () -> kid.take(Attr.RHO), String.format("Kid of should have %s attribute after dispatch", Attr.RHO) @@ -102,8 +112,8 @@ void doesNotHaveRhoAfterCopying() { @Test void copiesKid() { final Phi phi = new PhDefaultTest.Int(); - final Phi first = phi.take("plus"); - final Phi second = phi.copy().take("plus"); + final Phi first = phi.take(PhDefaultTest.PLUS_ATT); + final Phi second = phi.copy().take(PhDefaultTest.PLUS_ATT); MatcherAssert.assertThat( "Child attributes should be copied after copying main object", first, @@ -118,9 +128,9 @@ void takesDifferentAbstractKidsEveryDispatch() { final Phi phi = new PhDefaultTest.Int(); MatcherAssert.assertThat( "Child attributes should be copied on every dispatch", - phi.take("plus"), + phi.take(PhDefaultTest.PLUS_ATT), Matchers.not( - Matchers.equalTo(phi.take("plus")) + Matchers.equalTo(phi.take(PhDefaultTest.PLUS_ATT)) ) ); } @@ -128,7 +138,7 @@ void takesDifferentAbstractKidsEveryDispatch() { @Test void hasKidWithSetRhoAfterCopying() { final Phi phi = new PhDefaultTest.Int().copy(); - final Phi plus = phi.take("plus"); + final Phi plus = phi.take(PhDefaultTest.PLUS_ATT); Assertions.assertDoesNotThrow( () -> plus.take(Attr.RHO), String.format( @@ -153,9 +163,9 @@ void hasDifferentKidsAfterDoubleCopying() { final Phi second = first.copy(); MatcherAssert.assertThat( "Child objects after double copying should be different", - first.take("plus"), + first.take(PhDefaultTest.PLUS_ATT), Matchers.not( - Matchers.equalTo(second.take("plus")) + Matchers.equalTo(second.take(PhDefaultTest.PLUS_ATT)) ) ); } @@ -168,15 +178,15 @@ void changesKidRhoAfterSelfCopying() { String.format( "%s attribute of original object kid should refer to original object", Attr.RHO ), - phi.take("plus").take(Attr.RHO), - Matchers.not(Matchers.equalTo(copy.take("plus").take(Attr.RHO))) + phi.take(PhDefaultTest.PLUS_ATT).take(Attr.RHO), + Matchers.not(Matchers.equalTo(copy.take(PhDefaultTest.PLUS_ATT).take(Attr.RHO))) ); MatcherAssert.assertThat( String.format( "%s attribute of copied object kid should refer to copied object", Attr.RHO ), - copy.take("plus").take(Attr.RHO), + copy.take(PhDefaultTest.PLUS_ATT).take(Attr.RHO), Matchers.equalTo(copy) ); } @@ -184,7 +194,7 @@ void changesKidRhoAfterSelfCopying() { @Test void doesNotChangeRhoAfterDirectKidCopying() { final Phi phi = new PhDefaultTest.Int(); - final Phi first = phi.take("plus"); + final Phi first = phi.take(PhDefaultTest.PLUS_ATT); final Phi second = first.copy(); MatcherAssert.assertThat( String.format( @@ -201,7 +211,7 @@ void doesNotChangeRhoAfterDirectKidCopying() { @Test void doesNotCopyRhoWhileDispatch() { final Phi phi = new PhDefaultTest.Int(); - final Phi plus = phi.take("plus"); + final Phi plus = phi.take(PhDefaultTest.PLUS_ATT); MatcherAssert.assertThat( String.format("%s attributes should not be copied while dispatch", Attr.RHO), plus.take(Attr.RHO), @@ -215,7 +225,7 @@ void copiesUnsetVoidAttribute() { final Phi copy = phi.copy(); Assertions.assertThrows( EOerror.ExError.class, - () -> copy.take("void"), + () -> copy.take(PhDefaultTest.VOID_ATT), "Unset void attribute should be copied with unset value" ); } @@ -223,13 +233,13 @@ void copiesUnsetVoidAttribute() { @Test void copiesSetVoidAttributeOnCopy() { final Phi phi = new PhDefaultTest.Int(); - phi.put("void", new Data.ToPhi(10L)); + phi.put(PhDefaultTest.VOID_ATT, new Data.ToPhi(10L)); final Phi copy = phi.copy(); MatcherAssert.assertThat( "Copied set void attribute should be different from original one", - phi.take("void"), + phi.take(PhDefaultTest.VOID_ATT), Matchers.not( - Matchers.equalTo(copy.take("void")) + Matchers.equalTo(copy.take(PhDefaultTest.VOID_ATT)) ) ); } @@ -237,11 +247,11 @@ void copiesSetVoidAttributeOnCopy() { @Test void doesNotCopySetVoidAttributeWithRho() { final Phi phi = new PhDefaultTest.Int(); - phi.put("void", new Data.ToPhi(10L)); + phi.put(PhDefaultTest.VOID_ATT, new Data.ToPhi(10L)); MatcherAssert.assertThat( AtCompositeTest.TO_ADD_MESSAGE, - phi.take("void"), - Matchers.equalTo(phi.take("void")) + phi.take(PhDefaultTest.VOID_ATT), + Matchers.equalTo(phi.take(PhDefaultTest.VOID_ATT)) ); } @@ -263,7 +273,7 @@ void hasAccessToDependentOnContextAttribute() { () -> phi.take(Attr.PHI), AtCompositeTest.TO_ADD_MESSAGE ); - phi.put("void", new Data.ToPhi(10L)); + phi.put(PhDefaultTest.VOID_ATT, new Data.ToPhi(10L)); Assertions.assertDoesNotThrow( () -> phi.take(Attr.PHI), AtCompositeTest.TO_ADD_MESSAGE @@ -379,10 +389,10 @@ void cachesPhiViaNewRecursively() { @Test void refersToOriginalObjectAndDoesNotResetCache() { final Phi phi = new PhDefaultTest.Dummy(); - phi.take("plus"); + phi.take(PhDefaultTest.PLUS_ATT); final Phi copy = phi.copy(); - copy.take("plus"); - phi.take("plus"); + copy.take(PhDefaultTest.PLUS_ATT); + phi.take(PhDefaultTest.PLUS_ATT); MatcherAssert.assertThat( AtCompositeTest.TO_ADD_MESSAGE, PhDefaultTest.Dummy.count, @@ -422,7 +432,7 @@ void hasDifferentFormaWithBoundedMethod() { Matchers.not( Matchers.equalTo( new PhWith( - five.take("plus").copy(), + five.take(PhDefaultTest.PLUS_ATT).copy(), "x", new Data.ToPhi(5L) ).forma() @@ -436,13 +446,13 @@ void hasTheSameFormaWithDifferentInstances() { MatcherAssert.assertThat( AtCompositeTest.TO_ADD_MESSAGE, new PhWith( - new Data.ToPhi(5L).take("plus").copy(), + new Data.ToPhi(5L).take(PhDefaultTest.PLUS_ATT).copy(), "x", new Data.ToPhi(5L) ).forma(), Matchers.equalTo( new PhWith( - new Data.ToPhi(6L).take("plus").copy(), + new Data.ToPhi(6L).take(PhDefaultTest.PLUS_ATT).copy(), "x", new Data.ToPhi(6L) ).forma() @@ -456,11 +466,11 @@ void doesNotCalculateRandomTwice() { new PhMethod( new PhWith( new PhMethod( - new Rnd(), "plus" + new Rnd(), PhDefaultTest.PLUS_ATT ), 0, new Data.ToPhi(1.2) ), - "plus" + PhDefaultTest.PLUS_ATT ), 0, new Data.ToPhi(1.2) ); @@ -488,6 +498,7 @@ private static class Rnd extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Rnd() { this.add( "φ", @@ -509,15 +520,16 @@ private static class Int extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Int() { - this.add("void", new AtVoid("void")); - this.add("plus", new AtSimple(new PhDefault())); + this.add(PhDefaultTest.VOID_ATT, new AtVoid(PhDefaultTest.VOID_ATT)); + this.add(PhDefaultTest.PLUS_ATT, new AtSimple(new PhDefault())); this.add( Attr.PHI, new AtOnce( new AtComposite( this, - rho -> rho.take("void") + rho -> rho.take(PhDefaultTest.VOID_ATT) ) ) ); @@ -527,7 +539,9 @@ private static class Int extends PhDefault { new AtComposite( this, rho -> { - final Phi plus = new Data.ToPhi(5L).take("plus").copy(); + final Phi plus = new Data.ToPhi(5L).take( + PhDefaultTest.PLUS_ATT + ).copy(); plus.put(0, new Data.ToPhi(6L)); return plus; } @@ -545,6 +559,7 @@ public static class Foo extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Foo() { this.add("x", new AtVoid("x")); this.add("kid", new AtSimple(new PhDefaultTest.Kid())); @@ -565,6 +580,7 @@ public static class Dummy extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Dummy() { this.add( Attr.PHI, @@ -591,6 +607,7 @@ public static class Counter extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Counter() { this.add( Attr.PHI, @@ -613,6 +630,7 @@ public static class Kid extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Kid() { this.add("z", new AtVoid("z")); this.add(Attr.PHI, new AtSimple(new EOstdout())); @@ -632,6 +650,7 @@ public static class EndlessRecursion extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") EndlessRecursion() { this.add( Attr.PHI, @@ -665,6 +684,7 @@ public static class RecursivePhi extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") RecursivePhi() { this.add( "φ", @@ -698,6 +718,7 @@ public static class RecursivePhiViaNew extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") RecursivePhiViaNew() { this.add( "φ", diff --git a/eo-runtime/src/test/java/org/eolang/PhLocatedTest.java b/eo-runtime/src/test/java/org/eolang/PhLocatedTest.java index 66678764b9..1a423fc4c3 100644 --- a/eo-runtime/src/test/java/org/eolang/PhLocatedTest.java +++ b/eo-runtime/src/test/java/org/eolang/PhLocatedTest.java @@ -32,7 +32,7 @@ * * @since 0.36.0 */ -class PhLocatedTest { +final class PhLocatedTest { @Test void savesLocationAfterCopying() { diff --git a/eo-runtime/src/test/java/org/eolang/PhLoggedTest.java b/eo-runtime/src/test/java/org/eolang/PhLoggedTest.java index 43caffb7af..37e5a6d2cb 100644 --- a/eo-runtime/src/test/java/org/eolang/PhLoggedTest.java +++ b/eo-runtime/src/test/java/org/eolang/PhLoggedTest.java @@ -33,7 +33,7 @@ * * @since 0.29.0 */ -class PhLoggedTest { +final class PhLoggedTest { @Test void convertsToOriginTerm() { @@ -99,6 +99,7 @@ private static final class Dummy extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") private Dummy() { this.add("x", new AtVoid("x")); } diff --git a/eo-runtime/src/test/java/org/eolang/PhMethodTest.java b/eo-runtime/src/test/java/org/eolang/PhMethodTest.java index fd128f4727..6215788f64 100644 --- a/eo-runtime/src/test/java/org/eolang/PhMethodTest.java +++ b/eo-runtime/src/test/java/org/eolang/PhMethodTest.java @@ -25,7 +25,6 @@ import org.hamcrest.MatcherAssert; import org.hamcrest.Matchers; -import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; /** @@ -122,6 +121,7 @@ public static class Dummy extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Dummy() { this.add( "φ", diff --git a/eo-runtime/src/test/java/org/eolang/PhPackageTest.java b/eo-runtime/src/test/java/org/eolang/PhPackageTest.java index f77f6b7423..4b8d5bed22 100644 --- a/eo-runtime/src/test/java/org/eolang/PhPackageTest.java +++ b/eo-runtime/src/test/java/org/eolang/PhPackageTest.java @@ -47,6 +47,7 @@ * * @since 0.24 */ +@SuppressWarnings("PMD.TooManyMethods") final class PhPackageTest { /** @@ -156,6 +157,7 @@ void convertsToString() { } @Test + @SuppressWarnings("PMD.CloseResource") void findsAttributesConcurrently() throws InterruptedException { final int threads = Runtime.getRuntime().availableProcessors() + 10; final ExecutorService service = Executors.newFixedThreadPool(threads); diff --git a/eo-runtime/src/test/java/org/eolang/PhWithTest.java b/eo-runtime/src/test/java/org/eolang/PhWithTest.java index 40ee2f7398..bd532c831b 100644 --- a/eo-runtime/src/test/java/org/eolang/PhWithTest.java +++ b/eo-runtime/src/test/java/org/eolang/PhWithTest.java @@ -134,6 +134,7 @@ private static class DummyWithAtFree extends PhDefault { * Ctor. * @param attr Free attribute name */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") DummyWithAtFree(final String attr) { this.add(attr, new AtVoid(attr)); } @@ -148,6 +149,7 @@ public static class Dummy extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") Dummy() { this.add("φ", new AtComposite(this, self -> new Data.ToPhi(1L))); } diff --git a/eo-runtime/src/test/java/org/eolang/SnippetTestCase.java b/eo-runtime/src/test/java/org/eolang/SnippetTestCase.java index 1d9d875850..eeb313f57b 100755 --- a/eo-runtime/src/test/java/org/eolang/SnippetTestCase.java +++ b/eo-runtime/src/test/java/org/eolang/SnippetTestCase.java @@ -67,6 +67,22 @@ @ExtendWith(WeAreOnline.class) @SuppressWarnings({"JTCOP.RuleAllTestsHaveProductionClass", "JTCOP.RuleNotContainsTestWord"}) final class SnippetTestCase { + + /** + * True. + */ + private static final String TRUE = "true"; + + /** + * False. + */ + private static final String FALSE = "false"; + + /** + * UTF-8. + */ + private static final String UTF_8 = "UTF-8"; + @ParameterizedTest @Tag("slow") @ExtendWith(WeAreOnline.class) @@ -79,8 +95,8 @@ void runsAllSnippets(final String yml, final @TempDir Path temp) throws IOExcept new Farea(temp).together( f -> { f.properties() - .set("project.build.sourceEncoding", "UTF-8") - .set("project.reporting.outputEncoding", "UTF-8"); + .set("project.build.sourceEncoding", SnippetTestCase.UTF_8) + .set("project.reporting.outputEncoding", SnippetTestCase.UTF_8); SnippetTestCase.copySources(f, "src/main/eo"); f.files() .file(String.format("src/main/eo/%s", file)) @@ -98,7 +114,7 @@ void runsAllSnippets(final String yml, final @TempDir Path temp) throws IOExcept .phase("generate-sources") .goals("register", "assemble", "verify", "transpile") .configuration() - .set("failOnWarnings", "true"); + .set("failOnWarnings", SnippetTestCase.TRUE); f.build() .plugins() .append("org.codehaus.mojo", "exec-maven-plugin", "3.1.1") @@ -136,8 +152,8 @@ void runsTestsAfterPhiAndUnphi(final @TempDir Path temp) throws IOException { SnippetTestCase.copySources(f, "src/main"); SnippetTestCase.copySources(f, "src/test/eo"); f.properties() - .set("project.build.sourceEncoding", "UTF-8") - .set("project.reporting.outputEncoding", "UTF-8"); + .set("project.build.sourceEncoding", SnippetTestCase.UTF_8) + .set("project.reporting.outputEncoding", SnippetTestCase.UTF_8); f.dependencies().append( "net.sf.saxon", "Saxon-HE", @@ -170,7 +186,7 @@ void runsTestsAfterPhiAndUnphi(final @TempDir Path temp) throws IOException { .set("unphiMetas", new String[]{"+tests"}) .set("printSourcesDir", "${project.build.directory}/generated-eo-test/1-parse") .set("printOutputDir", "${project.basedir}/src/test/generated-eo") - .set("printReversed", "true"); + .set("printReversed", SnippetTestCase.TRUE); f.exec("clean", "compile"); final String phi = f.log(); MatcherAssert.assertThat( @@ -180,8 +196,8 @@ void runsTestsAfterPhiAndUnphi(final @TempDir Path temp) throws IOException { ); f.files().file("pom.xml").delete(); f.properties() - .set("project.build.sourceEncoding", "UTF-8") - .set("project.reporting.outputEncoding", "UTF-8"); + .set("project.build.sourceEncoding", SnippetTestCase.UTF_8) + .set("project.reporting.outputEncoding", SnippetTestCase.UTF_8); f.dependencies().append( "org.junit.jupiter", "junit-jupiter-api", @@ -207,10 +223,10 @@ void runsTestsAfterPhiAndUnphi(final @TempDir Path temp) throws IOException { .configuration() .set("foreign", "${project.basedir}/target/eo-foreign.csv") .set("foreignFormat", "csv") - .set("failOnWarning", "false") - .set("offline", "true") - .set("withRuntimeDependency", "false") - .set("placeBinariesThatHaveSources", "true"); + .set("failOnWarning", SnippetTestCase.FALSE) + .set("offline", SnippetTestCase.TRUE) + .set("withRuntimeDependency", SnippetTestCase.FALSE) + .set("placeBinariesThatHaveSources", SnippetTestCase.TRUE); f.build() .plugins() .append( @@ -230,17 +246,17 @@ void runsTestsAfterPhiAndUnphi(final @TempDir Path temp) throws IOException { .configuration() .set("foreign", "${project.basedir}/target/eo-foreign.csv") .set("foreignFormat", "csv") - .set("failOnWarning", "false") - .set("offline", "true") + .set("failOnWarning", SnippetTestCase.FALSE) + .set("offline", SnippetTestCase.TRUE) .set("scope", "test") .set("sourcesDir", "${project.basedir}/src/test/generated-eo") .set("targetDir", "${project.basedir}/target/eo-test") - .set("addSourcesRoot", "false") - .set("addTestSourcesRoot", "true") - .set("failOnWarning", "false") + .set("addSourcesRoot", SnippetTestCase.FALSE) + .set("addTestSourcesRoot", SnippetTestCase.TRUE) + .set("failOnWarning", SnippetTestCase.FALSE) .set("generatedDir", "${project.basedir}/target/generated-test-sources") - .set("withRuntimeDependency", "false") - .set("placeBinariesThatHaveSources", "true"); + .set("withRuntimeDependency", SnippetTestCase.FALSE) + .set("placeBinariesThatHaveSources", SnippetTestCase.TRUE); f.exec("clean", "test"); final String log = f.log(); Logger.debug(this, log); diff --git a/eo-runtime/src/test/java/org/eolang/UniverseDefaultTest.java b/eo-runtime/src/test/java/org/eolang/UniverseDefaultTest.java index b97d9a77b5..d0d3fcf9cb 100644 --- a/eo-runtime/src/test/java/org/eolang/UniverseDefaultTest.java +++ b/eo-runtime/src/test/java/org/eolang/UniverseDefaultTest.java @@ -52,7 +52,7 @@ final class UniverseDefaultTest { /** * Data byte array. */ - private static final byte[] DATA = new BytesOf(123456789L).take(); + private static final byte[] DATA = new BytesOf(123_456_789L).take(); @Test void findsSimpleAtt() { @@ -206,6 +206,7 @@ private static class DummyWithAt extends PhDefault { * Ctor. * @param sigma Sigma. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") DummyWithAt(final Phi sigma) { this.add( UniverseDefaultTest.ABSTRACT_ATT, @@ -227,6 +228,7 @@ private static class DummyWithStructure extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") DummyWithStructure() { this.add(UniverseDefaultTest.ABSTRACT_ATT, new AtComposite(this, DummyWithAt::new)); } @@ -241,6 +243,7 @@ private static class DummyAbstract extends PhDefault { /** * Ctor. */ + @SuppressWarnings("PMD.ConstructorOnlyInitializesOrCallOtherConstructors") DummyAbstract() { this.add( UniverseDefaultTest.ABSTRACT_ATT, diff --git a/eo-runtime/src/test/java/org/eolang/VerboseBytesAsStringTest.java b/eo-runtime/src/test/java/org/eolang/VerboseBytesAsStringTest.java index 825931e6d1..89bda4db6f 100644 --- a/eo-runtime/src/test/java/org/eolang/VerboseBytesAsStringTest.java +++ b/eo-runtime/src/test/java/org/eolang/VerboseBytesAsStringTest.java @@ -35,6 +35,7 @@ * * @since 0.1 */ +@SuppressWarnings("PMD.JUnit5TestShouldBePackagePrivate") public final class VerboseBytesAsStringTest { @ParameterizedTest @@ -118,9 +119,9 @@ public String toString(final Object object) { */ public Stream getTestSources() { return Stream.of( - 12345L, + 12_345L, "qwerty", - 12.34567D, + 12.345_67D, true, false, new byte[]{}, diff --git a/eo-runtime/src/test/java/org/eolang/XmirObjectTest.java b/eo-runtime/src/test/java/org/eolang/XmirObjectTest.java index b6e85faa58..d11f492a39 100644 --- a/eo-runtime/src/test/java/org/eolang/XmirObjectTest.java +++ b/eo-runtime/src/test/java/org/eolang/XmirObjectTest.java @@ -40,14 +40,14 @@ * * @since 0.38 */ -public class XmirObjectTest { +final class XmirObjectTest { @Test - public void annotatesOnlyPublicClasses() throws IOException { + void annotatesOnlyPublicClasses() throws IOException { final Set> clazzes = ClassPath.from(ClassLoader.getSystemClassLoader()) .getAllClasses() .stream() - .filter(clazz -> clazz.getPackageName().equals("EOorg.EOeolang")) + .filter(clazz -> "EOorg.EOeolang".equals(clazz.getPackageName())) .map(ClassPath.ClassInfo::load) .filter( clazz -> clazz.getSimpleName().startsWith("EO")