test: fix for cacheContext-option in order to normalize the snapshots path (#52)

Author: mistic

package-lock.json

@@ -38,8 +38,8 @@
     "@webpack-contrib/eslint-config-webpack": "^3.0.0",
     "babel-cli": "^6.0.0",
     "babel-core": "^6.26.3",
-    "babel-loader": "^7.1.5",
     "babel-jest": "^21.0.0",
+    "babel-loader": "^7.1.5",
     "babel-plugin-transform-object-rest-spread": "^6.0.0",
     "babel-polyfill": "^6.0.0",
     "babel-preset-env": "^1.6.0",
@@ -53,8 +53,9 @@
     "jest": "^21.0.0",
     "lint-staged": "^5.0.0",
     "memory-fs": "^0.4.1",
-    "prettier": "^1.15.2",
+    "normalize-path": "^3.0.0",
     "pre-commit": "^1.0.0",
+    "prettier": "^1.15.2",
     "standard-version": "^4.0.0",
     "webpack": "^4.27.1",
     "webpack-cli": "^3.1.2",

package.json

@@ -11,17 +11,20 @@ Array [
     "dependencies": Array [
       Object {
         "mtime": null,
-        "path": "test/fixtures/basic/file_1.js",
+        "path": "test/fixtures/basic/index.js",
       },
       Object {
         "mtime": null,
         "path": "src/index.js",
       },
     ],
-    "remainingRequest": "test/fixtures/basic/file_1.js",
+    "remainingRequest": "test/fixtures/basic/index.js",
     "result": Array [
       "/* eslint-disable */
-console.log('file_1');
+require('./file_1.js');
+require('./file_2.js');
+
+console.log('basic_entry');
 ",
     ],
   },
@@ -30,17 +33,17 @@ console.log('file_1');
     "dependencies": Array [
       Object {
         "mtime": null,
-        "path": "test/fixtures/basic/file_2.js",
+        "path": "test/fixtures/basic/file_1.js",
       },
       Object {
         "mtime": null,
         "path": "src/index.js",
       },
     ],
-    "remainingRequest": "test/fixtures/basic/file_2.js",
+    "remainingRequest": "test/fixtures/basic/file_1.js",
     "result": Array [
       "/* eslint-disable */
-console.log('file_2');
+console.log('file_1');
 ",
     ],
   },
@@ -49,20 +52,17 @@ console.log('file_2');
     "dependencies": Array [
       Object {
         "mtime": null,
-        "path": "test/fixtures/basic/index.js",
+        "path": "test/fixtures/basic/file_2.js",
       },
       Object {
         "mtime": null,
         "path": "src/index.js",
       },
     ],
-    "remainingRequest": "test/fixtures/basic/index.js",
+    "remainingRequest": "test/fixtures/basic/file_2.js",
     "result": Array [
       "/* eslint-disable */
-require('./file_1.js');
-require('./file_2.js');
-
-console.log('basic_entry');
+console.log('file_2');
 ",
     ],
   },

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

@@ -1,4 +1,5 @@
 const path = require('path');
+const normalizePath = require('normalize-path');
 const { webpack } = require('./helpers');
 
 const mockCacheLoaderWriteFn = jest.fn();
@@ -25,17 +26,27 @@ const mockRelativeWebpackConfig = {
 };
 
 const buildSnapshotReadyDeps = (deps) =>
-  deps.map((dep) => Object.assign({}, dep, { mtime: null })).sort();
+  deps.map((dep) =>
+    Object.assign({}, dep, { mtime: null, path: normalizePath(dep.path) })
+  );
 
 const buildCacheLoaderCallsData = (calls) =>
-  calls.sort().map((rawCall) => {
-    const call = rawCall[1];
-    return {
-      ...call,
-      dependencies: buildSnapshotReadyDeps(call.dependencies),
-      contextDependencies: buildSnapshotReadyDeps(call.contextDependencies),
-    };
-  });
+  Array.from(
+    calls
+      .reduce((builtCalls, call) => {
+        const [, rawData] = call;
+
+        return builtCalls.set(rawData.remainingRequest, {
+          ...rawData,
+          remainingRequest: normalizePath(rawData.remainingRequest),
+          dependencies: buildSnapshotReadyDeps(rawData.dependencies),
+          contextDependencies: buildSnapshotReadyDeps(
+            rawData.contextDependencies
+          ),
+        });
+      }, new Map())
+      .values()
+  );
 
 describe('cacheContext option', () => {
   it('should generate relative paths to the project root', async () => {