webpack-contrib · Jul 20, 2021 · Jul 20, 2021
Showing with 56 additions and 2 deletions.

+7 −0 CHANGELOG.md

+1 −1 package-lock.json

+1 −1 package.json

+2 −0 src/utils.js

+26 −0 test/__snapshots__/styleTagTransform-option.test.js.snap

+19 −0 test/styleTagTransform-option.test.js
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [3.2.1](https://github.com/webpack-contrib/style-loader/compare/v3.2.0...v3.2.1) (2021-07-20)
+
+
+### Bug Fixes
+
+* added the `styletagtransform` option when it is a module to `addBuildDependency` ([#528](https://github.com/webpack-contrib/style-loader/issues/528)) ([270513f](https://github.com/webpack-contrib/style-loader/commit/270513fa76e13c96a36c2ae11e4dd526dfb9d72f))
+
 ## [3.2.0](https://github.com/webpack-contrib/style-loader/compare/v3.1.0...v3.2.0) (2021-07-20)
 
 

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "style-loader",
-  "version": "3.2.0",
+  "version": "3.2.1",
   "description": "style loader module for webpack",
   "license": "MIT",
   "repository": "webpack-contrib/style-loader",

diff --git a/src/utils.js b/src/utils.js
@@ -314,6 +314,8 @@ function getStyleTagTransformFnCode(
       `${options.styleTagTransform}`
     );
 
+    loaderContext.addBuildDependency(options.styleTagTransform);
+
     return esModule
       ? `import styleTagTransformFn from ${modulePath};`
       : `var styleTagTransformFn = require(${modulePath});`;

diff --git a/test/__snapshots__/styleTagTransform-option.test.js.snap b/test/__snapshots__/styleTagTransform-option.test.js.snap
@@ -1,5 +1,31 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`"styleTagTransform" option should "styleTagTransform" function path added to buildDependencies when injectType lazyStyleTag: DOM 1`] = `
+"<!DOCTYPE html><html><head>
+  <title>style-loader test</title>
+  <style id=\\"existing-style\\">.existing { color: yellow }</style>
+<style>body {
+  color: red;
+}
+.modify{}
+</style><style>h1 {
+  color: blue;
+}
+.modify{}
+</style></head>
+<body>
+  <h1>Body</h1>
+  <div class=\\"target\\"></div>
+  <iframe class=\\"iframeTarget\\"></iframe>
+
+
+</body></html>"
+`;
+
+exports[`"styleTagTransform" option should "styleTagTransform" function path added to buildDependencies when injectType lazyStyleTag: errors 1`] = `Array []`;
+
+exports[`"styleTagTransform" option should "styleTagTransform" function path added to buildDependencies when injectType lazyStyleTag: warnings 1`] = `Array []`;
+
 exports[`"styleTagTransform" option should work when the "styleTagTransform" option is not specify and injectType lazyStyleTag: DOM 1`] = `
 "<!DOCTYPE html><html><head>
   <title>style-loader test</title>

diff --git a/test/styleTagTransform-option.test.js b/test/styleTagTransform-option.test.js
@@ -99,4 +99,23 @@ describe('"styleTagTransform" option', () => {
     expect(getWarnings(stats)).toMatchSnapshot("warnings");
     expect(getErrors(stats)).toMatchSnapshot("errors");
   });
+
+  it(`should "styleTagTransform" function path added to buildDependencies when injectType lazyStyleTag`, async () => {
+    const styleTagTransformFn = require.resolve("./fixtures/styleTagTransform");
+    const entry = getEntryByInjectType("simple.js", "lazyStyleTag");
+    const compiler = getCompiler(entry, {
+      injectType: "lazyStyleTag",
+      styleTagTransform: styleTagTransformFn,
+    });
+    const stats = await compile(compiler);
+    const { buildDependencies } = stats.compilation;
+
+    runInJsDom("main.bundle.js", compiler, stats, (dom) => {
+      expect(dom.serialize()).toMatchSnapshot("DOM");
+    });
+
+    expect(buildDependencies.has(styleTagTransformFn)).toBe(true);
+    expect(getWarnings(stats)).toMatchSnapshot("warnings");
+    expect(getErrors(stats)).toMatchSnapshot("errors");
+  });
 });