fix: clarify report messages for no-missing-placeholders and no-unused-placeholders (#278)

Author: bmish

Diff for: lib/rules/no-missing-placeholders.js

@@ -26,7 +26,7 @@ module.exports = {
     schema: [],
     messages: {
       placeholderDoesNotExist:
-        'The placeholder {{{{missingKey}}}} does not exist.',
+        "The placeholder {{{{missingKey}}}} is missing (must provide it in the report's `data` object).",
     },
   },
 

Diff for: lib/rules/no-unused-placeholders.js

@@ -25,7 +25,8 @@ module.exports = {
     fixable: null,
     schema: [],
     messages: {
-      placeholderUnused: 'The placeholder {{{{unusedKey}}}} is unused.',
+      placeholderUnused:
+        'The placeholder {{{{unusedKey}}}} is unused (does not exist in the actual message).',
     },
   },
 

Diff for: tests/lib/rules/no-missing-placeholders.js

@@ -18,7 +18,10 @@ const RuleTester = require('eslint').RuleTester;
  * @returns {object} An expected error
  */
 function error(missingKey, type = 'Literal') {
-  return { type, message: `The placeholder {{${missingKey}}} does not exist.` };
+  return {
+    type,
+    message: `The placeholder {{${missingKey}}} is missing (must provide it in the report's \`data\` object).`,
+  };
 }
 
 // ------------------------------------------------------------------------------

Diff for: tests/lib/rules/no-unused-placeholders.js

@@ -18,7 +18,10 @@ const RuleTester = require('eslint').RuleTester;
  * @returns {object} An expected error
  */
 function error(unusedKey, type = 'Literal') {
-  return { type, message: `The placeholder {{${unusedKey}}} is unused.` };
+  return {
+    type,
+    message: `The placeholder {{${unusedKey}}} is unused (does not exist in the actual message).`,
+  };
 }
 
 // ------------------------------------------------------------------------------