Fix for #321 - don't error if target is named "src"

Author: nycdotnet

tasks/modules/tsconfig.js

@@ -125,13 +125,19 @@ function warnOnBadConfiguration(options: IGruntTSOptions, projectSpec: ITSConfig
 
 function getGlobs(taskOptions: ITargetOptions, targetOptions: ITargetOptions) {
   let globs = null;
-  if (taskOptions && (<any>taskOptions).src) {
+
+  if (taskOptions && isStringOrArray((<any>taskOptions).src)) {
     globs = _.map([...(<any>taskOptions).src], item => templateProcessor(item, {}));
   }
-  if (targetOptions && (<any>targetOptions).src) {
+  if (targetOptions && isStringOrArray((<any>targetOptions).src)) {
     globs = _.map([...(<any>targetOptions).src], item => templateProcessor(item, {}));
   }
+
   return globs;
+
+  function isStringOrArray(thing: any) {
+    return (_.isArray(thing) || _.isString(thing));
+  }
 }
 
 

tasks/modules/tsconfig.ts

@@ -220,6 +220,20 @@ export var tests : nodeunit.ITestGroup = {
           test.ok(allWarnings.indexOf(`keyword "watch"`) > -1, "expected warning about keyword watch");
           test.done();
         }).catch((err) => {test.ifError(err); test.done();});
+    },
+    "Works OK when using grunt keyword (src) as target name": (test: nodeunit.Test) => {
+        test.expect(3);
+
+        const cfg = getConfig("minimalist", true);
+        const tsTaskCfg: any = { "src": cfg };
+        const files: IGruntTSCompilationInfo[] = [{src: ['test/gruntkeyword.ts']}];
+
+        const result = or.resolveAsync(tsTaskCfg, cfg, "src", files).then((result) => {
+          test.strictEqual(result.warnings.length, 0, "expected zero warnings.");
+          test.strictEqual(result.errors.length, 0, "expected zero errors.");
+          test.strictEqual(result.CompilationTasks[0].src.join(""), "test/gruntkeyword.ts", "expected just the test file");
+          test.done();
+        }).catch((err) => {test.ifError(err); test.done();});
     }
   },