Merge pull request #2 from jsilvax/dev-branch

Merged dev-branch into master

Author: jsilvax

.gitignore

@@ -5,4 +5,5 @@ test/fixtures/**/dir
 .coveralls.yml
 *.log
 /dist
-.vscode
+.vscode/
+.yarn-error.log

__tests__/css-entry-array.test.js

@@ -0,0 +1,67 @@
+const webpack = require('webpack');
+const path = require('path');
+const rimraf = require('rimraf');
+const options = require('./fixtures/css-array-entry/webpack.config.js');
+const optionsSass = require('./fixtures/css-array-entry/webpack.config.sass');
+const optionsLess = require('./fixtures/css-array-entry/webpack.config.less');
+const dirPath = path.join(__dirname, './fixtures/css-array-entry/dir');
+const fileShouldNotExist = require('./utils/file-should-not-exist.js');
+
+describe('Array of CSS Dependencies as Entry', () => {
+	beforeEach(done => {
+		rimraf(dirPath, () => {
+			done();
+		});
+	});
+
+	it('JS entry should not exist', done => {
+		webpack(options, () => {
+			fileShouldNotExist(dirPath, '/a.js');
+			done();
+		});
+	});
+
+	it('JS entry source map should not exist', done => {
+		const optionSourceMap = Object.assign({}, options);
+		optionSourceMap.devtool = 'source-map';
+
+		webpack(optionSourceMap, () => {
+			fileShouldNotExist(dirPath, '/a.js.map');
+			done();
+		});
+	});
+
+	it('JS entry should not exist w/ sass', done => {
+		webpack(optionsSass, () => {
+			fileShouldNotExist(dirPath, '/s.js');
+			done();
+		});
+	});
+
+	it('JS entry source map should not exist w/ sass', done => {
+		const optionSourceMap = Object.assign({}, options);
+		optionSourceMap.devtool = 'source-map';
+
+		webpack(optionSourceMap, () => {
+			fileShouldNotExist(dirPath, '/s.js.map');
+			done();
+		});
+	});	
+
+	it('JS entry should not exist w/ less', done => {
+		webpack(optionsLess, () => {
+			fileShouldNotExist(dirPath, '/l.js');
+			done();
+		});
+	});
+
+	it('JS entry source map should not exist w/ less', done => {
+		const optionSourceMap = Object.assign({}, optionsLess);
+		optionSourceMap.devtool = 'source-map';
+
+		webpack(optionSourceMap, () => {
+			fileShouldNotExist(dirPath, '/l.js.map');
+			done();
+		});
+	});	
+});

__tests__/fixtures/css-array-entry/dir/l.css

@@ -0,0 +1,6 @@
+@nice-blue: #5B83AD;
+@light-blue: @nice-blue + #111;
+
+#header {
+  color: @light-blue;
+}

__tests__/fixtures/css-array-entry/dir/l.css.map

@@ -0,0 +1,6 @@
+$font-stack:    Helvetica, sans-serif
+$primary-color: #333
+
+body
+  font: 100% $font-stack
+  color: $primary-color

test/fixtures/css-array-entry/one.css renamed to __tests__/fixtures/css-array-entry/one.css

@@ -0,0 +1,5 @@
+@nice-yellow: yellow;
+
+.duck {
+  color: @nice-yellow;
+}

__tests__/fixtures/css-array-entry/one.less

@@ -0,0 +1,11 @@
+$font-stack:    Helvetica, sans-serif;
+$primary-color: #333;
+
+body {
+  font: 100% $font-stack;
+  color: $primary-color;
+}
+
+.green {
+	color : green;
+}

__tests__/fixtures/css-array-entry/three.sass

@@ -0,0 +1,28 @@
+const MiniCssExtractPlugin = require("mini-css-extract-plugin");
+const OmitJSforCSSPlugin = require('../../../src/index.js');
+const path = require('path');
+
+module.exports = {
+	entry: {
+		l: [path.join(__dirname, 'one.less'), path.join(__dirname, 'two.less')]
+	},
+	output: {
+		filename: '[name].js',
+		path: path.join(__dirname, '/dir')
+	},
+	module: {
+		rules: [
+			{
+				test: /\.(less)$/,
+				use: [
+					MiniCssExtractPlugin.loader,
+					'css-loader',
+					'less-loader'
+				]
+			}
+		]
+	},
+	mode: 'development',
+	plugins: [new MiniCssExtractPlugin({ filename: '[name].css' }), new OmitJSforCSSPlugin()],
+	stats: 'none'
+};

