test: fix

Author: alexander-akait

src/utils.js

@@ -1183,6 +1183,7 @@ const keywords = new Set([
   "volatile",
   "while",
   "with",
+  "module",
 ]);
 
 function getExportCode(

test/__snapshots__/modules-option.test.js.snap

@@ -11817,7 +11817,7 @@ ___CSS_LOADER_EXPORT___.push([module.id, \`.ComponentName-header {
 }
 
 .ComponentName-footer {
-  color: blue;  
+  color: blue;
 }
 \`, ""]);
 // Exports
@@ -11841,7 +11841,7 @@ exports[`"modules" option should work with [folder] #3: result 1`] = `
 }
 
 .ComponentName-footer {
-  color: blue;  
+  color: blue;
 }
 ",
     "",
@@ -11868,7 +11868,7 @@ ___CSS_LOADER_EXPORT___.push([module.id, \`.ComponentName-header {
 }
 
 .ComponentName-footer {
-  color: blue;  
+  color: blue;
 }
 \`, ""]);
 // Exports
@@ -11892,7 +11892,7 @@ exports[`"modules" option should work with [folder] #4: result 1`] = `
 }
 
 .ComponentName-footer {
-  color: blue;  
+  color: blue;
 }
 ",
     "",
@@ -12431,6 +12431,29 @@ exports[`"modules" option should work with \`default\` class and without named e
 
 exports[`"modules" option should work with \`default\` class and without named export: warnings 1`] = `[]`;
 
+exports[`"modules" option should work with \`module\` class and with named export: errors 1`] = `[]`;
+
+exports[`"modules" option should work with \`module\` class and with named export: module 1`] = `
+"// Imports
+import ___CSS_LOADER_API_NO_SOURCEMAP_IMPORT___ from "../../../../src/runtime/noSourceMaps.js";
+import ___CSS_LOADER_API_IMPORT___ from "../../../../src/runtime/api.js";
+var ___CSS_LOADER_EXPORT___ = ___CSS_LOADER_API_IMPORT___(___CSS_LOADER_API_NO_SOURCEMAP_IMPORT___);
+// Module
+___CSS_LOADER_EXPORT___.push([module.id, \`.jeU4gq1x5FP0ehaW1cBG {
+  background: red
+}
+\`, ""]);
+// Exports
+var _1 = \`jeU4gq1x5FP0ehaW1cBG\`;
+export { _1 as "module" };
+export default ___CSS_LOADER_EXPORT___;
+"
+`;
+
+exports[`"modules" option should work with \`module\` class and with named export: result 1`] = `"jeU4gq1x5FP0ehaW1cBG"`;
+
+exports[`"modules" option should work with \`module\` class and with named export: warnings 1`] = `[]`;
+
 exports[`"modules" option should work with CSS nesting: errors 1`] = `[]`;
 
 exports[`"modules" option should work with CSS nesting: module 1`] = `

test/fixtures/modules/issue-1590/source.css

@@ -0,0 +1,3 @@
+.module {
+  background: red
+}

test/fixtures/modules/issue-1590/source.js

@@ -0,0 +1,5 @@
+import * as css from './source.css';
+
+__export__ = css.module;
+
+export default css;

test/modules-option.test.js

@@ -2692,4 +2692,20 @@ describe('"modules" option', () => {
     expect(getWarnings(stats)).toMatchSnapshot("warnings");
     expect(getErrors(stats)).toMatchSnapshot("errors");
   });
+
+  it("should work with `module` class and with named export", async () => {
+    const compiler = getCompiler("./modules/issue-1590/source.js", {
+      modules: true,
+    });
+    const stats = await compile(compiler);
+
+    expect(
+      getModuleSource("./modules/issue-1590/source.css", stats),
+    ).toMatchSnapshot("module");
+    expect(getExecutedCode("main.bundle.js", compiler, stats)).toMatchSnapshot(
+      "result",
+    );
+    expect(getWarnings(stats)).toMatchSnapshot("warnings");
+    expect(getErrors(stats)).toMatchSnapshot("errors");
+  });
 });