fix: webpack 5 support

src/loader.js

@@ -170,7 +170,7 @@ export function pitch(request) {
 
  const callback = this.async();
 
- childCompiler.runAsChild((err, entries, compilation) => {
+ childCompiler.runAsChild((error, entries, compilation) => {
  const assets = Object.create(null);
  const assetsInfo = new Map();
 
@@ -199,7 +199,15 @@ export function pitch(request) {
  }
 
  const count = identifierCountMap.get(dependency.identifier) || 0;
- const { CssDependency } = shared(this._compiler.webpack);
+ const { CssDependency } = shared(webpack, () => {
+ return {};
+ });
+
+ if (!CssDependency) {
+ throw new Error(
+ "You forgot to add 'mini-css-extract-plugin' plugin (i.e. `{ plugins: [new MiniCssExtractPlugin()] }`), please read https://github.com/webpack-contrib/mini-css-extract-plugin#getting-started"
+ );
+ }
 
  this._module.addDependency(
  (lastDep = new CssDependency(dependency, dependency.context, count))
@@ -213,8 +221,8 @@ export function pitch(request) {
  }
  };
 
- if (err) {
- return callback(err);
+ if (error) {
+ return callback(error);
  }
 
  if (compilation.errors.length > 0) {

test/TestCases.test.js

@@ -127,18 +127,42 @@ describe('TestCases', () => {
  );
  }
 
- webpack(webpackConfig, (err, stats) => {
- if (err) {
- done(err);
+ webpack(webpackConfig, (error, stats) => {
+ if (error) {
+ done(error);
+
  return;
  }
+
  if (stats.hasErrors()) {
+ const errorsPath = path.join(directoryForCase, './errors.test.js');
+
+ if (fs.existsSync(errorsPath)) {
+ const { errors } = stats.compilation;
+ // eslint-disable-next-line global-require, import/no-dynamic-require
+ const errorFilters = require(errorsPath);
+ const filteredErrors = errors.filter(
+ // eslint-disable-next-line no-shadow
+ (error) =>
+ !errorFilters.some((errorFilter) => errorFilter.test(error))
+ );
+
+ if (filteredErrors.length > 0) {
+ done(new Error(`Errors:\n${filteredErrors.join(',\n')}`));
+
+ return;
+ }
+
+ done();
+
+ return;
+ }
+
  done(new Error(stats.toString()));
+
  return;
  }
 
- done();
-
  if (stats.hasErrors() && stats.hasWarnings()) {
  done(
  new Error(

test/cases/no-loader/index.js

@@ -0,0 +1 @@
+import './style.css';

test/cases/no-loader/style.css

@@ -0,0 +1,3 @@
+body {
+ background: red;
+}

test/cases/no-loader/webpack.config.js

@@ -0,0 +1,18 @@
+import Self from '../../../src';
+
+module.exports = {
+ entry: './index.js',
+ module: {
+ rules: [
+ {
+ test: /\.css$/,
+ use: ['css-loader'],
+ },
+ ],
+ },
+ plugins: [
+ new Self({
+ filename: '[name].css',
+ }),
+ ],
+};