[apex] Update tests (ApexUnitTestClassShouldHaveAsserts, ApexAssertio…

…nsShouldIncludeMessage) #4097
pmd · Aug 29, 2022 · 7c6277c · 7c6277c
1 parent 29c4e99
commit 7c6277c
Show file tree

Hide file tree

Showing 2 changed files with 70 additions and 2 deletions.
diff --git a/...t/sourceforge/pmd/lang/apex/rule/bestpractices/xml/ApexAssertionsShouldIncludeMessage.xml b/...t/sourceforge/pmd/lang/apex/rule/bestpractices/xml/ApexAssertionsShouldIncludeMessage.xml
@@ -116,4 +116,72 @@ public class Foo {
 }
         ]]></code>
    </test-code>
+
+    <test-code>
+        <description>[apex] Support new Assert class (Apex v56.0) - negative test case #4097</description>
+        <expected-problems>9</expected-problems>
+        <expected-linenumbers>6,12,19,28,34,40,46,52,58</expected-linenumbers>
+        <code><![CDATA[
+@isTest
+public class Foo {
+    @isTest
+    static void testAreEqual() {
+        String sub = 'abcde'.substring(2);
+        Assert.areEqual('cde', sub);
+   }
+
+    @isTest
+    static void testAreNotEqual() {
+        String sub = 'abcde'.substring(2);
+        Assert.areNotEqual('xyz', sub);
+   }
+
+    @isTest
+    static void testFail() {
+        try {
+            SomeClass.methodUnderTest();
+            Assert.fail();
+        } catch (DmlException ex) {
+            // Add assertions here about the expected exception
+        }
+   }
+
+    @isTest
+    static void testIsFalse() {
+        Boolean containsCode = 'Salesforce'.contains('code');
+        Assert.isFalse(containsCode);
+   }
+
+    @isTest
+    static void testIsInstanceOf() {
+        Account o = new Account();
+        Assert.isInstanceOfType(o, Account.class);
+   }
+
+    @isTest
+    static void testIsNotInstanceOf() {
+        Contact con = new Contact();
+        Assert.isNotInstanceOfType(con, Account.class);
+   }
+
+    @isTest
+    static void testIsNotNull() {
+        String myString = 'value';
+        Assert.isNotNull(myString);
+   }
+
+    @isTest
+    static void testIsNull() {
+        String myString = null;
+        Assert.isNull(myString);
+   }
+
+    @isTest
+    static void testIsTrue() {
+        Boolean containsForce = 'Salesforce'.contains('force');
+        Assert.isTrue(containsForce);
+   }
+}
+        ]]></code>
+    </test-code>
 </test-data>
diff --git a/...t/sourceforge/pmd/lang/apex/rule/bestpractices/xml/ApexUnitTestClassShouldHaveAsserts.xml b/...t/sourceforge/pmd/lang/apex/rule/bestpractices/xml/ApexUnitTestClassShouldHaveAsserts.xml
@@ -76,13 +76,13 @@ private class C2_Assignment_Report_Job_Test {
 
     <test-code>
         <description>#1089 [apex] ApexUnitTestClassShouldHaveAsserts: Verify use of additionalAssertMethodPattern, positive test</description>
-        <rule-property name="additionalAssertMethodPattern">(Assert\.\w+|verify\w+)</rule-property>
+        <rule-property name="additionalAssertMethodPattern">(MyAssert\.\w+|verify\w+)</rule-property>
         <expected-problems>0</expected-problems>
         <code><![CDATA[
 @isTest
 public class Foo {
    public static testMethod void testAssertIsTrue() {
-      Assert.isTrue(someCondition);
+      MyAssert.isTrue(someCondition);
    }
 
    public static testMethod void testLocalVerify() {