fix(plugin-webpack): fix incorrect PRELOAD_WEBPACK_ENTRY.

packages/plugin/webpack/package.json

@@ -8,9 +8,12 @@
   "main": "dist/WebpackPlugin.js",
   "typings": "dist/WebpackPlugin.d.ts",
   "scripts": {
-    "test": "exit 0"
+    "test": "mocha --require ts-node/register test/**/*_spec.ts test/**/**/*_spec.ts --opts ../../../mocha.opts"
+  },
+  "devDependencies": {
+    "chai": "4.2.0",
+    "mocha": "^5.0.0"
   },
-  "devDependencies": {},
   "engines": {
     "node": ">= 6.0"
   },
@@ -30,4 +33,4 @@
     "webpack-hot-middleware": "^2.21.0",
     "webpack-merge": "^4.1.1"
   }
-}
+}

packages/plugin/webpack/src/WebpackPlugin.ts

@@ -24,7 +24,7 @@ const BASE_PORT = 3000;
 
 export default class WebpackPlugin extends PluginBase<WebpackPluginConfig> {
   name = 'webpack';
-  private isProd = false;
+  isProd = false;
   private projectDir!: string;
   private baseDir!: string;
   private watchers: webpack.Compiler.Watching[] = [];
@@ -164,8 +164,8 @@ Your packaged app may be larger than expected if you dont ignore everything othe
       if (entryPoint.preload) {
         defines[`${entryPoint.name.toUpperCase().replace(/ /g, '_')}_PRELOAD_WEBPACK_ENTRY`] =
           this.isProd
-          ? `\`file://\$\{require('path').resolve(__dirname, '../renderer', '${entryPoint.name}', 'preload.js')\}\``
-          : `'${path.resolve(this.baseDir, 'renderer', entryPoint.name, 'preload.js')}'`;
+          ? `require('path').resolve(__dirname, '../renderer', '${entryPoint.name}', 'preload.js')`
+          : `'${path.resolve(this.baseDir, 'renderer', entryPoint.name, 'preload.js').replace(/\\/g, '\\\\')}'`;
       }
     }
     return defines;

packages/plugin/webpack/test/WebpackPlugin_spec.ts

@@ -0,0 +1,55 @@
+import { expect } from 'chai';
+
+import WebpackPlugin from '../src/WebpackPlugin';
+
+describe('WebpackPlugin', () => {
+  describe('PRELOAD_WEBPACK_ENTRY', () => {
+    it('should assign absolute preload script path in development', () => {
+      const p = new WebpackPlugin({
+        mainConfig: {},
+        renderer: {
+          config: {},
+          entryPoints: [
+            {
+              js: 'window.js',
+              name: 'window',
+              preload: {
+                js: 'preload.js',
+              },
+            },
+          ],
+        },
+      });
+      p.init(process.platform === 'win32' ? 'C:\\baseDir' : '/baseDir');
+      p.isProd = false;
+      const defines = p.getDefines();
+
+      if (process.platform === 'win32') {
+        expect(defines.WINDOW_PRELOAD_WEBPACK_ENTRY).to.be.eq("'C:\\\\baseDir\\\\.webpack\\\\renderer\\\\window\\\\preload.js'");
+      } else {
+        expect(defines.WINDOW_PRELOAD_WEBPACK_ENTRY).to.be.eq("'/baseDir/.webpack/renderer/window/preload.js'");
+      }
+    });
+    it('should assign an expression to resolve the preload script in production', () => {
+      const p = new WebpackPlugin({
+        mainConfig: {},
+        renderer: {
+          config: {},
+          entryPoints: [
+            {
+              js: 'window.js',
+              name: 'window',
+              preload: {
+                js: 'preload.js',
+              },
+            },
+          ],
+        },
+      });
+      p.init(process.platform === 'win32' ? 'C:\\baseDir' : '/baseDir');
+      p.isProd = true;
+      const defines = p.getDefines();
+      expect(defines.WINDOW_PRELOAD_WEBPACK_ENTRY).to.be.eq("require('path').resolve(__dirname, '../renderer', 'window', 'preload.js')");
+    });
+  });
+});