Add a few more test cases

#17 (comment)
eslint-community · Apr 24, 2022 · 8425c80 · 8425c80
1 parent a4ec500
commit 8425c80
Showing 1 changed file with 29 additions and 0 deletions.
diff --git a/tests/lib/rules/exports-style.js b/tests/lib/rules/exports-style.js
@@ -201,6 +201,26 @@ new RuleTester().run("exports-style", rule, {
                 "Unexpected access to 'module.exports'. Use 'exports' instead.",
             ],
         },
+        {
+            code:
+                "if(foo){ module.exports = { foo: 1};} else { module.exports = {foo: 2};}",
+            output: null,
+            options: ["exports"],
+            globals: { module: false, exports: true },
+            errors: [
+                "Unexpected access to 'module.exports'. Use 'exports' instead.",
+                "Unexpected access to 'module.exports'. Use 'exports' instead.",
+            ],
+        },
+        {
+            code: "function bar() { module.exports = { foo: 1 }; }",
+            output: null,
+            options: ["exports"],
+            globals: { module: false, exports: true },
+            errors: [
+                "Unexpected access to 'module.exports'. Use 'exports' instead.",
+            ],
+        },
         {
             code: "module.exports = { get a() {} }",
             output: null,
@@ -210,6 +230,15 @@ new RuleTester().run("exports-style", rule, {
                 "Unexpected access to 'module.exports'. Use 'exports' instead.",
             ],
         },
+        {
+            code: "module.exports = { set a(a) {} }",
+            output: null,
+            options: ["exports"],
+            globals: { module: false, exports: true },
+            errors: [
+                "Unexpected access to 'module.exports'. Use 'exports' instead.",
+            ],
+        },
         {
             code: "module.exports = { a }",
             output: "exports.a = a;",