refactor: remove reloadAll option

Author: cap-Bernardito

src/hmr/hotModuleReplacement.js

@@ -216,7 +216,7 @@ module.exports = function (moduleId, options) {
       return;
     }
 
-    if (reloaded && !options.reloadAll) {
+    if (reloaded) {
       console.log('[HMR] css reload %s', src.join(' '));
     } else {
       console.log('[HMR] Reload all css');

src/loader-options.json

@@ -14,9 +14,6 @@
     },
     "esModule": {
       "type": "boolean"
-    },
-    "reloadAll": {
-      "type": "boolean"
     }
   }
 }

test/HMR.test.js

@@ -126,31 +126,6 @@ describe('HMR', () => {
     }, 100);
   });
 
-  it('should reloads with reloadAll option', (done) => {
-    const update = hotModuleReplacement('./src/style.css', {
-      reloadAll: true,
-    });
-
-    update();
-
-    setTimeout(() => {
-      expect(console.log.mock.calls[0][0]).toMatchSnapshot();
-
-      const links = Array.prototype.slice.call(
-        document.querySelectorAll('link')
-      );
-
-      expect(links[0].visited).toBe(true);
-      expect(document.head.innerHTML).toMatchSnapshot();
-
-      links[1].dispatchEvent(getLoadEvent());
-
-      expect(links[1].isLoaded).toBe(true);
-
-      done();
-    }, 100);
-  });
-
   it('should reloads with non http/https link href', (done) => {
     document.head.innerHTML =
       '<link rel="stylesheet" href="/dist/main.css" /><link rel="shortcut icon" href="data:;base64,=" />';

test/__snapshots__/HMR.test.js.snap

@@ -24,10 +24,6 @@ exports[`HMR should reloads with non http/https link href 1`] = `"[HMR] css relo
 
 exports[`HMR should reloads with non http/https link href 2`] = `"<link rel=\\"stylesheet\\" href=\\"/dist/main.css\\"><link rel=\\"stylesheet\\" href=\\"http://localhost/dist/main.css?1479427200000\\"><link rel=\\"shortcut icon\\" href=\\"data:;base64,=\\">"`;
 
-exports[`HMR should reloads with reloadAll option 1`] = `"[HMR] Reload all css"`;
-
-exports[`HMR should reloads with reloadAll option 2`] = `"<link rel=\\"stylesheet\\" href=\\"/dist/main.css\\"><link rel=\\"stylesheet\\" href=\\"http://localhost/dist/main.css?1479427200000\\">"`;
-
 exports[`HMR should works 1`] = `"[HMR] css reload %s"`;
 
 exports[`HMR should works 2`] = `"<link rel=\\"stylesheet\\" href=\\"/dist/main.css\\"><link rel=\\"stylesheet\\" href=\\"http://localhost/dist/main.css?1479427200000\\">"`;

test/__snapshots__/validate-loader-options.test.js.snap

@@ -13,8 +13,3 @@ exports[`validate options should throw an error on the "publicPath" option with
     * options.publicPath should be a string.
     * options.publicPath should be an instance of function."
 `;
-
-exports[`validate options should throw an error on the "reloadAll" option with "1" value 1`] = `
-"Invalid options object. Mini CSS Extract Plugin Loader has been initialized using an options object that does not match the API schema.
- - options.reloadAll should be a boolean."
-`;

test/cases/hmr/expected/webpack-4/main.js

@@ -1092,7 +1092,7 @@ module.exports = function (moduleId, options) {
       return;
     }
 
-    if (reloaded && !options.reloadAll) {
+    if (reloaded) {
       console.log('[HMR] css reload %s', src.join(' '));
     } else {
       console.log('[HMR] Reload all css');

test/cases/hmr/expected/webpack-5/main.js

@@ -234,7 +234,7 @@ module.exports = function (moduleId, options) {
       return;
     }
 
-    if (reloaded && !options.reloadAll) {
+    if (reloaded) {
       console.log('[HMR] css reload %s', src.join(' '));
     } else {
       console.log('[HMR] Reload all css');

test/validate-loader-options.test.js

@@ -10,10 +10,6 @@ describe('validate options', () => {
       success: [true, false],
       failure: [1],
     },
-    reloadAll: {
-      success: [true, false],
-      failure: [1],
-    },
     unknown: {
       success: [],
       // TODO failed in next release