Update testing commands

README.md

@@ -507,11 +507,19 @@ Configurations:
 
 Command | Testing TypeScript
 ---     | ---
-default | Run through PhantomJS one time with no file watching
-headless (aliases: hl, h)| Run through PhantomJS with files being watched & tests automatically re-run
-watch (alias: w)| Run through Chrome with files being watched & tests automatically re-run
-
-Note that Chrome still requires a manual refresh on the Debug tab to see updated test results.
+default | Run through defined browsers one time with no file watching
+headless (aliases: hl, h)| Run through PhantomJS
+watch (alias: w)| Run through defined browsers with files being watched & tests automatically re-run
+browsers (alias: b)| Overrides the browsers to use for testing. Note: These are applied last and merges 
+will be lost
+
+These commands can be combined allowing you to work the way you need to for a variety of situations. For
+instance using `ngl test watch headless` will run though PhantomJS with files being watched & tests
+automatically re-run. Or if you have defined in your karma config file to run against Chrome, Firefox, IE
+and Edge but there is only an issue with IE you want to work on, then you can simply run 
+`ngl test watch browsers=IE` and only IE will run with files being watched and tests automatically re-run.
+
+Note that browsers still requires a manual refresh on the Debug tab to see updated test results.
 
 ## <a id="custom-config"></a>Custom Configurations
 

commands/initial/templates/tasks/test.js

@@ -13,45 +13,40 @@ function run(type) {
     server.start();
 }
 
-function getConfig(type) {
-    switch (type) {
-        case 'headless':
-        case 'hl':
-        case 'h':
-            return getHeadlessConfig();
-        case 'all':
-        case 'a':
-            return getAllConfig();
-        case 'watch':
-        case 'w':
-            return getWatchConfig();
-        default:
-            return getSingleConfig();
-    }
-}
-
-function getSingleConfig() {
-    let config = getHeadlessConfig();
-
-    config.singleRun = true;
+function getConfig(options) {
+	let config = getAllConfig(options.watch);
+	config.browsers = options.browsers;
+	config.singleRun = !options.watch;
 
-    return config;
+	return config;
 }
 
-function getHeadlessConfig() {
-    let config = getAllConfig();
-
-    config.browsers = ['PhantomJS'];
-
-    return config;
-}
-
-function getWatchConfig() {
-    let config = getAllConfig(true);
-
-    config.browsers = ['Chrome'];
-
-    return config;
+function parseOptions(args){
+	let options = {};
+
+	args.forEach((arg, index)=>{
+		let value = arg.match(/^(\w+)/)[1];
+		switch(value){	//there are probably libraries that parse cmd line arguments...
+			case "headless":
+			case "hl":
+			case "h":
+				options.browsers = ["PhantomJS"];
+				break;
+			case "watch":
+			case "w":
+				options.watch = true;
+				break;
+			case "browsers":
+			case "b":
+				let browsers = arg.match(/\w+=([\w,]*)/i);
+				options.browsers = (browsers && !options.browsers) ? browsers[1].split(',') : options.browsers;
+				break;
+			//the 'all' option did not modify the browser options and it did not change the watch option.
+			//therefore removing it will not break current setups. Unless the developer removed all browsers
+			//from the base karma.config.js file
+		}
+	});
+	return options;
 }
 
 const getAllConfig = (watch) => ({
@@ -62,5 +57,7 @@ const getAllConfig = (watch) => ({
 module.exports = run;
 
 if (!module.parent) {
-    run(process.argv[2]);
+	//skip the first two args (exe and script) and grab all options that start with a 'word'
+	let optionArgs = process.argv.filter((value, index) => index > 1 && value.match(/^\w+/));	
+    run(parseOptions(optionArgs));
 }