From 6627ac3974c94666e5986023e9bb09eae28a3d73 Mon Sep 17 00:00:00 2001
From: Andreas Dangel
Date: Thu, 30 Jun 2022 10:26:56 +0200
Subject: [PATCH 01/31] [apex] Migrate tests to JUnit5
---
.../net/sourceforge/pmd/cpd/ApexCpdTest.java | 16 ++---
.../pmd/cpd/ApexTokenizerTest.java | 14 ++--
.../pmd/lang/apex/ast/ASTFieldTest.java | 24 +++----
.../pmd/lang/apex/ast/ASTMethodTest.java | 15 +++--
.../ASTNewKeyValueObjectExpressionTest.java | 19 +++---
.../lang/apex/ast/ASTSoqlExpressionTest.java | 11 ++--
.../lang/apex/ast/ASTSwitchStatementTest.java | 50 ++++++++-------
.../ASTTryCatchFinallyBlockStatementTest.java | 51 ++++++++-------
.../pmd/lang/apex/ast/ASTUserClassTest.java | 27 ++++----
.../pmd/lang/apex/ast/ASTUserEnumTest.java | 11 ++--
.../lang/apex/ast/ASTUserInterfaceTest.java | 23 +++----
.../pmd/lang/apex/ast/ASTUserTriggerTest.java | 18 +++---
.../lang/apex/ast/ApexCompilerSoqlTest.java | 6 +-
.../pmd/lang/apex/ast/ApexCompilerTest.java | 12 ++--
.../pmd/lang/apex/ast/ApexLexerTest.java | 16 +++--
.../pmd/lang/apex/ast/ApexParserTest.java | 64 +++++++++----------
.../lang/apex/ast/ApexQualifiedNameTest.java | 25 ++++----
.../pmd/lang/apex/ast/ApexTreeDumpTest.java | 8 +--
18 files changed, 214 insertions(+), 196 deletions(-)
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexCpdTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexCpdTest.java
index 412c4ea024..7ce83fcfcb 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexCpdTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexCpdTest.java
@@ -4,30 +4,30 @@
package net.sourceforge.pmd.cpd;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
import java.io.File;
import java.io.IOException;
import java.util.Iterator;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.lang.apex.ApexLanguageModule;
import net.sourceforge.pmd.util.IOUtil;
-public class ApexCpdTest {
+class ApexCpdTest {
private File testdir;
- @Before
- public void setUp() {
+ @BeforeEach
+ void setUp() {
String path = IOUtil.normalizePath("src/test/resources/net/sourceforge/pmd/cpd/issue427");
testdir = new File(path);
}
@Test
- public void testIssue427() throws IOException {
+ void testIssue427() throws IOException {
CPDConfiguration configuration = new CPDConfiguration();
configuration.setMinimumTileSize(10);
configuration.setLanguage(LanguageFactory.createLanguage(ApexLanguageModule.TERSE_NAME));
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexTokenizerTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexTokenizerTest.java
index 5e64f0082d..aeb4a51252 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexTokenizerTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/cpd/ApexTokenizerTest.java
@@ -6,13 +6,13 @@ package net.sourceforge.pmd.cpd;
import java.util.Properties;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.cpd.test.CpdTextComparisonTest;
-public class ApexTokenizerTest extends CpdTextComparisonTest {
+class ApexTokenizerTest extends CpdTextComparisonTest {
- public ApexTokenizerTest() {
+ ApexTokenizerTest() {
super(".cls");
}
@@ -30,12 +30,12 @@ public class ApexTokenizerTest extends CpdTextComparisonTest {
@Test
- public void testTokenize() {
+ void testTokenize() {
doTest("Simple");
}
@Test
- public void testTokenizeCaseSensitive() {
+ void testTokenizeCaseSensitive() {
doTest("Simple", "_caseSensitive", caseSensitive());
}
@@ -43,12 +43,12 @@ public class ApexTokenizerTest extends CpdTextComparisonTest {
* Comments are ignored since using ApexLexer.
*/
@Test
- public void testTokenizeWithComments() {
+ void testTokenizeWithComments() {
doTest("comments");
}
@Test
- public void testTabWidth() {
+ void testTabWidth() {
doTest("tabWidth");
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTFieldTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTFieldTest.java
index 4b9e418590..6c4797a7b6 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTFieldTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTFieldTest.java
@@ -4,34 +4,36 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Assert;
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
-public class ASTFieldTest extends ApexParserTestBase {
+import org.junit.jupiter.api.Test;
+
+class ASTFieldTest extends ApexParserTestBase {
@Test
- public void testGetType() {
+ void testGetType() {
ASTField field = parse("public class Foo { private String myField = 'a'; }")
.descendants(ASTField.class).firstOrThrow();
- Assert.assertEquals("myField", field.getImage());
- Assert.assertEquals("String", field.getType());
- Assert.assertEquals("a", field.getValue());
+ assertEquals("myField", field.getImage());
+ assertEquals("String", field.getType());
+ assertEquals("a", field.getValue());
}
@Test
- public void testGetValue() {
+ void testGetValue() {
ASTField field = parse("public class Foo { private String myField = 'a'; }")
.descendants(ASTField.class).firstOrThrow();
- Assert.assertEquals("a", field.getValue());
+ assertEquals("a", field.getValue());
}
@Test
- public void testGetNoValue() {
+ void testGetNoValue() {
ASTField field = parse("public class Foo { private String myField; }")
.descendants(ASTField.class).firstOrThrow();
- Assert.assertNull(field.getValue());
+ assertNull(field.getValue());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTMethodTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTMethodTest.java
index 7ef6efbe66..0cd0033383 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTMethodTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTMethodTest.java
@@ -4,19 +4,20 @@
package net.sourceforge.pmd.lang.apex.ast;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
import java.util.List;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
-public class ASTMethodTest extends ApexParserTestBase {
+class ASTMethodTest extends ApexParserTestBase {
@Test
- public void testConstructorName() {
+ void testConstructorName() {
ASTUserClass node = (ASTUserClass) parse("public class Foo { public Foo() {} public void bar() {} }");
List methods = node.children(ASTMethod.class).toList();
- Assert.assertEquals("Foo", methods.get(0).getImage()); // constructor
- Assert.assertEquals("", methods.get(0).getCanonicalName());
- Assert.assertEquals("bar", methods.get(1).getImage()); // normal method
+ assertEquals("Foo", methods.get(0).getImage()); // constructor
+ assertEquals("", methods.get(0).getCanonicalName());
+ assertEquals("bar", methods.get(1).getImage()); // normal method
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTNewKeyValueObjectExpressionTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTNewKeyValueObjectExpressionTest.java
index 691372eaab..bd5e58553b 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTNewKeyValueObjectExpressionTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTNewKeyValueObjectExpressionTest.java
@@ -4,15 +4,16 @@
package net.sourceforge.pmd.lang.apex.ast;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
import java.util.List;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
-public class ASTNewKeyValueObjectExpressionTest extends ApexParserTestBase {
+class ASTNewKeyValueObjectExpressionTest extends ApexParserTestBase {
@Test
- public void testParameterName() {
+ void testParameterName() {
ASTUserClassOrInterface> node = parse("public class Foo { \n"
+ " public void foo(String newName, String tempID) { \n"
+ " if (Contact.sObjectType.getDescribe().isCreateable() && Contact.sObjectType.getDescribe().isUpdateable()) {\n"
@@ -20,13 +21,13 @@ public class ASTNewKeyValueObjectExpressionTest extends ApexParserTestBase {
+ " }\n" + " } \n" + "}");
ASTNewKeyValueObjectExpression keyValueExpr = node.getFirstDescendantOfType(ASTNewKeyValueObjectExpression.class);
- Assert.assertEquals(3, keyValueExpr.getParameterCount());
+ assertEquals(3, keyValueExpr.getParameterCount());
List literals = keyValueExpr.findDescendantsOfType(ASTLiteralExpression.class);
- Assert.assertEquals(3, literals.size());
- Assert.assertEquals("FirstName", literals.get(0).getName());
- Assert.assertEquals("LastName", literals.get(1).getName());
- Assert.assertEquals("Phone", literals.get(2).getName());
+ assertEquals(3, literals.size());
+ assertEquals("FirstName", literals.get(0).getName());
+ assertEquals("LastName", literals.get(1).getName());
+ assertEquals("Phone", literals.get(2).getName());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSoqlExpressionTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSoqlExpressionTest.java
index 22ef9278b9..d8673390a3 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSoqlExpressionTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSoqlExpressionTest.java
@@ -4,15 +4,16 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Assert;
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
-public class ASTSoqlExpressionTest extends ApexParserTestBase {
+import org.junit.jupiter.api.Test;
+
+class ASTSoqlExpressionTest extends ApexParserTestBase {
@Test
- public void testQuery() {
+ void testQuery() {
ApexNode> root = parse("class Foo { void test1() { Account acc = [SELECT col FROM Account]; } }");
ASTSoqlExpression soqlExpression = root.descendants(ASTSoqlExpression.class).firstOrThrow();
- Assert.assertEquals("SELECT col FROM Account", soqlExpression.getQuery());
+ assertEquals("SELECT col FROM Account", soqlExpression.getQuery());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSwitchStatementTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSwitchStatementTest.java
index f5dddb17c0..28feed0833 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSwitchStatementTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTSwitchStatementTest.java
@@ -4,41 +4,43 @@
package net.sourceforge.pmd.lang.apex.ast;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
import java.util.List;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
-public class ASTSwitchStatementTest extends ApexParserTestBase {
+class ASTSwitchStatementTest extends ApexParserTestBase {
@Test
- public void testExamples() {
+ void testExamples() {
ApexNode> node = parseResource("SwitchStatements.cls");
List switchStatements = node.findDescendantsOfType(ASTSwitchStatement.class);
- Assert.assertEquals(4, switchStatements.size());
+ assertEquals(4, switchStatements.size());
- Assert.assertTrue(switchStatements.get(0).getChild(0) instanceof ASTVariableExpression);
- Assert.assertEquals(5, switchStatements.get(0).findChildrenOfType(ASTValueWhenBlock.class).size());
- Assert.assertEquals(3, switchStatements.get(0).findChildrenOfType(ASTValueWhenBlock.class)
- .get(1).findChildrenOfType(ASTLiteralCase.class).size());
- Assert.assertEquals(1, switchStatements.get(0).findChildrenOfType(ASTElseWhenBlock.class).size());
+ assertTrue(switchStatements.get(0).getChild(0) instanceof ASTVariableExpression);
+ assertEquals(5, switchStatements.get(0).findChildrenOfType(ASTValueWhenBlock.class).size());
+ assertEquals(3, switchStatements.get(0).findChildrenOfType(ASTValueWhenBlock.class)
+ .get(1).findChildrenOfType(ASTLiteralCase.class).size());
+ assertEquals(1, switchStatements.get(0).findChildrenOfType(ASTElseWhenBlock.class).size());
- Assert.assertTrue(switchStatements.get(1).getChild(0) instanceof ASTMethodCallExpression);
- Assert.assertEquals(2, switchStatements.get(1).findChildrenOfType(ASTValueWhenBlock.class).size());
- Assert.assertEquals(1, switchStatements.get(1).findChildrenOfType(ASTElseWhenBlock.class).size());
+ assertTrue(switchStatements.get(1).getChild(0) instanceof ASTMethodCallExpression);
+ assertEquals(2, switchStatements.get(1).findChildrenOfType(ASTValueWhenBlock.class).size());
+ assertEquals(1, switchStatements.get(1).findChildrenOfType(ASTElseWhenBlock.class).size());
- Assert.assertTrue(switchStatements.get(2).getChild(0) instanceof ASTVariableExpression);
- Assert.assertEquals(2, switchStatements.get(2).findChildrenOfType(ASTTypeWhenBlock.class).size());
- Assert.assertEquals("Account", switchStatements.get(2).findChildrenOfType(ASTTypeWhenBlock.class)
- .get(0).getType());
- Assert.assertEquals("a", switchStatements.get(2).findChildrenOfType(ASTTypeWhenBlock.class)
- .get(0).getName());
- Assert.assertEquals(1, switchStatements.get(2).findChildrenOfType(ASTValueWhenBlock.class).size());
- Assert.assertEquals(1, switchStatements.get(2).findChildrenOfType(ASTElseWhenBlock.class).size());
+ assertTrue(switchStatements.get(2).getChild(0) instanceof ASTVariableExpression);
+ assertEquals(2, switchStatements.get(2).findChildrenOfType(ASTTypeWhenBlock.class).size());
+ assertEquals("Account", switchStatements.get(2).findChildrenOfType(ASTTypeWhenBlock.class)
+ .get(0).getType());
+ assertEquals("a", switchStatements.get(2).findChildrenOfType(ASTTypeWhenBlock.class)
+ .get(0).getName());
+ assertEquals(1, switchStatements.get(2).findChildrenOfType(ASTValueWhenBlock.class).size());
+ assertEquals(1, switchStatements.get(2).findChildrenOfType(ASTElseWhenBlock.class).size());
- Assert.assertTrue(switchStatements.get(3).getChild(0) instanceof ASTVariableExpression);
- Assert.assertEquals(2, switchStatements.get(3).findChildrenOfType(ASTValueWhenBlock.class).size());
- Assert.assertEquals(1, switchStatements.get(3).findChildrenOfType(ASTElseWhenBlock.class).size());
+ assertTrue(switchStatements.get(3).getChild(0) instanceof ASTVariableExpression);
+ assertEquals(2, switchStatements.get(3).findChildrenOfType(ASTValueWhenBlock.class).size());
+ assertEquals(1, switchStatements.get(3).findChildrenOfType(ASTElseWhenBlock.class).size());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTTryCatchFinallyBlockStatementTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTTryCatchFinallyBlockStatementTest.java
index a6cd4df9b9..d71a59f6df 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTTryCatchFinallyBlockStatementTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTTryCatchFinallyBlockStatementTest.java
@@ -4,44 +4,47 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Assert;
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
-public class ASTTryCatchFinallyBlockStatementTest extends ApexParserTestBase {
+import org.junit.jupiter.api.Test;
+
+class ASTTryCatchFinallyBlockStatementTest extends ApexParserTestBase {
@Test
- public void testTryFinally() {
+ void testTryFinally() {
ApexNode> node = parse("class Foo { void bar() { try { methodCall(); } finally { methodCall(); } } }");
ASTTryCatchFinallyBlockStatement statement = node.getFirstDescendantOfType(ASTTryCatchFinallyBlockStatement.class);
- Assert.assertNotNull(statement.getTryBlock());
- Assert.assertEquals(0, statement.getTryBlock().getIndexInParent());
- Assert.assertNotNull(statement.getFinallyBlock());
- Assert.assertEquals(1, statement.getFinallyBlock().getIndexInParent());
- Assert.assertEquals(0, statement.getCatchClauses().size());
+ assertNotNull(statement.getTryBlock());
+ assertEquals(0, statement.getTryBlock().getIndexInParent());
+ assertNotNull(statement.getFinallyBlock());
+ assertEquals(1, statement.getFinallyBlock().getIndexInParent());
+ assertEquals(0, statement.getCatchClauses().size());
}
@Test
- public void testTryCatch() {
+ void testTryCatch() {
ApexNode> node = parse("class Foo { void bar() { try { methodCall(); } catch (Exception e) { methodCall(); } } }");
ASTTryCatchFinallyBlockStatement statement = node.getFirstDescendantOfType(ASTTryCatchFinallyBlockStatement.class);
- Assert.assertNotNull(statement.getTryBlock());
- Assert.assertEquals(0, statement.getTryBlock().getIndexInParent());
- Assert.assertNull(statement.getFinallyBlock());
- Assert.assertEquals(1, statement.getCatchClauses().size());
- Assert.assertNotNull(statement.getCatchClauses().get(0).getBody());
- Assert.assertEquals(1, statement.getCatchClauses().get(0).getIndexInParent());
+ assertNotNull(statement.getTryBlock());
+ assertEquals(0, statement.getTryBlock().getIndexInParent());
+ assertNull(statement.getFinallyBlock());
+ assertEquals(1, statement.getCatchClauses().size());
+ assertNotNull(statement.getCatchClauses().get(0).getBody());
+ assertEquals(1, statement.getCatchClauses().get(0).getIndexInParent());
}
@Test
- public void testTryCatchFinally() {
+ void testTryCatchFinally() {
ApexNode> node = parse("class Foo { void bar() { try { methodCall(); } catch (Exception e) { methodCall(); } finally { } } }");
ASTTryCatchFinallyBlockStatement statement = node.getFirstDescendantOfType(ASTTryCatchFinallyBlockStatement.class);
- Assert.assertNotNull(statement.getTryBlock());
- Assert.assertEquals(0, statement.getTryBlock().getIndexInParent());
- Assert.assertNotNull(statement.getFinallyBlock());
- Assert.assertEquals(2, statement.getFinallyBlock().getIndexInParent());
- Assert.assertEquals(1, statement.getCatchClauses().size());
- Assert.assertNotNull(statement.getCatchClauses().get(0).getBody());
- Assert.assertEquals(1, statement.getCatchClauses().get(0).getIndexInParent());
+ assertNotNull(statement.getTryBlock());
+ assertEquals(0, statement.getTryBlock().getIndexInParent());
+ assertNotNull(statement.getFinallyBlock());
+ assertEquals(2, statement.getFinallyBlock().getIndexInParent());
+ assertEquals(1, statement.getCatchClauses().size());
+ assertNotNull(statement.getCatchClauses().get(0).getBody());
+ assertEquals(1, statement.getCatchClauses().get(0).getIndexInParent());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserClassTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserClassTest.java
index ffa83ee1fa..97120ce364 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserClassTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserClassTest.java
@@ -4,41 +4,42 @@
package net.sourceforge.pmd.lang.apex.ast;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
import java.util.Arrays;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
-public class ASTUserClassTest extends ApexParserTestBase {
+class ASTUserClassTest extends ApexParserTestBase {
@Test
- public void testClassName() {
+ void testClassName() {
ASTUserClass node = (ASTUserClass) parse("class Foo { }");
- Assert.assertEquals("Foo", node.getSimpleName());
+ assertEquals("Foo", node.getSimpleName());
}
@Test
- public void testInnerClassName() {
+ void testInnerClassName() {
ASTUserClass foo = (ASTUserClass) parse("class Foo { class Bar { } }");
ASTUserClass innerNode = foo.descendants(ASTUserClass.class).firstOrThrow();
- Assert.assertEquals("Bar", innerNode.getSimpleName());
+ assertEquals("Bar", innerNode.getSimpleName());
}
@Test
- public void testSuperClassName() {
+ void testSuperClassName() {
ASTUserClass toplevel = (ASTUserClass) parse("public class AccountTriggerHandler extends TriggerHandler {}");
- Assert.assertEquals("TriggerHandler", toplevel.getSuperClassName());
+ assertEquals("TriggerHandler", toplevel.getSuperClassName());
}
@Test
- public void testSuperClassName2() {
+ void testSuperClassName2() {
ASTUserClass toplevel = (ASTUserClass) parse("public class AccountTriggerHandler extends Other.TriggerHandler {}");
- Assert.assertEquals("Other.TriggerHandler", toplevel.getSuperClassName());
+ assertEquals("Other.TriggerHandler", toplevel.getSuperClassName());
}
@Test
- public void testInterfaces() {
+ void testInterfaces() {
ASTUserClass toplevel = (ASTUserClass) parse("public class AccountTriggerHandler implements TriggerHandler, Other.Interface2 {}");
- Assert.assertEquals(Arrays.asList("TriggerHandler", "Other.Interface2"), toplevel.getInterfaceNames());
+ assertEquals(Arrays.asList("TriggerHandler", "Other.Interface2"), toplevel.getInterfaceNames());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserEnumTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserEnumTest.java
index 9284c78e43..406177cc70 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserEnumTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserEnumTest.java
@@ -4,15 +4,16 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Assert;
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
-public class ASTUserEnumTest extends ApexParserTestBase {
+import org.junit.jupiter.api.Test;
+
+class ASTUserEnumTest extends ApexParserTestBase {
@Test
- public void testEnumName() {
+ void testEnumName() {
ASTUserClass node = (ASTUserClass) parse("class Foo { enum Bar { } }");
ASTUserEnum enumNode = node.descendants(ASTUserEnum.class).firstOrThrow();
- Assert.assertEquals("Bar", enumNode.getSimpleName());
+ assertEquals("Bar", enumNode.getSimpleName());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserInterfaceTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserInterfaceTest.java
index 3d6eace318..1786022e8a 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserInterfaceTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserInterfaceTest.java
@@ -4,33 +4,34 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Assert;
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
-public class ASTUserInterfaceTest extends ApexParserTestBase {
+import org.junit.jupiter.api.Test;
+
+class ASTUserInterfaceTest extends ApexParserTestBase {
@Test
- public void testInterfaceName() {
+ void testInterfaceName() {
ASTUserInterface node = (ASTUserInterface) parse("interface Foo { }");
- Assert.assertEquals("Foo", node.getSimpleName());
+ assertEquals("Foo", node.getSimpleName());
}
@Test
- public void testInnerInterfaceName() {
+ void testInnerInterfaceName() {
ASTUserClass node = (ASTUserClass) parse("class Foo { interface Bar { } }");
ASTUserInterface innerNode = node.descendants(ASTUserInterface.class).firstOrThrow();
- Assert.assertEquals("Bar", innerNode.getSimpleName());
+ assertEquals("Bar", innerNode.getSimpleName());
}
@Test
- public void testSuperInterface() {
+ void testSuperInterface() {
ASTUserInterface toplevel = (ASTUserInterface) parse("public interface CustomInterface extends A {}");
- Assert.assertEquals("A", toplevel.getSuperInterfaceName());
+ assertEquals("A", toplevel.getSuperInterfaceName());
}
@Test
- public void testSuperInterface2() {
+ void testSuperInterface2() {
ASTUserInterface toplevel = (ASTUserInterface) parse("public interface CustomInterface extends Other.A {}");
- Assert.assertEquals("Other.A", toplevel.getSuperInterfaceName());
+ assertEquals("Other.A", toplevel.getSuperInterfaceName());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserTriggerTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserTriggerTest.java
index 728588cd61..0195fcb0d9 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserTriggerTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ASTUserTriggerTest.java
@@ -4,21 +4,23 @@
package net.sourceforge.pmd.lang.apex.ast;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertSame;
+
import java.util.Arrays;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
-public class ASTUserTriggerTest extends ApexParserTestBase {
+class ASTUserTriggerTest extends ApexParserTestBase {
@Test
- public void testTriggerName() {
+ void testTriggerName() {
ApexNode> node = parse("trigger HelloWorldTrigger on Book__c (before insert, after update) {\n"
+ " Book__c[] books = Trigger.new;\n" + " MyHelloWorld.applyDiscount(books);\n" + "}\n");
- Assert.assertSame(ASTUserTrigger.class, node.getClass());
- Assert.assertEquals("HelloWorldTrigger", node.getImage());
+ assertSame(ASTUserTrigger.class, node.getClass());
+ assertEquals("HelloWorldTrigger", node.getImage());
ASTUserTrigger trigger = (ASTUserTrigger) node;
- Assert.assertEquals("Book__c", trigger.getTargetName());
- Assert.assertEquals(Arrays.asList(TriggerUsage.AFTER_UPDATE, TriggerUsage.BEFORE_INSERT), trigger.getUsages());
+ assertEquals("Book__c", trigger.getTargetName());
+ assertEquals(Arrays.asList(TriggerUsage.AFTER_UPDATE, TriggerUsage.BEFORE_INSERT), trigger.getUsages());
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerSoqlTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerSoqlTest.java
index 32a4914fef..e585078d7f 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerSoqlTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerSoqlTest.java
@@ -4,9 +4,9 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
-public class ApexCompilerSoqlTest extends ApexParserTestBase {
+class ApexCompilerSoqlTest extends ApexParserTestBase {
private static final String CODE = "public class Foo {\n"
+ " public List test1() {\n"
@@ -15,7 +15,7 @@ public class ApexCompilerSoqlTest extends ApexParserTestBase {
+ "}\n";
@Test
- public void testSoqlCompilation() {
+ void testSoqlCompilation() {
apex.parse(CODE);
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerTest.java
index 320e82b234..107da7d444 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexCompilerTest.java
@@ -4,14 +4,16 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.lang.ast.ParseException;
-public class ApexCompilerTest extends ApexParserTestBase {
+class ApexCompilerTest extends ApexParserTestBase {
- @Test(expected = ParseException.class)
- public void compileShouldFail() {
- apex.parse("public class Foo { private String myField = \"a\"; }");
+ @Test
+ void compileShouldFail() {
+ assertThrows(ParseException.class, () -> apex.parse("public class Foo { private String myField = \"a\"; }"));
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexLexerTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexLexerTest.java
index df56adb9d5..79d6b0e17a 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexLexerTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexLexerTest.java
@@ -4,18 +4,20 @@
package net.sourceforge.pmd.lang.apex.ast;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+
import org.antlr.runtime.ANTLRStringStream;
import org.antlr.runtime.CharStream;
import org.antlr.runtime.CommonTokenStream;
import org.antlr.runtime.Token;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import apex.jorje.data.ast.CompilationUnit;
import apex.jorje.parser.impl.ApexLexer;
import apex.jorje.parser.impl.ApexParser;
-public class ApexLexerTest {
+class ApexLexerTest {
private static final String CODE = "public class Foo {\n"
+ " public List test1() {\n"
@@ -24,7 +26,7 @@ public class ApexLexerTest {
+ "}\n";
@Test
- public void testLexer() throws Exception {
+ void testLexer() throws Exception {
CharStream in = new ANTLRStringStream(CODE);
ApexLexer lexer = new ApexLexer(in);
@@ -34,15 +36,15 @@ public class ApexLexerTest {
tokenCount++;
token = lexer.nextToken();
}
- Assert.assertEquals(43, tokenCount);
+ assertEquals(43, tokenCount);
}
@Test
- public void testParser() throws Exception {
+ void testParser() throws Exception {
CharStream in = new ANTLRStringStream(CODE);
ApexLexer lexer = new ApexLexer(in);
ApexParser parser = new ApexParser(new CommonTokenStream(lexer));
CompilationUnit compilationUnit = parser.compilationUnit();
- Assert.assertNotNull(compilationUnit);
+ assertNotNull(compilationUnit);
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexParserTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexParserTest.java
index 3bf160b1a1..b2b7215234 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexParserTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexParserTest.java
@@ -7,24 +7,24 @@ package net.sourceforge.pmd.lang.apex.ast;
import static net.sourceforge.pmd.lang.ast.test.TestUtilsKt.assertPosition;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.core.IsInstanceOf.instanceOf;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
import java.io.File;
import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.util.List;
-import org.junit.Assert;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.lang.ast.Node;
import net.sourceforge.pmd.util.IOUtil;
-public class ApexParserTest extends ApexParserTestBase {
+class ApexParserTest extends ApexParserTestBase {
@Test
- public void understandsSimpleFile() {
+ void understandsSimpleFile() {
// Setup
String code = "@isTest\n"
@@ -43,7 +43,7 @@ public class ApexParserTest extends ApexParserTestBase {
}
@Test
- public void fileName() {
+ void fileName() {
String code = "class Outer { class Inner {}}";
ASTUserClass rootNode = (ASTUserClass) parse(code, "src/filename.cls");
@@ -60,13 +60,13 @@ public class ApexParserTest extends ApexParserTestBase {
+ "}\n"; // line 6
@Test
- public void verifyLineColumnNumbers() {
+ void verifyLineColumnNumbers() {
ASTUserClassOrInterface> rootNode = parse(testCodeForLineNumbers);
assertLineNumbersForTestCode(rootNode);
}
@Test
- public void verifyLineColumnNumbersWithWindowsLineEndings() {
+ void verifyLineColumnNumbersWithWindowsLineEndings() {
String windowsLineEndings = testCodeForLineNumbers.replaceAll(" \n", "\r\n");
ASTUserClassOrInterface> rootNode = parse(windowsLineEndings);
assertLineNumbersForTestCode(rootNode);
@@ -100,7 +100,7 @@ public class ApexParserTest extends ApexParserTestBase {
}
@Test
- public void verifyEndLine() {
+ void verifyEndLine() {
String code = "public class SimpleClass {\n" // line 1
+ " public void method1() {\n" // line 2
@@ -112,16 +112,16 @@ public class ApexParserTest extends ApexParserTestBase {
ASTUserClassOrInterface> rootNode = parse(code);
Node method1 = rootNode.getChild(1);
- assertEquals("Wrong begin line", 2, method1.getBeginLine());
- assertEquals("Wrong end line", 3, method1.getEndLine());
+ assertEquals(2, method1.getBeginLine(), "Wrong begin line");
+ assertEquals(3, method1.getEndLine(), "Wrong end line");
Node method2 = rootNode.getChild(2);
- assertEquals("Wrong begin line", 4, method2.getBeginLine());
- assertEquals("Wrong end line", 5, method2.getEndLine());
+ assertEquals(4, method2.getBeginLine(), "Wrong begin line");
+ assertEquals(5, method2.getEndLine(), "Wrong end line");
}
@Test
- public void checkComments() {
+ void checkComments() {
String code = "public /** Comment on Class */ class SimpleClass {\n" // line 1
+ " /** Comment on m1 */"
@@ -149,14 +149,14 @@ public class ApexParserTest extends ApexParserTestBase {
}
@Test
- public void parsesRealWorldClasses() throws Exception {
+ void parsesRealWorldClasses() throws Exception {
File directory = new File("src/test/resources");
File[] fList = directory.listFiles();
for (File file : fList) {
if (file.isFile() && file.getName().endsWith(".cls")) {
String sourceCode = IOUtil.readFileToString(file, StandardCharsets.UTF_8);
- Assert.assertNotNull(parse(sourceCode));
+ assertNotNull(parse(sourceCode));
}
}
}
@@ -166,7 +166,7 @@ public class ApexParserTest extends ApexParserTestBase {
* @see [apex] PMD parsing exception for Apex classes using 'inherited sharing' keyword
*/
@Test
- public void parseInheritedSharingClass() throws IOException {
+ void parseInheritedSharingClass() throws IOException {
String source = IOUtil.readToString(ApexParserTest.class.getResourceAsStream("InheritedSharing.cls"),
StandardCharsets.UTF_8);
parse(source);
@@ -178,17 +178,17 @@ public class ApexParserTest extends ApexParserTestBase {
* @see #1485 [apex] Analysis of some apex classes cause a stackoverflow error
*/
@Test
- public void stackOverflowDuringClassParsing() throws Exception {
+ void stackOverflowDuringClassParsing() throws Exception {
String source = IOUtil.readToString(ApexParserTest.class.getResourceAsStream("StackOverflowClass.cls"),
StandardCharsets.UTF_8);
ASTUserClassOrInterface> rootNode = parse(source);
int count = visitPosition(rootNode, 0);
- Assert.assertEquals(487, count);
+ assertEquals(487, count);
}
@Test
- public void verifyLineColumnNumbersInnerClasses() throws Exception {
+ void verifyLineColumnNumbersInnerClasses() throws Exception {
String source = IOUtil.readToString(ApexParserTest.class.getResourceAsStream("InnerClassLocations.cls"),
StandardCharsets.UTF_8);
source = source.replaceAll("\r\n", "\n");
@@ -196,20 +196,20 @@ public class ApexParserTest extends ApexParserTestBase {
visitPosition(rootNode, 0);
- Assert.assertEquals("InnerClassLocations", rootNode.getSimpleName());
+ assertEquals("InnerClassLocations", rootNode.getSimpleName());
// Note: Apex parser doesn't provide positions for "public class" keywords. The
// position of the UserClass node is just the identifier. So, the node starts
// with the identifier and not with the first keyword in the file...
assertPosition(rootNode, 1, 14, 16, 2);
List classes = rootNode.descendants(ASTUserClass.class).toList();
- Assert.assertEquals(2, classes.size());
- Assert.assertEquals("bar1", classes.get(0).getSimpleName());
+ assertEquals(2, classes.size());
+ assertEquals("bar1", classes.get(0).getSimpleName());
List methods = classes.get(0).findChildrenOfType(ASTMethod.class);
- Assert.assertEquals(2, methods.size()); // m() and synthetic clone()
- Assert.assertEquals("m", methods.get(0).getImage());
+ assertEquals(2, methods.size()); // m() and synthetic clone()
+ assertEquals("m", methods.get(0).getImage());
assertPosition(methods.get(0), 4, 21, 7, 9);
- Assert.assertEquals("clone", methods.get(1).getImage());
+ assertEquals("clone", methods.get(1).getImage());
assertPosition(methods.get(1), 7, 9, 7, 9);
// Position of the first inner class: starts with the identifier "bar1" and ends with
@@ -217,7 +217,7 @@ public class ApexParserTest extends ApexParserTestBase {
// don't see this in the AST.
assertPosition(classes.get(0), 3, 18, 7, 9);
- Assert.assertEquals("bar2", classes.get(1).getImage());
+ assertEquals("bar2", classes.get(1).getImage());
assertPosition(classes.get(1), 10, 18, 14, 9);
}
@@ -225,10 +225,10 @@ public class ApexParserTest extends ApexParserTestBase {
private int visitPosition(Node node, int count) {
int result = count + 1;
- Assert.assertTrue(node.getBeginLine() > 0);
- Assert.assertTrue(node.getBeginColumn() > 0);
- Assert.assertTrue(node.getEndLine() > 0);
- Assert.assertTrue(node.getEndColumn() > 0);
+ assertTrue(node.getBeginLine() > 0);
+ assertTrue(node.getBeginColumn() > 0);
+ assertTrue(node.getEndLine() > 0);
+ assertTrue(node.getEndColumn() > 0);
for (int i = 0; i < node.getNumChildren(); i++) {
result = visitPosition(node.getChild(i), result);
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexQualifiedNameTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexQualifiedNameTest.java
index 4384b4cca2..3c5ccd5f6a 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexQualifiedNameTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexQualifiedNameTest.java
@@ -4,21 +4,20 @@
package net.sourceforge.pmd.lang.apex.ast;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotEquals;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-
-import org.junit.Test;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
+import org.junit.jupiter.api.Test;
/**
* @author Clément Fournier
*/
-public class ApexQualifiedNameTest extends ApexParserTestBase {
+class ApexQualifiedNameTest extends ApexParserTestBase {
@Test
- public void testClass() {
+ void testClass() {
ASTUserClass root = (ASTUserClass) parse("public class Foo {}");
ApexQualifiedName qname = root.getQualifiedName();
@@ -30,7 +29,7 @@ public class ApexQualifiedNameTest extends ApexParserTestBase {
@Test
- public void testNestedClass() {
+ void testNestedClass() {
ASTUserClass root = (ASTUserClass) parse("public class Foo { class Bar {}}");
ASTUserClass inner = root.descendants(ASTUserClass.class).firstOrThrow();
@@ -43,7 +42,7 @@ public class ApexQualifiedNameTest extends ApexParserTestBase {
@Test
- public void testSimpleMethod() {
+ void testSimpleMethod() {
ASTUserClass root = (ASTUserClass) parse("public class Foo { String foo() {}}");
ApexQualifiedName qname = root.descendants(ASTMethod.class).firstOrThrow().getQualifiedName();
assertEquals("c__Foo#foo()", qname.toString());
@@ -54,7 +53,7 @@ public class ApexQualifiedNameTest extends ApexParserTestBase {
@Test
- public void testMethodWithArguments() {
+ void testMethodWithArguments() {
ASTUserClass root = (ASTUserClass) parse("public class Foo { String foo(String h, Foo g) {}}");
ApexQualifiedName qname = root.descendants(ASTMethod.class).firstOrThrow().getQualifiedName();
assertEquals("c__Foo#foo(String, Foo)", qname.toString());
@@ -65,7 +64,7 @@ public class ApexQualifiedNameTest extends ApexParserTestBase {
@Test
- public void testOverLoads() {
+ void testOverLoads() {
ASTUserClass root = (ASTUserClass) parse("public class Foo { "
+ "String foo(String h) {} "
+ "String foo(int c) {}"
@@ -82,7 +81,7 @@ public class ApexQualifiedNameTest extends ApexParserTestBase {
@Test
- public void testTrigger() {
+ void testTrigger() {
ASTUserTrigger root = (ASTUserTrigger) parse("trigger myAccountTrigger on Account (before insert, before update) {}");
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeDumpTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeDumpTest.java
index 6ce49d352e..13f8b60c68 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeDumpTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/ast/ApexTreeDumpTest.java
@@ -5,15 +5,15 @@
package net.sourceforge.pmd.lang.apex.ast;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.lang.ast.test.BaseParsingHelper;
import net.sourceforge.pmd.lang.ast.test.BaseTreeDumpTest;
import net.sourceforge.pmd.lang.ast.test.RelevantAttributePrinter;
-public class ApexTreeDumpTest extends BaseTreeDumpTest {
+class ApexTreeDumpTest extends BaseTreeDumpTest {
- public ApexTreeDumpTest() {
+ ApexTreeDumpTest() {
super(new RelevantAttributePrinter(), ".cls");
}
@@ -23,7 +23,7 @@ public class ApexTreeDumpTest extends BaseTreeDumpTest {
}
@Test
- public void safeNavigationOperator() {
+ void safeNavigationOperator() {
doTest("SafeNavigationOperator");
}
}
From cb70ab5268aed4e683db9f5dbccc90204a542db7 Mon Sep 17 00:00:00 2001
From: Andreas Dangel
Date: Fri, 1 Jul 2022 10:11:58 +0200
Subject: [PATCH 02/31] [apex] Migrate tests to JUnit5
---
pmd-apex/pom.xml | 2 +-
.../pmd/lang/apex/DefaultRulesetTest.java | 28 +++++----
.../pmd/lang/apex/SuppressWarningsTest.java | 36 +++++------
.../apex/metrics/impl/AllMetricsTest.java | 2 +-
.../multifile/ApexMultifileAnalysisTest.java | 61 ++++++++++---------
.../lang/apex/rule/AbstractApexRuleTest.java | 12 ++--
.../pmd/lang/apex/rule/ApexXPathRuleTest.java | 8 +--
...pexAssertionsShouldIncludeMessageTest.java | 2 +-
...pexUnitTestClassShouldHaveAssertsTest.java | 2 +-
...tMethodShouldHaveIsTestAnnotationTest.java | 2 +-
...nitTestShouldNotUseSeeAllDataTrueTest.java | 2 +-
.../AvoidGlobalModifierTest.java | 2 +-
.../AvoidLogicInTriggerTest.java | 2 +-
.../DebugsShouldUseLoggingLevelTest.java | 2 +-
.../UnusedLocalVariableTest.java | 2 +-
.../codestyle/ClassNamingConventionsTest.java | 2 +-
.../FieldDeclarationsShouldBeAtStartTest.java | 2 +-
.../codestyle/FieldNamingConventionsTest.java | 2 +-
.../codestyle/ForLoopsMustUseBracesTest.java | 2 +-
.../FormalParameterNamingConventionsTest.java | 2 +-
.../IfElseStmtsMustUseBracesTest.java | 2 +-
.../codestyle/IfStmtsMustUseBracesTest.java | 2 +-
.../LocalVariableNamingConventionsTest.java | 2 +-
.../MethodNamingConventionsTest.java | 2 +-
.../codestyle/OneDeclarationPerLineTest.java | 2 +-
.../PropertyNamingConventionsTest.java | 2 +-
.../WhileLoopsMustUseBracesTest.java | 2 +-
.../design/AvoidDeeplyNestedIfStmtsTest.java | 2 +-
.../rule/design/CognitiveComplexityTest.java | 3 +-
.../rule/design/CyclomaticComplexityTest.java | 3 +-
.../rule/design/ExcessiveClassLengthTest.java | 2 +-
.../design/ExcessiveParameterListTest.java | 2 +-
.../rule/design/ExcessivePublicCountTest.java | 2 +-
.../rule/design/NcssConstructorCountTest.java | 2 +-
.../apex/rule/design/NcssMethodCountTest.java | 2 +-
.../apex/rule/design/NcssTypeCountTest.java | 2 +-
.../apex/rule/design/TooManyFieldsTest.java | 2 +-
.../apex/rule/documentation/ApexDocTest.java | 2 +-
.../apex/rule/errorprone/ApexCSRFTest.java | 2 +-
.../AvoidDirectAccessTriggerMapTest.java | 2 +-
.../errorprone/AvoidHardcodingIdTest.java | 2 +-
.../AvoidNonExistentAnnotationsTest.java | 2 +-
.../rule/errorprone/EmptyCatchBlockTest.java | 2 +-
.../apex/rule/errorprone/EmptyIfStmtTest.java | 2 +-
.../errorprone/EmptyStatementBlockTest.java | 2 +-
.../EmptyTryOrFinallyBlockTest.java | 2 +-
.../rule/errorprone/EmptyWhileStmtTest.java | 2 +-
.../InaccessibleAuraEnabledGetterTest.java | 2 +-
...ethodWithSameNameAsEnclosingClassTest.java | 2 +-
.../OverrideBothEqualsAndHashcodeTest.java | 2 +-
.../TestMethodsMustBeInTestClassesTest.java | 2 +-
.../performance/AvoidDebugStatementsTest.java | 2 +-
.../AvoidDmlStatementsInLoopsTest.java | 2 +-
.../performance/AvoidSoqlInLoopsTest.java | 2 +-
.../performance/AvoidSoslInLoopsTest.java | 2 +-
...agerlyLoadedDescribeSObjectResultTest.java | 2 +-
.../OperationWithLimitsInLoopTest.java | 2 +-
.../apex/rule/security/ApexBadCryptoTest.java | 2 +-
.../rule/security/ApexCRUDViolationTest.java | 2 +-
.../security/ApexDangerousMethodsTest.java | 2 +-
.../security/ApexInsecureEndpointTest.java | 2 +-
.../rule/security/ApexOpenRedirectTest.java | 2 +-
.../rule/security/ApexSOQLInjectionTest.java | 2 +-
.../ApexSharingViolationsNestedClassTest.java | 46 ++++----------
.../security/ApexSharingViolationsTest.java | 2 +-
.../ApexSuggestUsingNamedCredTest.java | 2 +-
.../security/ApexXSSFromEscapeFalseTest.java | 2 +-
.../security/ApexXSSFromURLParamTest.java | 2 +-
68 files changed, 151 insertions(+), 166 deletions(-)
diff --git a/pmd-apex/pom.xml b/pmd-apex/pom.xml
index 08f0ea8b2a..8e607d3441 100644
--- a/pmd-apex/pom.xml
+++ b/pmd-apex/pom.xml
@@ -84,7 +84,7 @@
com.github.stefanbirkner
- system-rules
+ system-lambda
test
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/DefaultRulesetTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/DefaultRulesetTest.java
index d59ebf19b8..9f8dd289b3 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/DefaultRulesetTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/DefaultRulesetTest.java
@@ -4,29 +4,31 @@
package net.sourceforge.pmd.lang.apex;
-import org.junit.Assert;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.contrib.java.lang.system.SystemErrRule;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.RuleSet;
import net.sourceforge.pmd.RuleSetLoader;
-public class DefaultRulesetTest {
- @Rule
- public final SystemErrRule systemErrRule = new SystemErrRule().enableLog().muteForSuccessfulTests();
+import com.github.stefanbirkner.systemlambda.SystemLambda;
+
+class DefaultRulesetTest {
@Test
- public void loadDefaultRuleset() {
+ void loadDefaultRuleset() {
RuleSet ruleset = rulesetLoader().loadFromResource("rulesets/apex/ruleset.xml");
- Assert.assertNotNull(ruleset);
+ assertNotNull(ruleset);
}
@Test
- public void loadQuickstartRuleset() {
- RuleSet ruleset = rulesetLoader().loadFromResource("rulesets/apex/quickstart.xml");
- Assert.assertNotNull(ruleset);
- Assert.assertTrue("No Logging expected", systemErrRule.getLog().isEmpty());
+ void loadQuickstartRuleset() throws Exception {
+ String log = SystemLambda.tapSystemErr(() -> {
+ RuleSet ruleset = rulesetLoader().loadFromResource("rulesets/apex/quickstart.xml");
+ assertNotNull(ruleset);
+ });
+ assertTrue(log.isEmpty(), "No Logging expected");
}
private RuleSetLoader rulesetLoader() {
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/SuppressWarningsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/SuppressWarningsTest.java
index abe3d1cad0..17c5eebf7d 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/SuppressWarningsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/SuppressWarningsTest.java
@@ -6,11 +6,11 @@ package net.sourceforge.pmd.lang.apex;
import static net.sourceforge.pmd.lang.ast.test.TestUtilsKt.assertSize;
import static net.sourceforge.pmd.lang.ast.test.TestUtilsKt.assertSuppressed;
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
import java.util.List;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.PMD;
import net.sourceforge.pmd.Report;
@@ -19,7 +19,7 @@ import net.sourceforge.pmd.lang.apex.ast.ASTUserClass;
import net.sourceforge.pmd.lang.apex.ast.ApexParserTestBase;
import net.sourceforge.pmd.lang.apex.rule.AbstractApexRule;
-public class SuppressWarningsTest extends ApexParserTestBase {
+class SuppressWarningsTest extends ApexParserTestBase {
// This could be a regular xml test
@@ -45,7 +45,7 @@ public class SuppressWarningsTest extends ApexParserTestBase {
}
@Test
- public void testClassLevelSuppression() {
+ void testClassLevelSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST1);
assertSize(rpt, 0);
rpt = apex.executeRule(new FooRule(), TEST2);
@@ -53,86 +53,86 @@ public class SuppressWarningsTest extends ApexParserTestBase {
}
@Test
- public void testInheritedSuppression() {
+ void testInheritedSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST3);
assertSize(rpt, 0);
}
@Test
- public void testMethodLevelSuppression() {
+ void testMethodLevelSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST4);
assertSize(rpt, 1);
}
@Test
- public void testConstructorLevelSuppression() {
+ void testConstructorLevelSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST5);
assertSize(rpt, 0);
}
@Test
- public void testFieldLevelSuppression() {
+ void testFieldLevelSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST6);
assertSize(rpt, 1);
}
@Test
- public void testParameterLevelSuppression() {
+ void testParameterLevelSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST7);
assertSize(rpt, 1);
}
@Test
- public void testLocalVariableLevelSuppression() {
+ void testLocalVariableLevelSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST8);
assertSize(rpt, 1);
}
@Test
- public void testSpecificSuppression() {
+ void testSpecificSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST9);
assertSize(rpt, 1);
}
@Test
- public void testSpecificSuppressionMulitpleValues() {
+ void testSpecificSuppressionMulitpleValues() {
Report rpt = apex.executeRule(new FooRule(), TEST9_MULTIPLE_VALUES);
assertSize(rpt, 0);
}
@Test
- public void testNoSuppressionBlank() {
+ void testNoSuppressionBlank() {
Report rpt = apex.executeRule(new FooRule(), TEST10);
assertSize(rpt, 2);
}
@Test
- public void testNoSuppressionSomethingElseS() {
+ void testNoSuppressionSomethingElseS() {
Report rpt = apex.executeRule(new FooRule(), TEST11);
assertSize(rpt, 2);
}
@Test
- public void testSuppressAll() {
+ void testSuppressAll() {
Report rpt = apex.executeRule(new FooRule(), TEST12);
assertSize(rpt, 0);
}
@Test
- public void testSpecificSuppressionAtTopLevel() {
+ void testSpecificSuppressionAtTopLevel() {
Report rpt = apex.executeRule(new BarRule(), TEST13);
assertSize(rpt, 0);
}
@Test
- public void testCommentSuppression() {
+ void testCommentSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST14);
assertSize(rpt, 0);
assertSuppressed(rpt, 1);
}
@Test
- public void testMessageWithCommentSuppression() {
+ void testMessageWithCommentSuppression() {
Report rpt = apex.executeRule(new FooRule(), TEST15);
assertSize(rpt, 0);
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/metrics/impl/AllMetricsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/metrics/impl/AllMetricsTest.java
index f0b8d664cc..2936a9d81a 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/metrics/impl/AllMetricsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/metrics/impl/AllMetricsTest.java
@@ -13,7 +13,7 @@ import net.sourceforge.pmd.testframework.SimpleAggregatorTst;
*
* @author Clément Fournier
*/
-public class AllMetricsTest extends SimpleAggregatorTst {
+class AllMetricsTest extends SimpleAggregatorTst {
private static final String RULESET = "rulesets/apex/metrics_test.xml";
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/multifile/ApexMultifileAnalysisTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/multifile/ApexMultifileAnalysisTest.java
index 7f3effa2b7..b2b54727e6 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/multifile/ApexMultifileAnalysisTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/multifile/ApexMultifileAnalysisTest.java
@@ -6,70 +6,73 @@ package net.sourceforge.pmd.lang.apex.multifile;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.containsStringIgnoringCase;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
-import java.io.File;
import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
+import java.nio.file.Path;
import java.util.Arrays;
import org.checkerframework.checker.nullness.qual.NonNull;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.contrib.java.lang.system.SystemErrRule;
-import org.junit.rules.TemporaryFolder;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.io.TempDir;
import net.sourceforge.pmd.util.IOUtil;
+import com.github.stefanbirkner.systemlambda.SystemLambda;
+
public class ApexMultifileAnalysisTest {
- @Rule
- public final SystemErrRule systemErrRule = new SystemErrRule().muteForSuccessfulTests().enableLog();
-
- @Rule
- public final TemporaryFolder tempFolder = new TemporaryFolder();
+ @TempDir
+ private Path tempFolder;
@Test
- public void testNoSfdxProjectJsonProducesFailedAnalysis() {
- ApexMultifileAnalysis analysisInstance = getAnalysisForTempFolder();
+ void testNoSfdxProjectJsonProducesFailedAnalysis() throws Exception {
+ String log = SystemLambda.tapSystemErr(() -> {
+ ApexMultifileAnalysis analysisInstance = getAnalysisForTempFolder();
- assertTrue(analysisInstance.isFailed());
- assertTrue(analysisInstance.getFileIssues("any file").isEmpty());
- assertThat(systemErrRule.getLog(), containsStringIgnoringCase("Missing project file"));
+ assertTrue(analysisInstance.isFailed());
+ assertTrue(analysisInstance.getFileIssues("any file").isEmpty());
+ });
+ assertThat(log, containsStringIgnoringCase("Missing project file"));
}
@Test
- public void testMalformedSfdxProjectJsonProducesFailedAnalysis() throws IOException {
+ void testMalformedSfdxProjectJsonProducesFailedAnalysis() throws Exception {
copyResource("malformedSfdxFile.json", "sfdx-project.json");
- ApexMultifileAnalysis analysisInstance = getAnalysisForTempFolder();
+ String log = SystemLambda.tapSystemErr(() -> {
+ ApexMultifileAnalysis analysisInstance = getAnalysisForTempFolder();
- assertTrue(analysisInstance.isFailed());
- assertTrue(analysisInstance.getFileIssues("any file").isEmpty());
- assertThat(systemErrRule.getLog(),
+ assertTrue(analysisInstance.isFailed());
+ assertTrue(analysisInstance.getFileIssues("any file").isEmpty());
+ });
+ assertThat(log,
containsStringIgnoringCase("error: 'path' is required for all 'packageDirectories' elements"));
}
@Test
- public void testWellFormedSfdxProjectJsonProducesFunctionalAnalysis() throws IOException {
+ void testWellFormedSfdxProjectJsonProducesFunctionalAnalysis() throws Exception {
copyResource("correctSfdxFile.json", "sfdx-project.json");
- ApexMultifileAnalysis analysisInstance = getAnalysisForTempFolder();
+ String log = SystemLambda.tapSystemErr(() -> {
+ ApexMultifileAnalysis analysisInstance = getAnalysisForTempFolder();
- assertFalse(analysisInstance.isFailed());
- assertTrue(systemErrRule.getLog().isEmpty());
+ assertFalse(analysisInstance.isFailed());
+ });
+ assertTrue(log.isEmpty());
}
private @NonNull ApexMultifileAnalysis getAnalysisForTempFolder() {
- return ApexMultifileAnalysis.getAnalysisInstance(tempFolder.getRoot().getAbsolutePath());
+ return ApexMultifileAnalysis.getAnalysisInstance(tempFolder.toAbsolutePath().toString());
}
private void copyResource(String resourcePath, String relativePathInTempDir) throws IOException {
- File file = tempFolder.newFile(relativePathInTempDir);
+ Path file = tempFolder.resolve(relativePathInTempDir);
String fileContents = IOUtil.readToString(getClass().getResourceAsStream(resourcePath), StandardCharsets.UTF_8);
- Files.write(file.toPath(), Arrays.asList(fileContents.split("\\R").clone()));
+ Files.write(file, Arrays.asList(fileContents.split("\\R").clone()));
}
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/AbstractApexRuleTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/AbstractApexRuleTest.java
index bbc3a29a1a..cf2f2c3d6b 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/AbstractApexRuleTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/AbstractApexRuleTest.java
@@ -4,7 +4,7 @@
package net.sourceforge.pmd.lang.apex.rule;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.Report;
import net.sourceforge.pmd.lang.apex.ast.ASTAnonymousClass;
@@ -15,25 +15,25 @@ import net.sourceforge.pmd.lang.apex.ast.ASTUserTrigger;
import net.sourceforge.pmd.lang.apex.ast.ApexParserTestBase;
import net.sourceforge.pmd.lang.ast.test.TestUtilsKt;
-public class AbstractApexRuleTest extends ApexParserTestBase {
+class AbstractApexRuleTest extends ApexParserTestBase {
@Test
- public void shouldVisitTopLevelClass() {
+ void shouldVisitTopLevelClass() {
run("class Foo { }");
}
@Test
- public void shouldVisitTopLevelInterface() {
+ void shouldVisitTopLevelInterface() {
run("interface Foo { }");
}
@Test
- public void shouldVisitTopLevelTrigger() {
+ void shouldVisitTopLevelTrigger() {
run("trigger Foo on Account (before insert, before update) { }");
}
@Test
- public void shouldVisitTopLevelEnum() {
+ void shouldVisitTopLevelEnum() {
run("enum Foo { }");
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/ApexXPathRuleTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/ApexXPathRuleTest.java
index b90d4a26ce..a0286ca0f7 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/ApexXPathRuleTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/ApexXPathRuleTest.java
@@ -6,7 +6,7 @@ package net.sourceforge.pmd.lang.apex.rule;
import static net.sourceforge.pmd.lang.ast.test.TestUtilsKt.assertSize;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import net.sourceforge.pmd.Report;
import net.sourceforge.pmd.lang.LanguageRegistry;
@@ -18,7 +18,7 @@ import net.sourceforge.pmd.lang.rule.xpath.XPathVersion;
/**
* @author daniels
*/
-public class ApexXPathRuleTest extends ApexParserTestBase {
+class ApexXPathRuleTest extends ApexParserTestBase {
private XPathRule makeXPath(String expression) {
XPathRule rule = new XPathRule(XPathVersion.XPATH_2_0, expression);
@@ -29,7 +29,7 @@ public class ApexXPathRuleTest extends ApexParserTestBase {
@Test
- public void testFileNameInXpath() {
+ void testFileNameInXpath() {
Report report = apex.executeRule(makeXPath("/UserClass[pmd:fileName() = 'Foo.cls']"),
"class Foo {}",
"src/Foo.cls");
@@ -38,7 +38,7 @@ public class ApexXPathRuleTest extends ApexParserTestBase {
}
@Test
- public void testBooleanExpressions() {
+ void testBooleanExpressions() {
Report report = apex.executeRuleOnResource(makeXPath("//BooleanExpression[@Operator='&&']"),
"BooleanExpressions.cls");
assertSize(report, 1);
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexAssertionsShouldIncludeMessageTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexAssertionsShouldIncludeMessageTest.java
index 8b9464e9e9..46ab3aff19 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexAssertionsShouldIncludeMessageTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexAssertionsShouldIncludeMessageTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexAssertionsShouldIncludeMessageTest extends PmdRuleTst {
+class ApexAssertionsShouldIncludeMessageTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestClassShouldHaveAssertsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestClassShouldHaveAssertsTest.java
index 5c3d68226e..8da6170cb0 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestClassShouldHaveAssertsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestClassShouldHaveAssertsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexUnitTestClassShouldHaveAssertsTest extends PmdRuleTst {
+class ApexUnitTestClassShouldHaveAssertsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestMethodShouldHaveIsTestAnnotationTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestMethodShouldHaveIsTestAnnotationTest.java
index b753e8fb8a..983448dce7 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestMethodShouldHaveIsTestAnnotationTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestMethodShouldHaveIsTestAnnotationTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexUnitTestMethodShouldHaveIsTestAnnotationTest extends PmdRuleTst {
+class ApexUnitTestMethodShouldHaveIsTestAnnotationTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestShouldNotUseSeeAllDataTrueTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestShouldNotUseSeeAllDataTrueTest.java
index 00084bc52e..150bcfecd6 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestShouldNotUseSeeAllDataTrueTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/ApexUnitTestShouldNotUseSeeAllDataTrueTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexUnitTestShouldNotUseSeeAllDataTrueTest extends PmdRuleTst {
+class ApexUnitTestShouldNotUseSeeAllDataTrueTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidGlobalModifierTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidGlobalModifierTest.java
index 6a1780bfc9..eff39e8739 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidGlobalModifierTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidGlobalModifierTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidGlobalModifierTest extends PmdRuleTst {
+class AvoidGlobalModifierTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidLogicInTriggerTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidLogicInTriggerTest.java
index 08cfa53ada..21cd2021d9 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidLogicInTriggerTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/AvoidLogicInTriggerTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidLogicInTriggerTest extends PmdRuleTst {
+class AvoidLogicInTriggerTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/DebugsShouldUseLoggingLevelTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/DebugsShouldUseLoggingLevelTest.java
index 4632c9a051..939ad1b510 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/DebugsShouldUseLoggingLevelTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/DebugsShouldUseLoggingLevelTest.java
@@ -7,6 +7,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class DebugsShouldUseLoggingLevelTest extends PmdRuleTst {
+class DebugsShouldUseLoggingLevelTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/UnusedLocalVariableTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/UnusedLocalVariableTest.java
index cef1960afc..fe40d55ecc 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/UnusedLocalVariableTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/bestpractices/UnusedLocalVariableTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.bestpractices;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class UnusedLocalVariableTest extends PmdRuleTst {
+class UnusedLocalVariableTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ClassNamingConventionsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ClassNamingConventionsTest.java
index 6e44c82f66..0ebe54c2a2 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ClassNamingConventionsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ClassNamingConventionsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ClassNamingConventionsTest extends PmdRuleTst {
+class ClassNamingConventionsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldDeclarationsShouldBeAtStartTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldDeclarationsShouldBeAtStartTest.java
index c22140d52d..3a3224f1dc 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldDeclarationsShouldBeAtStartTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldDeclarationsShouldBeAtStartTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class FieldDeclarationsShouldBeAtStartTest extends PmdRuleTst {
+class FieldDeclarationsShouldBeAtStartTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldNamingConventionsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldNamingConventionsTest.java
index 823a7f32fc..cc04dbc5ca 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldNamingConventionsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FieldNamingConventionsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class FieldNamingConventionsTest extends PmdRuleTst {
+class FieldNamingConventionsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ForLoopsMustUseBracesTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ForLoopsMustUseBracesTest.java
index 32e64fa7c8..22891df41c 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ForLoopsMustUseBracesTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/ForLoopsMustUseBracesTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ForLoopsMustUseBracesTest extends PmdRuleTst {
+class ForLoopsMustUseBracesTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FormalParameterNamingConventionsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FormalParameterNamingConventionsTest.java
index 45dba2964e..79561ea4a8 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FormalParameterNamingConventionsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/FormalParameterNamingConventionsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class FormalParameterNamingConventionsTest extends PmdRuleTst {
+class FormalParameterNamingConventionsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfElseStmtsMustUseBracesTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfElseStmtsMustUseBracesTest.java
index 8c118c38d7..a5b04aaf90 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfElseStmtsMustUseBracesTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfElseStmtsMustUseBracesTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class IfElseStmtsMustUseBracesTest extends PmdRuleTst {
+class IfElseStmtsMustUseBracesTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfStmtsMustUseBracesTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfStmtsMustUseBracesTest.java
index 049d251fbd..8d91dd649b 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfStmtsMustUseBracesTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/IfStmtsMustUseBracesTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class IfStmtsMustUseBracesTest extends PmdRuleTst {
+class IfStmtsMustUseBracesTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/LocalVariableNamingConventionsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/LocalVariableNamingConventionsTest.java
index 6ddf91a354..0915626fce 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/LocalVariableNamingConventionsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/LocalVariableNamingConventionsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class LocalVariableNamingConventionsTest extends PmdRuleTst {
+class LocalVariableNamingConventionsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/MethodNamingConventionsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/MethodNamingConventionsTest.java
index 2be02e5acb..291fa815fc 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/MethodNamingConventionsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/MethodNamingConventionsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class MethodNamingConventionsTest extends PmdRuleTst {
+class MethodNamingConventionsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/OneDeclarationPerLineTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/OneDeclarationPerLineTest.java
index fa97125829..37fbf81509 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/OneDeclarationPerLineTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/OneDeclarationPerLineTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class OneDeclarationPerLineTest extends PmdRuleTst {
+class OneDeclarationPerLineTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/PropertyNamingConventionsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/PropertyNamingConventionsTest.java
index 413e80ed09..6e78c8d37f 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/PropertyNamingConventionsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/PropertyNamingConventionsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class PropertyNamingConventionsTest extends PmdRuleTst {
+class PropertyNamingConventionsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/WhileLoopsMustUseBracesTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/WhileLoopsMustUseBracesTest.java
index e5a005e726..03b7109932 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/WhileLoopsMustUseBracesTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/codestyle/WhileLoopsMustUseBracesTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.codestyle;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class WhileLoopsMustUseBracesTest extends PmdRuleTst {
+class WhileLoopsMustUseBracesTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/AvoidDeeplyNestedIfStmtsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/AvoidDeeplyNestedIfStmtsTest.java
index abf3e78e0c..5002fe75d2 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/AvoidDeeplyNestedIfStmtsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/AvoidDeeplyNestedIfStmtsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidDeeplyNestedIfStmtsTest extends PmdRuleTst {
+class AvoidDeeplyNestedIfStmtsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CognitiveComplexityTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CognitiveComplexityTest.java
index 3258d92160..07ece5a9a3 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CognitiveComplexityTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CognitiveComplexityTest.java
@@ -6,5 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class CognitiveComplexityTest extends PmdRuleTst {
+class CognitiveComplexityTest extends PmdRuleTst {
+ // no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CyclomaticComplexityTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CyclomaticComplexityTest.java
index 50ffb556ea..e0c5c3696d 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CyclomaticComplexityTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/CyclomaticComplexityTest.java
@@ -6,5 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class CyclomaticComplexityTest extends PmdRuleTst {
+class CyclomaticComplexityTest extends PmdRuleTst {
+ // no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveClassLengthTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveClassLengthTest.java
index 5cde5c180c..99b4becd66 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveClassLengthTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveClassLengthTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ExcessiveClassLengthTest extends PmdRuleTst {
+class ExcessiveClassLengthTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveParameterListTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveParameterListTest.java
index d2263e88b6..78bb4a0c43 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveParameterListTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessiveParameterListTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ExcessiveParameterListTest extends PmdRuleTst {
+class ExcessiveParameterListTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessivePublicCountTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessivePublicCountTest.java
index 20a52344b1..256d0f3847 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessivePublicCountTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/ExcessivePublicCountTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ExcessivePublicCountTest extends PmdRuleTst {
+class ExcessivePublicCountTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssConstructorCountTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssConstructorCountTest.java
index ec837f8518..100985cb5c 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssConstructorCountTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssConstructorCountTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class NcssConstructorCountTest extends PmdRuleTst {
+class NcssConstructorCountTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssMethodCountTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssMethodCountTest.java
index 8f9b35c018..af10fcdbfd 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssMethodCountTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssMethodCountTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class NcssMethodCountTest extends PmdRuleTst {
+class NcssMethodCountTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssTypeCountTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssTypeCountTest.java
index 3416a142c9..92ec3a77be 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssTypeCountTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/NcssTypeCountTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class NcssTypeCountTest extends PmdRuleTst {
+class NcssTypeCountTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/TooManyFieldsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/TooManyFieldsTest.java
index 71becae7c6..cdd02ad007 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/TooManyFieldsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/design/TooManyFieldsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.design;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class TooManyFieldsTest extends PmdRuleTst {
+class TooManyFieldsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/documentation/ApexDocTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/documentation/ApexDocTest.java
index 93273a11ad..080b396560 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/documentation/ApexDocTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/documentation/ApexDocTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.documentation;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexDocTest extends PmdRuleTst {
+class ApexDocTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/ApexCSRFTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/ApexCSRFTest.java
index 5549abc726..85f7f104e0 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/ApexCSRFTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/ApexCSRFTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexCSRFTest extends PmdRuleTst {
+class ApexCSRFTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidDirectAccessTriggerMapTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidDirectAccessTriggerMapTest.java
index fbe6cd530e..09f59fac05 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidDirectAccessTriggerMapTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidDirectAccessTriggerMapTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidDirectAccessTriggerMapTest extends PmdRuleTst {
+class AvoidDirectAccessTriggerMapTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidHardcodingIdTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidHardcodingIdTest.java
index 51f261aec0..2a88bb417b 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidHardcodingIdTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidHardcodingIdTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidHardcodingIdTest extends PmdRuleTst {
+class AvoidHardcodingIdTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidNonExistentAnnotationsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidNonExistentAnnotationsTest.java
index 45139b9d0a..b6519db2cd 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidNonExistentAnnotationsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/AvoidNonExistentAnnotationsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidNonExistentAnnotationsTest extends PmdRuleTst {
+class AvoidNonExistentAnnotationsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyCatchBlockTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyCatchBlockTest.java
index a6c0dc0cea..1bc971edbc 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyCatchBlockTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyCatchBlockTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class EmptyCatchBlockTest extends PmdRuleTst {
+class EmptyCatchBlockTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyIfStmtTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyIfStmtTest.java
index cc8f0a982e..4c131f7cd4 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyIfStmtTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyIfStmtTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class EmptyIfStmtTest extends PmdRuleTst {
+class EmptyIfStmtTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyStatementBlockTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyStatementBlockTest.java
index 7a39e1aba5..44cad0042f 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyStatementBlockTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyStatementBlockTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class EmptyStatementBlockTest extends PmdRuleTst {
+class EmptyStatementBlockTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyTryOrFinallyBlockTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyTryOrFinallyBlockTest.java
index d2d24d91af..1e68857b57 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyTryOrFinallyBlockTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyTryOrFinallyBlockTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class EmptyTryOrFinallyBlockTest extends PmdRuleTst {
+class EmptyTryOrFinallyBlockTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyWhileStmtTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyWhileStmtTest.java
index 3a98d7a9cd..a7a297124d 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyWhileStmtTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/EmptyWhileStmtTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class EmptyWhileStmtTest extends PmdRuleTst {
+class EmptyWhileStmtTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/InaccessibleAuraEnabledGetterTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/InaccessibleAuraEnabledGetterTest.java
index 61ee79a54f..31e1ebd383 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/InaccessibleAuraEnabledGetterTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/InaccessibleAuraEnabledGetterTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class InaccessibleAuraEnabledGetterTest extends PmdRuleTst {
+class InaccessibleAuraEnabledGetterTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/MethodWithSameNameAsEnclosingClassTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/MethodWithSameNameAsEnclosingClassTest.java
index 87e1ce6431..3a15b5dae8 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/MethodWithSameNameAsEnclosingClassTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/MethodWithSameNameAsEnclosingClassTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class MethodWithSameNameAsEnclosingClassTest extends PmdRuleTst {
+class MethodWithSameNameAsEnclosingClassTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/OverrideBothEqualsAndHashcodeTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/OverrideBothEqualsAndHashcodeTest.java
index 0b334a5f9f..29ab1310c3 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/OverrideBothEqualsAndHashcodeTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/OverrideBothEqualsAndHashcodeTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class OverrideBothEqualsAndHashcodeTest extends PmdRuleTst {
+class OverrideBothEqualsAndHashcodeTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/TestMethodsMustBeInTestClassesTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/TestMethodsMustBeInTestClassesTest.java
index 6557797677..1d4f6539a5 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/TestMethodsMustBeInTestClassesTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/errorprone/TestMethodsMustBeInTestClassesTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.errorprone;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class TestMethodsMustBeInTestClassesTest extends PmdRuleTst {
+class TestMethodsMustBeInTestClassesTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDebugStatementsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDebugStatementsTest.java
index 53aa68ae84..38e591773e 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDebugStatementsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDebugStatementsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.performance;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidDebugStatementsTest extends PmdRuleTst {
+class AvoidDebugStatementsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDmlStatementsInLoopsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDmlStatementsInLoopsTest.java
index e63930e585..9ceb16e5d1 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDmlStatementsInLoopsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidDmlStatementsInLoopsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.performance;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidDmlStatementsInLoopsTest extends PmdRuleTst {
+class AvoidDmlStatementsInLoopsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoqlInLoopsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoqlInLoopsTest.java
index cec9d93bf4..af15d9ce39 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoqlInLoopsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoqlInLoopsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.performance;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidSoqlInLoopsTest extends PmdRuleTst {
+class AvoidSoqlInLoopsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoslInLoopsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoslInLoopsTest.java
index ce3a33a7c8..c0657a5904 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoslInLoopsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/AvoidSoslInLoopsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.performance;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class AvoidSoslInLoopsTest extends PmdRuleTst {
+class AvoidSoslInLoopsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/EagerlyLoadedDescribeSObjectResultTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/EagerlyLoadedDescribeSObjectResultTest.java
index 3a50ee847e..ddc191c0a6 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/EagerlyLoadedDescribeSObjectResultTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/EagerlyLoadedDescribeSObjectResultTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.performance;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class EagerlyLoadedDescribeSObjectResultTest extends PmdRuleTst {
+class EagerlyLoadedDescribeSObjectResultTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/OperationWithLimitsInLoopTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/OperationWithLimitsInLoopTest.java
index 56a609bad8..e80764adbc 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/OperationWithLimitsInLoopTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/performance/OperationWithLimitsInLoopTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.performance;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class OperationWithLimitsInLoopTest extends PmdRuleTst {
+class OperationWithLimitsInLoopTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexBadCryptoTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexBadCryptoTest.java
index 05134ff78c..280a1c8cb2 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexBadCryptoTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexBadCryptoTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.security;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexBadCryptoTest extends PmdRuleTst {
+class ApexBadCryptoTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexCRUDViolationTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexCRUDViolationTest.java
index 4d4e8a6e22..c6310ba371 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexCRUDViolationTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexCRUDViolationTest.java
@@ -7,7 +7,7 @@ package net.sourceforge.pmd.lang.apex.rule.security;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexCRUDViolationTest extends PmdRuleTst {
+class ApexCRUDViolationTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexDangerousMethodsTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexDangerousMethodsTest.java
index 74fe589b3d..8796b360a4 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexDangerousMethodsTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexDangerousMethodsTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.security;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexDangerousMethodsTest extends PmdRuleTst {
+class ApexDangerousMethodsTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexInsecureEndpointTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexInsecureEndpointTest.java
index c118cbb916..000e2544c3 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexInsecureEndpointTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexInsecureEndpointTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.security;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexInsecureEndpointTest extends PmdRuleTst {
+class ApexInsecureEndpointTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectTest.java
index 5cf72744fa..4108b3b9a0 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.security;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexOpenRedirectTest extends PmdRuleTst {
+class ApexOpenRedirectTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSOQLInjectionTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSOQLInjectionTest.java
index a9a619d356..cde095f6e4 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSOQLInjectionTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSOQLInjectionTest.java
@@ -6,6 +6,6 @@ package net.sourceforge.pmd.lang.apex.rule.security;
import net.sourceforge.pmd.testframework.PmdRuleTst;
-public class ApexSOQLInjectionTest extends PmdRuleTst {
+class ApexSOQLInjectionTest extends PmdRuleTst {
// no additional unit tests
}
diff --git a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSharingViolationsNestedClassTest.java b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSharingViolationsNestedClassTest.java
index 4d87add0c5..dd2ae6531b 100644
--- a/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSharingViolationsNestedClassTest.java
+++ b/pmd-apex/src/test/java/net/sourceforge/pmd/lang/apex/rule/security/ApexSharingViolationsNestedClassTest.java
@@ -4,16 +4,15 @@
package net.sourceforge.pmd.lang.apex.rule.security;
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
import java.util.ArrayList;
import java.util.Collection;
import java.util.List;
import java.util.stream.Collectors;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.junit.runners.Parameterized;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.MethodSource;
import net.sourceforge.pmd.Report;
import net.sourceforge.pmd.RuleViolation;
@@ -27,8 +26,7 @@ import net.sourceforge.pmd.lang.apex.ast.ApexParserTestBase;
* based on the boolean permutations. Any classes that includes data access cod, but doesn't declare a sharing setting
* should trigger a violation.
*/
-@RunWith(Parameterized.class)
-public class ApexSharingViolationsNestedClassTest extends ApexParserTestBase {
+class ApexSharingViolationsNestedClassTest extends ApexParserTestBase {
/**
* Type of operation that may require a sharing declaration.
*/
@@ -56,45 +54,25 @@ public class ApexSharingViolationsNestedClassTest extends ApexParserTestBase {
/**
* The permutations used for class generation. See {@link #generateClass(boolean, Operation, boolean, Operation)}
*/
- private final boolean outerSharingDeclared;
- private final Operation outerOperation;
- private final boolean innerSharingDeclared;
- private final Operation innerOperation;
-
- /**
- * Track the expected violations based on the permutations.
- */
- private final int expectedViolations;
- private final List expectedLineNumbers;
-
- public ApexSharingViolationsNestedClassTest(boolean outerSharingDeclared, Operation outerOperation,
- boolean innerSharingDeclared, Operation innerOperation,
- int expectedViolations, List expectedLineNumbers) {
- this.outerSharingDeclared = outerSharingDeclared;
- this.outerOperation = outerOperation;
- this.innerSharingDeclared = innerSharingDeclared;
- this.innerOperation = innerOperation;
- this.expectedViolations = expectedViolations;
- this.expectedLineNumbers = expectedLineNumbers;
- }
-
- @Test
- public void testSharingPermutation() {
+ @ParameterizedTest
+ @MethodSource("data")
+ void testSharingPermutation(boolean outerSharingDeclared, Operation outerOperation,
+ boolean innerSharingDeclared, Operation innerOperation,
+ int expectedViolations, List expectedLineNumbers) {
String apexClass = generateClass(outerSharingDeclared, outerOperation, innerSharingDeclared, innerOperation);
ApexSharingViolationsRule rule = new ApexSharingViolationsRule();
rule.setMessage("a message");
Report rpt = apex.executeRule(rule, apexClass);
List violations = rpt.getViolations();
- assertEquals("Unexpected Violation Size\n" + apexClass, expectedViolations, violations.size());
+ assertEquals(expectedViolations, violations.size(), "Unexpected Violation Size\n" + apexClass);
List lineNumbers = violations.stream().map(v -> v.getBeginLine()).collect(Collectors.toList());
- assertEquals("Unexpected Line Numbers\n" + apexClass, expectedLineNumbers, lineNumbers);
+ assertEquals(expectedLineNumbers, lineNumbers, "Unexpected Line Numbers\n" + apexClass);
}
/**
* Parameter provider that covers are all permutations
*/
- @Parameterized.Parameters
- public static Collection> data() {
+ static Collection> data() {
List