Use path.join instead of path.resolve to support bundle names sta…

…rting with `/`

.eslintrc.json

@@ -1,7 +1,7 @@
 {
   "root": true,
   "parserOptions": {
-    "ecmaVersion": 2015,
+    "ecmaVersion": 2017,
     "sourceType": "module",
     "ecmaFeatures": {
       "impliedStrict": true,

package.json

@@ -43,21 +43,25 @@
     "opener": "^1.4.2"
   },
   "devDependencies": {
-    "babel-core": "^6.18.0",
-    "babel-plugin-transform-runtime": "^6.15.0",
-    "babel-polyfill": "^6.16.0",
-    "babel-preset-es2015": "^6.18.0",
-    "babel-preset-stage-2": "^6.18.0",
-    "chai": "^3.5.0",
-    "del": "^2.2.2",
-    "eslint": "^3.8.1",
-    "gulp": "^3.9.1",
-    "gulp-babel": "^6.1.2",
-    "gulp-plumber": "^1.1.0",
-    "gulp-util": "^3.0.7",
-    "gulp-watch": "^4.3.10",
-    "mocha": "^3.1.2",
-    "stream-combiner2": "^1.1.1"
+    "babel-core": "6.18.0",
+    "babel-plugin-transform-runtime": "6.15.0",
+    "babel-polyfill": "6.16.0",
+    "babel-preset-es2015": "6.18.0",
+    "babel-preset-stage-2": "6.18.0",
+    "chai": "3.5.0",
+    "chai-subset": "1.3.0",
+    "del": "2.2.2",
+    "eslint": "3.8.1",
+    "gulp": "3.9.1",
+    "gulp-babel": "6.1.2",
+    "gulp-plumber": "1.1.0",
+    "gulp-util": "3.0.7",
+    "gulp-watch": "4.3.10",
+    "mocha": "3.1.2",
+    "nightmare": "2.8.1",
+    "sinon": "1.17.6",
+    "stream-combiner2": "1.1.1",
+    "webpack": "1.13.3"
   },
   "keywords": [
     "webpack",

src/analyzer.js

@@ -23,7 +23,7 @@ function getChartData(bundleStats, bundleDir) {
   if (bundleDir) {
     // Checking if all assets are exist
     const bundleScriptsFound = _.every(bundleStats.assets, statAsset => {
-      const assetFile = path.resolve(bundleDir, statAsset.name);
+      const assetFile = path.join(bundleDir, statAsset.name);
       const assetExists = fs.existsSync(assetFile);
 
       if (!assetExists) {
@@ -40,7 +40,7 @@ function getChartData(bundleStats, bundleDir) {
       // Parsing assets and getting real module sizes
       parsedModuleSizes = _.transform(bundleStats.assets, (result, statAsset) => {
         _.assign(result,
-          getModuleSizesFromBundle(path.resolve(bundleDir, statAsset.name))
+          getModuleSizesFromBundle(path.join(bundleDir, statAsset.name))
         );
       }, {});
     }

test/.eslintrc.json

@@ -1,6 +1,13 @@
 {
   "extends": "../.eslintrc.json",
   "env": {
-    "mocha": true
+    "mocha": true,
+    "browser": true
+  },
+  "globals": {
+    "sinon": true,
+    "expect": true,
+    "makeWebpackConfig": true,
+    "webpackCompile": true
   }
 }

test/.gitignore

@@ -0,0 +1 @@
+output

test/helpers.js

@@ -0,0 +1,42 @@
+const chai = require('chai');
+const webpack = require('webpack');
+
+chai.use(require('chai-subset'));
+
+global.expect = chai.expect;
+global.sinon = require('sinon');
+global.webpackCompile = webpackCompile;
+global.makeWebpackConfig = makeWebpackConfig;
+
+const BundleAnalyzerPlugin = require('../lib/BundleAnalyzerPlugin');
+
+function webpackCompile(config) {
+  return new Promise((resolve, reject) => {
+    webpack(config, err => {
+      if (err) return reject(err);
+      resolve();
+    });
+  });
+}
+
+function makeWebpackConfig(opts) {
+  opts = {
+    analyzerOpts: {
+      analyzerMode: 'static',
+      openAnalyzer: false
+    },
+    ...opts
+  };
+
+  return {
+    context: __dirname,
+    entry: './src',
+    output: {
+      path: `${__dirname}/output`,
+      filename: 'bundle.js'
+    },
+    plugins: [
+      new BundleAnalyzerPlugin(opts.analyzerOpts)
+    ]
+  };
+}

test/parseUtils.js

@@ -1,7 +1,6 @@
 const fs = require('fs');
 
 const _ = require('lodash');
-const { expect } = require('chai');
 const { getModuleSizesFromBundle } = require('../lib/parseUtils');
 
 const BUNDLES_DIR = `${__dirname}/bundles`;

test/src/a-clone.js

@@ -0,0 +1 @@
+module.exports = 'module a';

test/src/a.js

@@ -0,0 +1 @@
+module.exports = 'module a';

test/src/b.js

@@ -0,0 +1 @@
+module.exports = 'module b';

test/src/index.js

@@ -0,0 +1,3 @@
+require('./a');
+require('./b');
+require('./a-clone');

test/webpack-configs.js

@@ -0,0 +1,38 @@
+const fs = require('fs');
+const del = require('del');
+const Nightmare = require('nightmare');
+const nightmare = Nightmare();
+
+describe('Webpack config', function () {
+  before(function () {
+    del.sync(`${__dirname}/output`);
+    this.clock = sinon.useFakeTimers();
+  });
+
+  afterEach(function () {
+    del.sync(`${__dirname}/output`);
+  });
+
+  after(function () {
+    this.clock.restore();
+  });
+
+  it('with head slash in bundle filename should be supported', async function () {
+    const config = makeWebpackConfig();
+
+    config.output.filename = '/bundle.js';
+
+    await webpackCompile(config);
+    this.clock.tick(1000);
+
+    expect(fs.existsSync(`${__dirname}/output/bundle.js`)).to.be.true;
+    expect(fs.existsSync(`${__dirname}/output/report.html`)).to.be.true;
+    const chartData = await nightmare
+      .goto(`file://${__dirname}/output/report.html`)
+      .evaluate(() => window.chartData);
+    expect(chartData[0]).to.containSubset({
+      parsedSize: 213,
+      statSize: 141
+    });
+  });
+});