Fix PMD warnings

Author: mitchspano

trigger-actions-framework/main/default/classes/MetadataTriggerHandler.cls

@@ -76,7 +76,7 @@
  *
  */
 @SuppressWarnings(
-	'PMD.CognitiveComplexity, PMD.StdCyclomaticComplexity, PMD.CyclomaticComplexity'
+	'PMD.CognitiveComplexity,PMD.StdCyclomaticComplexity,PMD.CyclomaticComplexity'
 )
 public inherited sharing class MetadataTriggerHandler extends TriggerBase implements TriggerAction.BeforeInsert, TriggerAction.AfterInsert, TriggerAction.BeforeUpdate, TriggerAction.AfterUpdate, TriggerAction.BeforeDelete, TriggerAction.AfterDelete, TriggerAction.AfterUndelete {
 	private static final String DOUBLE_UNDERSCORE = '__';

trigger-actions-framework/main/default/classes/MetadataTriggerHandlerTest.cls

@@ -15,7 +15,7 @@
  */
 
 @SuppressWarnings(
-	'PMD.ApexDoc, PMD.CyclomaticComplexity, PMD.ApexUnitTestClassShouldHaveRunAs'
+	'PMD.ApexDoc,PMD.CyclomaticComplexity,PMD.ApexUnitTestClassShouldHaveRunAs'
 )
 @IsTest(isParallel=true)
 private class MetadataTriggerHandlerTest {

trigger-actions-framework/main/default/classes/TriggerActionFlow.cls

@@ -19,7 +19,7 @@
  * @description This class implements the TriggerAction interface and provides a framework for
  * executing Flows before or after the insert, update, delete, or undelete of records.
  */
-@SuppressWarnings('PMD.CyclomaticComplexity, PMD.CognitiveComplexity')
+@SuppressWarnings('PMD.CyclomaticComplexity,PMD.CognitiveComplexity')
 public virtual inherited sharing class TriggerActionFlow implements TriggerAction.BeforeInsert, TriggerAction.AfterInsert, TriggerAction.BeforeUpdate, TriggerAction.AfterUpdate, TriggerAction.BeforeDelete, TriggerAction.AfterDelete, TriggerAction.AfterUndelete {
 	@TestVisible
 	private static final String RECORD_VARIABLE_NOT_FOUND_ERROR = 'There must be a variable defined in this flow with api name of "record" and type of "record" that is marked as "available for output"';
@@ -449,6 +449,7 @@ public virtual inherited sharing class TriggerActionFlow implements TriggerActio
 		}
 	}
 
+	@SuppressWarnings('PMD.ApexDoc')
 	@TestVisible
 	private virtual class NameExtractor {
 		public virtual String extractName(System.Type flowType) {

trigger-actions-framework/main/default/classes/TriggerActionFlowTest.cls

@@ -15,7 +15,7 @@
  */
 
 @SuppressWarnings(
-	'PMD.ApexDoc, PMD.ApexUnitTestClassShouldHaveRunAs, PMD.CyclomaticComplexity'
+	'PMD.ApexDoc,PMD.ApexUnitTestClassShouldHaveRunAs,PMD.CyclomaticComplexity'
 )
 @IsTest(isParallel=true)
 private class TriggerActionFlowTest {
@@ -568,6 +568,7 @@ private class TriggerActionFlowTest {
 			this.outputParameters = outputParameters;
 			return this;
 		}
+		@SuppressWarnings('PMD.AvoidBooleanMethodParameters')
 		public InvocableActionResultBuilder setSuccess(Boolean success) {
 			this.success = success;
 			return this;

trigger-actions-framework/main/default/classes/TriggerBase.cls

@@ -33,7 +33,7 @@
  * The `TriggerAction` interface defines the methods that should be implemented by trigger actions.
  */
 @SuppressWarnings(
-	'PMD.StdCyclomaticComplexity, PMD.CyclomaticComplexity, PMD.CognitiveComplexity, PMD.PropertyNamingConventions'
+	'PMD.StdCyclomaticComplexity,PMD.CyclomaticComplexity,PMD.CognitiveComplexity,PMD.PropertyNamingConventions'
 )
 public inherited sharing virtual class TriggerBase {
 	@TestVisible
@@ -142,7 +142,9 @@ public inherited sharing virtual class TriggerBase {
 	 * @param sObjectType The sObjectType of the object to bypass.
 	 */
 	public static void bypass(Schema.sObjectType sObjectType) {
-		TriggerBase.bypassedObjects.add(sObjectType.getDescribe().getName());
+		TriggerBase.bypassedObjects.add(
+			sObjectType.getDescribe(SObjectDescribeOptions.DEFERRED).getName()
+		);
 	}
 
 	/**
@@ -151,7 +153,9 @@ public inherited sharing virtual class TriggerBase {
 	 * @param sObjectType The sObjectType of the object to clear the bypass for.
 	 */
 	public static void clearBypass(Schema.sObjectType sObjectType) {
-		TriggerBase.bypassedObjects.remove(sObjectType.getDescribe().getName());
+		TriggerBase.bypassedObjects.remove(
+			sObjectType.getDescribe(SObjectDescribeOptions.DEFERRED).getName()
+		);
 	}
 
 	/**
@@ -162,7 +166,7 @@ public inherited sharing virtual class TriggerBase {
 	 */
 	public static Boolean isBypassed(Schema.sObjectType sObjectType) {
 		return TriggerBase.bypassedObjects.contains(
-			sObjectType.getDescribe().getName()
+			sObjectType.getDescribe(SObjectDescribeOptions.DEFERRED).getName()
 		);
 	}
 

trigger-actions-framework/main/default/classes/TriggerBaseTest.cls

@@ -14,7 +14,9 @@
    limitations under the License.
  */
 
-@SuppressWarnings('PMD.ApexDoc, PMD.ApexUnitTestClassShouldHaveRunAs')
+@SuppressWarnings(
+	'PMD.ApexDoc,PMD.CyclomaticComplexity,PMD.ApexUnitTestClassShouldHaveRunAs'
+)
 @IsTest(isParallel=true)
 private class TriggerBaseTest {
 	private static final String ACCOUNT = 'Account';