improved error reporting for mandatory configuration

in the process, updated dependencies

Author: FND

lib/index.js

@@ -1,6 +1,7 @@
 "use strict";
 
 let Bundle = require("./bundle");
+let { abort, repr } = require("faucet-pipeline/lib/util")
 let path = require("path");
 
 module.exports = (config, assetManager, { watcher, browsers, compact } = {}) => {
@@ -56,6 +57,10 @@ function makeBundle(config, resolvePath, { browsers }) {
 	// dissect configuration for constructor
 	config = Object.assign({}, config);
 	let [entryPoint, target] = extract(config, "source", "target");
+	if(!entryPoint || !target) {
+		let setting = entryPoint ? "target" : "source";
+		abort(`ERROR: missing ${repr(setting, false)} configuration`);
+	}
 
 	entryPoint = resolvePath(entryPoint);
 	target = resolvePath(target, { enforceRelative: true });

package.json

@@ -27,7 +27,7 @@
 	},
 	"dependencies": {
 		"faucet-pipeline": "~1.0.0-rc.7",
-		"rollup": "~0.61.2",
+		"rollup": "~0.62.0",
 		"rollup-plugin-cleanup": "~3.0.0-beta.1",
 		"rollup-plugin-commonjs": "~9.1.3",
 		"rollup-plugin-node-resolve": "~3.3.0"

pkg/faucet-pipeline-typescript/package.json

@@ -17,7 +17,7 @@
 	},
 	"dependencies": {
 		"faucet-pipeline-js": "1.0.0-rc.6",
-		"rollup-plugin-typescript2": "~0.15.0",
+		"rollup-plugin-typescript2": "~0.15.1",
 		"typescript": "~2.9.2"
 	}
 }

test/unit/test_bundling.js

@@ -21,6 +21,16 @@ describe("bundling", _ => {
 		process.exit = exit;
 	});
 
+	it("should verify configuration", () => {
+		let config = [{}];
+		let assetManager = new MockAssetManager(FIXTURES_DIR);
+
+		let fn = _ => faucetJS(config, assetManager, DEFAULT_OPTIONS);
+		assert.throws(fn, /exit 1/); // aborts with "missing `source` configuration"
+		config[0].source = "./src/index.js";
+		assert.throws(fn, /exit 1/); // aborts with "missing `target` configuration"
+	});
+
 	it("should combine ES6 modules into a bundle", () => {
 		let config = [{
 			source: "./src/index.js",