WIP - updating test to fail on Windows... and fixing it

Author: weaverryan

lib/config/validator.js

@@ -51,7 +51,12 @@ class Validator {
         }
 
         if (!this.webpackConfig.manifestKeyPrefix) {
-            const outputPath = this.webpackConfig.outputPath.replace(/\/$/, '');
+            let outputPath = this.webpackConfig.outputPath;
+            // for comparison purposes, change \ to / on Windows
+            outputPath = outputPath.split('\\').join('/');
+
+            // remove trailing slash on each
+            outputPath = outputPath.replace(/\/$/, '');
             const publicPath = this.webpackConfig.publicPath.replace(/\/$/, '');
 
             /*

test/config/validator.js

@@ -55,7 +55,7 @@ describe('The validator function', () => {
 
     it('manifestKeyPrefix is correctly not required on windows', () => {
         const config = createConfig();
-        config.outputPath = '/some/path/to/build';
+        config.outputPath = 'C:\\projects\\webpack-encore\\web\\build';
         config.setPublicPath('/build/');
         config.addEntry('main', './main');