Merge pull request #733 from forcedotcom/dev-3

@W-11321290@ Merging dev-3 to release-3 for 3.2.0

Author: rmohan20

cli-messaging/build.gradle.kts

@@ -0,0 +1,26 @@
+plugins {
+  java
+}
+
+version = "1.0"
+java.sourceCompatibility = JavaVersion.VERSION_1_8
+group = "com.salesforce.messaging"
+
+repositories {
+  mavenCentral()
+}
+
+dependencies {
+  implementation ("com.googlecode.json-simple:json-simple:1.1.1") {
+    exclude("junit")
+  }
+  implementation("com.google.code.gson:gson:2.3")
+  testImplementation("junit", "junit", "4.12")
+  implementation("com.google.guava:guava:28.0-jre")
+  testImplementation("org.junit.jupiter:junit-jupiter-api:5.6.0")
+  testRuntimeOnly("org.junit.jupiter:junit-jupiter-engine")
+}
+
+tasks.getByName<Test>("test") {
+  useJUnitPlatform()
+}

cli-messaging/gradle/wrapper/gradle-wrapper.jar

@@ -0,0 +1,5 @@
+distributionBase=GRADLE_USER_HOME
+distributionPath=wrapper/dists
+distributionUrl=https\://services.gradle.org/distributions/gradle-6.2.1-bin.zip
+zipStoreBase=GRADLE_USER_HOME
+zipStorePath=wrapper/dists

cli-messaging/gradle/wrapper/gradle-wrapper.properties

@@ -0,0 +1 @@
+rootProject.name = "cli-messaging"

cli-messaging/settings.gradle.kts

@@ -1,26 +1,21 @@
-package sfdc.sfdx.scanner.messaging;
-
-import com.google.gson.Gson;
+package com.salesforce.messaging;
 
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
 