test/fixtures/css-array-entry/two.css renamed to __tests__/fixtures/css-array-entry/two.css

@@ -0,0 +1,29 @@
+const MiniCssExtractPlugin = require("mini-css-extract-plugin");
+const OmitJSforCSSPlugin = require('../../../src/index.js');
+const path = require('path');
+
+module.exports = {
+	entry: {
+		s: [path.join(__dirname, 'two.scss'), path.join(__dirname, 'three.sass')]
+	},
+	output: {
+		filename: '[name].js',
+		path: path.join(__dirname, '/dir')
+	},
+	module: {
+		rules: [
+			{
+				test: /\.(scss|sass)$/,
+				use: [
+					MiniCssExtractPlugin.loader,
+					'css-loader',
+					'sass-loader'
+				]
+			}
+		]
+	},
+	plugins: [new MiniCssExtractPlugin({ filename: '[name].css' }), new OmitJSforCSSPlugin()],
+	mode : 'development',
+	devtool: 'source-map',
+	stats: 'none'
+};

__tests__/fixtures/css-array-entry/two.less

@@ -0,0 +1,6 @@
+@nice-blue: #5B83AD;
+@light-blue: @nice-blue + #111;
+
+#header {
+  color: @light-blue;
+}

__tests__/fixtures/css-array-entry/two.scss

@@ -0,0 +1,2 @@
+require('./one.less');
+require('./two.less');

test/fixtures/css-array-entry/webpack.config.js renamed to __tests__/fixtures/css-array-entry/webpack.config.js

@@ -0,0 +1,2 @@
+require('./two.scss');
+require('./three.sass');

__tests__/fixtures/css-array-entry/webpack.config.less.js

@@ -0,0 +1,6 @@
+$font-stack:    Helvetica, sans-serif
+$primary-color: #333
+
+body
+  font: 100% $font-stack
+  color: $primary-color

__tests__/fixtures/css-array-entry/webpack.config.sass.js

@@ -0,0 +1,5 @@
+@nice-yellow: yellow;
+
+.duck {
+  color: @nice-yellow;
+}

__tests__/fixtures/js-file-entry/dir/s.css

@@ -0,0 +1,11 @@
+$font-stack:    Helvetica, sans-serif;
+$primary-color: #333;
+
+body {
+  font: 100% $font-stack;
+  color: $primary-color;
+}
+
+.green {
+	color : green;
+}

__tests__/fixtures/js-file-entry/dir/s.css.map

@@ -0,0 +1,28 @@
+const MiniCssExtractPlugin = require("mini-css-extract-plugin");
+const OmitJSforCSSPlugin = require('../../../src/index.js');
+const path = require('path');
+
+module.exports = {
+	entry: {
+		l: path.join(__dirname, 'styles-less.js')
+	},
+	output: {
+		filename: '[name].js',
+		path: path.join(__dirname, '/dir')
+	},
+	module: {
+		rules: [
+			{
+				test: /\.(less)$/,
+				use: [
+					MiniCssExtractPlugin.loader,
+					'css-loader',
+					'less-loader'
+				]
+			}
+		]
+	},
+	mode: 'development',
+	plugins: [new MiniCssExtractPlugin({ filename: '[name].css' }), new OmitJSforCSSPlugin()],
+	stats: 'none'
+};

test/fixtures/js-file-entry/one.css renamed to __tests__/fixtures/js-file-entry/one.css

@@ -0,0 +1,29 @@
+const MiniCssExtractPlugin = require("mini-css-extract-plugin");
+const OmitJSforCSSPlugin = require('../../../src/index.js');
+const path = require('path');
+
+module.exports = {
+	entry: {
+		s: path.join(__dirname, 'styles-sass.js')
+	},
+	output: {
+		filename: '[name].js',
+		path: path.join(__dirname, '/dir')
+	},
+	module: {
+		rules: [
+			{
+				test: /\.(scss|sass)$/,
+				use: [
+					MiniCssExtractPlugin.loader,
+					'css-loader',
+					'sass-loader'
+				]
+			}
+		]
+	},
+	plugins: [new MiniCssExtractPlugin({ filename: '[name].css' }), new OmitJSforCSSPlugin()],
+	mode : 'development',
+	devtool: 'source-map',
+	stats: 'none'
+};