fix(eslint-plugin): test falsy values and fix some metadata

typescript-eslint · Apr 23, 2022 · 84db88d · 84db88d
1 parent 44afc2d
commit 84db88d
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/packages/eslint-plugin/src/rules/no-duplicate-enum-values.ts b/packages/eslint-plugin/src/rules/no-duplicate-enum-values.ts
@@ -8,11 +8,10 @@ export default util.createRule({
     docs: {
       description: 'Disallow duplicate enum member values',
       recommended: false,
-      suggestion: true,
     },
     hasSuggestions: true,
     messages: {
-      duplicateMember: 'Duplicate enum member with value {{value}}.',
+      duplicateValue: 'Duplicate enum member value {{value}}.',
     },
     schema: [],
   },
@@ -58,7 +57,7 @@ export default util.createRule({
           if (seenValues.has(value)) {
             context.report({
               node: member,
-              messageId: 'duplicateMember',
+              messageId: 'duplicateValue',
               data: {
                 value,
               },

diff --git a/packages/eslint-plugin/tests/rules/no-duplicate-enum-values.test.ts b/packages/eslint-plugin/tests/rules/no-duplicate-enum-values.test.ts
@@ -59,6 +59,17 @@ enum E {
   B = 'B',
   C = 2,
   D = foo(),
+}
+    `,
+    `
+enum E {
+  A = false,
+  B = 0,
+  C = -0,
+  D = 0n,
+  E = null,
+  F = undefined,
+  G = NaN,
 }
     `,
   ],
@@ -74,7 +85,7 @@ enum E {
         {
           line: 4,
           column: 3,
-          messageId: 'duplicateMember',
+          messageId: 'duplicateValue',
           data: { value: 1 },
         },
       ],
@@ -90,7 +101,7 @@ enum E {
         {
           line: 4,
           column: 3,
-          messageId: 'duplicateMember',
+          messageId: 'duplicateValue',
           data: { value: 'A' },
         },
       ],
@@ -108,13 +119,13 @@ enum E {
         {
           line: 4,
           column: 3,
-          messageId: 'duplicateMember',
+          messageId: 'duplicateValue',
           data: { value: 'A' },
         },
         {
           line: 6,
           column: 3,
-          messageId: 'duplicateMember',
+          messageId: 'duplicateValue',
           data: { value: 1 },
         },
       ],