-public class SfdxMessager {
+import com.google.gson.Gson;
+
+public class CliMessager {
 	// The START string gives us something to scan for when we're processing output.
 	private static final String START = "SFDX-START";
 	// The END string lets us know when a message stops, which should prevent bugs involving multi-line output.
 	private static final String END = "SFDX-END";
 
 	private static final List<Message> MESSAGES = new ArrayList<>();
 
-	private static SfdxMessager INSTANCE = null;
-
-	public static SfdxMessager getInstance() {
-		if (INSTANCE == null) {
-			INSTANCE = new SfdxMessager();
-		}
-		return INSTANCE;
+	public static CliMessager getInstance() {
+		return LazyHolder.INSTANCE;
 	}
 
 	/**
@@ -31,7 +26,7 @@ public static SfdxMessager getInstance() {
 	 *
 	 * @param exception to send to Typescript layer
 	 */
-	public void addMessage(SfdxScannerException exception) {
+	public void addMessage(MessagePassableException exception) {
 		final EventKey eventKey = exception.getEventKey();
 		addMessage(
 			exception.getFullStacktrace(),
@@ -65,7 +60,12 @@ public void addMessage(String internalLog, EventKey eventKey, String... args) {
 		MESSAGES.add(message);
 	}
 
-
+	/**
+	 * Convert all messages stored by the instance into a JSON-formatted string, enclosed in the start and end strings.
+	 * Java code can use this method to log the messages to console, and TypeScript code can seek the start and stop
+	 * strings to get an array of messages that can be deserialized.
+	 * @return
+	 */
 	public String getAllMessagesWithFormatting() {
 		final String messagesAsJson = getMessagesAsJson();
 		return START + messagesAsJson + END;
@@ -75,7 +75,6 @@ private String getMessagesAsJson() {
 		return new Gson().toJson(MESSAGES);
 	}
 
-
 	/**
 	 * TO BE USED ONLY BY TESTS!
 	 *
@@ -93,15 +92,8 @@ public void resetMessages() {
 		MESSAGES.clear();
 	}
 
-	enum MessageHandler {
-		UX,
-		INTERNAL
-	}
-
-	enum MessageType {
-		INFO,
-		WARNING,
-		ERROR
+	private static final class LazyHolder {
+		// Postpone initialization until first use
+		private static final CliMessager INSTANCE = new CliMessager();
 	}
 }
-

pmd-cataloger/src/main/java/sfdc/sfdx/scanner/messaging/SfdxMessager.java renamed to cli-messaging/src/main/java/com/salesforce/messaging/CliMessager.java

@@ -1,8 +1,8 @@
-package sfdc.sfdx.scanner.messaging;
-import static sfdc.sfdx.scanner.messaging.SfdxMessager.*;
+package com.salesforce.messaging;
+import static com.salesforce.messaging.Message.*;
 
 public enum EventKey {
-	// MAKE SURE messageKey OF EVERY VALUE ADDED HERE HAS AN ENTRY IN 'messages/EventKeyTemplates.js'!
+	// MAKE SURE `messageKey` OF EVERY VALUE ADDED HERE HAS AN ENTRY IN 'messages/EventKeyTemplates.js'!
 	INFO_GENERAL_INTERNAL_LOG("info.generalInternalLog", 1, MessageType.INFO, MessageHandler.INTERNAL, true),
 	WARNING_INVALID_CAT_SKIPPED("warning.invalidCategorySkipped", 1, MessageType.WARNING, MessageHandler.UX, true),
 	WARNING_INVALID_RULESET_SKIPPED("warning.invalidRulesetSkipped", 1, MessageType.WARNING, MessageHandler.UX, true),
@@ -19,13 +19,15 @@ public enum EventKey {
 	ERROR_EXTERNAL_MULTIPLE_RULE_DESC("error.external.multipleRuleDesc", 2, MessageType.ERROR, MessageHandler.UX, false),
 	ERROR_EXTERNAL_RECURSION_LIMIT("error.external.recursionLimitReached", 2, MessageType.ERROR, MessageHandler.UX, false),
 	ERROR_EXTERNAL_XML_NOT_READABLE("error.external.xmlNotReadable", 2, MessageType.ERROR, MessageHandler.UX, false),
-	ERROR_EXTERNAL_XML_NOT_PARSABLE("error.external.xmlNotParsable", 2, MessageType.ERROR, MessageHandler.UX, false);
-
-	String messageKey;
-	int argCount;
-	MessageType messageType;
-	MessageHandler messageHandler;
-	boolean verbose;//true: only when verbose is true, false: ignores verbose flag and always prints
+	ERROR_EXTERNAL_XML_NOT_PARSABLE("error.external.xmlNotParsable", 2, MessageType.ERROR, MessageHandler.UX, false),
+	WARNING_MULTIPLE_METHOD_TARGET_MATCHES("warning.multipleMethodTargetMatches", 3, MessageType.WARNING, MessageHandler.UX, false),
+	WARNING_NO_METHOD_TARGET_MATCHES("warning.noMethodTargetMatches", 2, MessageType.WARNING, MessageHandler.UX, false);
+
+	final String messageKey;
+	final int argCount;
+	final MessageType messageType;
+	final MessageHandler messageHandler;
+	final boolean verbose;//true: only when verbose is true, false: ignores verbose flag and always prints
 
 	EventKey(String messageKey, int argCount, MessageType messageType, MessageHandler messageHandler, boolean verbose) {
 		this.messageKey = messageKey;

pmd-cataloger/src/main/java/sfdc/sfdx/scanner/messaging/EventKey.java renamed to cli-messaging/src/main/java/com/salesforce/messaging/EventKey.java

@@ -1,31 +1,25 @@
-package sfdc.sfdx.scanner.messaging;
+package com.salesforce.messaging;
 
-import com.google.gson.Gson;
-import static sfdc.sfdx.scanner.messaging.SfdxMessager.*;
 import java.time.Instant;
 import java.util.List;
 
 public class Message {
-	private String messageKey;
-	private List<String> args;
-	private String internalLog;
-	private MessageType type;
-	private MessageHandler handler;
-	private boolean verbose;
-	private long time;
+	final private String messageKey;
+	final private List<String> args;
+	final private String internalLog;
+	final private MessageType type;
+	final private MessageHandler handler;
+	final private boolean verbose;
+	final private long time;
 
 	Message(String messageKey, List<String> args, String internalLog, MessageType type, MessageHandler handler, boolean verbose) {
 		this.messageKey = messageKey;
 		this.args = args;
 		this.internalLog = internalLog;
 		this.type = type;
 		this.handler = handler;
-		this.time = Instant.now().toEpochMilli();
 		this.verbose = verbose;
-	}
-
-	String toJson() {
-		return new Gson().toJson(this);
+		this.time = Instant.now().toEpochMilli();
 	}
 
 	public String getMessageKey() {
@@ -40,4 +34,14 @@ public String getInternalLog() {
 		return internalLog;
 	}
 
+	enum MessageHandler {
+		UX,
+		INTERNAL
+	}
+
+	enum MessageType {
+		INFO,
+		WARNING,
+		ERROR
+	}
 }

pmd-cataloger/src/main/java/sfdc/sfdx/scanner/messaging/Message.java renamed to cli-messaging/src/main/java/com/salesforce/messaging/Message.java

@@ -1,25 +1,24 @@
-package sfdc.sfdx.scanner.messaging;
+package com.salesforce.messaging;
 
 import com.google.common.base.Throwables;
-import sfdc.sfdx.scanner.messaging.EventKey;
 
 import java.util.Arrays;
 
 /**
  * Internal exception representation.
  * Extends RuntimeException to avoid declaring everywhere
- * Handles capability to plug into SfdxMessager
+ * Handles capability to plug into CliMessager
  */
-public class SfdxScannerException extends RuntimeException {
+public class MessagePassableException extends RuntimeException {
 
 	private final EventKey eventKey;
 	private final String[] args;
 
-	public SfdxScannerException(EventKey eventKey, String... args) {
+	public MessagePassableException(EventKey eventKey, String... args) {
 		this(eventKey, null, args);
 	}
 
-	public SfdxScannerException(EventKey eventKey, Throwable throwable, String... args) {
+	public MessagePassableException(EventKey eventKey, Throwable throwable, String... args) {
 		super(throwable);
 
 		this.eventKey = eventKey;
@@ -40,7 +39,7 @@ public String getFullStacktrace() {
 
 	@Override
 	public String toString() {
-		return "SfdxScannerException{" +
+		return "MessagePassableException{" +
 			"eventKey=" + eventKey +
 			", args=" + Arrays.toString(args) +
 			'}';

pmd-cataloger/src/main/java/sfdc/sfdx/scanner/messaging/SfdxScannerException.java renamed to cli-messaging/src/main/java/com/salesforce/messaging/MessagePassableException.java

@@ -1,4 +1,4 @@
-package sfdc.sfdx.scanner.messaging;
+package com.salesforce.messaging;
 
 import org.json.simple.JSONObject;
 import org.json.simple.parser.JSONParser;
@@ -12,7 +12,7 @@
 import org.junit.runners.Parameterized;
 import org.junit.runners.Parameterized.Parameters;
 
-import static sfdc.sfdx.scanner.messaging.SfdxMessager.*;
+import static com.salesforce.messaging.Message.*;
 
 import java.io.IOException;
 import java.nio.file.Files;

pmd-cataloger/src/test/java/sfdc/sfdx/scanner/messaging/EventKeyTest.java renamed to cli-messaging/src/test/java/com/salesforce/messaging/EventKeyTest.java

@@ -18,6 +18,8 @@ module.exports = {
 		"pmdSkippedFile": "PMD failed to evaluate against file '%s'. Message: %s",
 		"pmdSuppressedViolation": "PMD suppressed violation against file '%s'. Message: %s. Suppression Type: %s. User Message: %s",
 		"unexpectedPmdNodeType": "Encountered unexpected PMD node of type '%s'",
+		"multipleMethodTargetMatches": "Total of %s methods in file %s matched name #%s",
+		"noMethodTargetMatches": "No methods in file %s matched name #%s()",
 		"pmdConfigError": "PMD failed to evaluate rule '%s'. Message: %s"
 	},
 	"error": {