temporarily only running 1 test with debugging for Windows

Author: weaverryan

fixtures/js/import_node_modules_image.js

@@ -1,7 +1,7 @@
 // this helps us trigger a manifest.json bug
 // https://github.com/shellscape/webpack-manifest-plugin/pull/249
 import 'mocha/assets/growl/ok.png';
-import '../images/symfony_logo.png';
+//import '../images/symfony_logo.png';
 
 
 // module.userRequest

lib/tmpLogger.js

@@ -0,0 +1,25 @@
+/*
+ * This file is part of the Symfony Webpack Encore package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+'use strict';
+
+const fs = require('fs');
+const path = require('path');
+
+module.exports = function tmpLogger(data) {
+    const logPath = path.join(__dirname, '../', 'log.txt');
+    let contents = '';
+    if (fs.existsSync(logPath)) {
+        contents = fs.readFileSync(logPath, 'utf8');
+        contents += '\n\n';
+    }
+
+    contents += JSON.stringify(data);
+    fs.writeFileSync(logPath, contents);
+};

lib/webpack-manifest-plugin/helpers.js

@@ -1,4 +1,5 @@
 const { dirname, join } = require('path');
+const tmpLogger = require('../tmpLogger');
 
 const generateManifest = (compilation, files, { generate, seed = {} }) => {
   let result;
@@ -27,6 +28,10 @@ const getFileType = (fileName, { transformExtensions }) => {
 };
 
 const reduceAssets = (files, asset, moduleAssets, assetTypeModuleAssets) => {
+    tmpLogger({
+        assetTypeModuleAssets,
+        sourceFilename: asset.info.sourceFilename,
+    });
   let name;
   if (moduleAssets[asset.name]) {
     name = moduleAssets[asset.name];

lib/webpack-manifest-plugin/hooks.js

@@ -6,6 +6,8 @@ const webpack = require('webpack');
 // eslint-disable-next-line global-require
 const { RawSource } = webpack.sources || require('webpack-sources');
 
+const tmpLogger = require('../tmpLogger');
+
 const { generateManifest, reduceAssets, reduceChunk, transformFiles } = require('./helpers');
 
 const compilerHookMap = new WeakMap();
@@ -119,14 +121,24 @@ const normalModuleLoaderHook = ({ moduleAssets, assetTypeModuleAssets }, loaderC
   // the "emitFile" callback is never called on asset modules
   // so, we create a different map that can be used later in the "emit" hook
   if (['asset', 'asset/inline', 'asset/resource', 'asset/source'].includes(module.type)) {
-      Object.assign(assetTypeModuleAssets, {
+      const data = {
+          userRequest: module.userRequest,
+          rootContext: loaderContext.rootContext
+      };
+
+      tmpLogger(data);
+
       // This takes the userRequest (which is an absolute path) and turns it into
       // a relative path to the root context. This is done so that the string
       // will match asset.info.sourceFilename in the emit hook.
-      [relative(loaderContext.rootContext, module.userRequest)]: basename(
-        module.userRequest
-      )
-    });
+      let sourceFilename = relative(loaderContext.rootContext, module.userRequest);
+      // at this point, Windows paths use \ in their paths
+      // but in the emit hook, asset.info.sourceFilename fill have UNIX slashes
+      sourceFilename = sourceFilename.replace(/\\/g, '/');
+      tmpLogger({'when': 'after replacing', sourceFilename});
+      Object.assign(assetTypeModuleAssets, {
+        [sourceFilename]: basename(module.userRequest),
+      });
   }
 
   // eslint-disable-next-line no-param-reassign

test/functional.js

@@ -152,7 +152,10 @@ describe('Functional tests using webpack', function() {
             });
         });
 
-        it('Check manifest.json with node_module includes', (done) => {
+        it.only('Check manifest.json with node_module includes', (done) => {
+            after(() => {
+                console.log(fs.readFileSync(path.join(__dirname, '../', 'log.txt'), 'utf8'));
+            });
             const config = createWebpackConfig('web/build', 'dev');
             config.addEntry('main', './js/import_node_modules_image');
             config.setPublicPath('/build');