chore(test): add --single-run option (angular#2841)

filipesilva · hansl · commit 1e50e5147e59 · 2016-11-09T02:41:26.000+01:00
Doing `ng test --single-run` always worked but outputted a ember-cli warning about a missing option.

This PR allows using the option without warning. It's overall better than `--watch=false` because it better aligns with Karma's options.

`--watch=false` still works though.
diff --git a/README.md b/README.md
@@ -185,7 +185,7 @@ or `ng serve --prod` will also make use of uglifying and tree-shaking functional
 ng test
 ```
 
-Tests will execute after a build is executed via [Karma](http://karma-runner.github.io/0.13/index.html), and it will automatically watch your files for changes. You can run tests a single time via `--watch=false`.
+Tests will execute after a build is executed via [Karma](http://karma-runner.github.io/0.13/index.html), and it will automatically watch your files for changes. You can run tests a single time via `--watch=false` or `--single-run`.
 
 ### Running end-to-end tests
 
diff --git a/packages/angular-cli/commands/test.ts b/packages/angular-cli/commands/test.ts
@@ -7,6 +7,7 @@ const NgCliTestCommand = TestCommand.extend({
     { name: 'watch', type: Boolean, default: true, aliases: ['w'] },
     { name: 'code-coverage', type: Boolean, default: false, aliases: ['cc'] },
     { name: 'lint', type: Boolean, default: false, aliases: ['l'] },
+    { name: 'single-run', type: Boolean, default: false, aliases: ['sr'] },
     { name: 'browsers', type: String },
     { name: 'colors', type: Boolean },
     { name: 'log-level', type: String },
diff --git a/tests/e2e/tests/generate/class.ts b/tests/e2e/tests/generate/class.ts
@@ -12,5 +12,5 @@ export default function() {
     .then(() => expectFileToExist(join(classDir, 'test-class.spec.ts')))
 
     // Try to run the unit tests.
-    .then(() => ng('test', '--watch=false'));
+    .then(() => ng('test', '--single-run'));
 }
diff --git a/tests/e2e/tests/generate/component/component-basic.ts b/tests/e2e/tests/generate/component/component-basic.ts
@@ -14,5 +14,5 @@ export default function() {
     .then(() => expectFileToExist(join(componentDir, 'test-component.component.css')))
 
     // Try to run the unit tests.
-    .then(() => ng('test', '--watch=false'));
+    .then(() => ng('test', '--single-run'));
 }
diff --git a/tests/e2e/tests/generate/interface.ts b/tests/e2e/tests/generate/interface.ts
@@ -11,5 +11,5 @@ export default function() {
     .then(() => expectFileToExist(join(interfaceDir, 'test-interface.model.ts')))
 
     // Try to run the unit tests.
-    .then(() => ng('test', '--watch=false'));
+    .then(() => ng('test', '--single-run'));
 }
diff --git a/tests/e2e/tests/generate/module/module-basic.ts b/tests/e2e/tests/generate/module/module-basic.ts
@@ -17,5 +17,5 @@ export default function() {
     .then(() => expectFileToExist(join(moduleDir, 'test-module.component.css')))
 
     // Try to run the unit tests.
-    .then(() => ng('test', '--watch=false'));
+    .then(() => ng('test', '--single-run'));
 }
diff --git a/tests/e2e/tests/generate/module/module-routing.ts b/tests/e2e/tests/generate/module/module-routing.ts
@@ -16,5 +16,5 @@ export default function() {
     .then(() => expectFileToExist(join(moduleDir, 'test-module.component.css')))
 
     // Try to run the unit tests.
-    .then(() => ng('test', '--watch=false'));
+    .then(() => ng('test', '--single-run'));
 }
diff --git a/tests/e2e/tests/generate/pipe.ts b/tests/e2e/tests/generate/pipe.ts
@@ -13,5 +13,5 @@ export default function() {
     .then(() => expectFileToExist(join(pipeDir, 'test-pipe.pipe.spec.ts')))
 
     // Try to run the unit tests.
-    .then(() => ng('test', '--watch=false'));
+    .then(() => ng('test', '--single-run'));
 }
diff --git a/tests/e2e/tests/generate/service.ts b/tests/e2e/tests/generate/service.ts
@@ -13,5 +13,5 @@ export default function() {
     .then(() => expectFileToExist(join(serviceDir, 'test-service.service.spec.ts')))
 
     // Try to run the unit tests.
-    .then(() => ng('test', '--watch=false'));
+    .then(() => ng('test', '--single-run'));
 }
diff --git a/tests/e2e/tests/misc/coverage.ts b/tests/e2e/tests/misc/coverage.ts
@@ -3,7 +3,7 @@ import {ng} from '../../utils/process';
 
 
 export default function() {
-  return ng('test', '--watch=false', '--code-coverage')
+  return ng('test', '--single-run', '--code-coverage')
     .then(() => expectFileToExist('coverage/src/app'))
     .then(() => expectFileToExist('coverage/coverage.lcov'));
 }
diff --git a/tests/e2e/tests/test/test.ts b/tests/e2e/tests/test/test.ts
@@ -2,5 +2,7 @@ import {ng} from '../../utils/process';
 
 
 export default function() {
-  return ng('test', '--watch=false');
+  // make sure both --watch=false and --single-run work
+  return ng('test', '--single-run')
+    .then(() => ng('test', '--watch=false'));
 }

Original file line number	Diff line number	Diff line change
`@@ -12,5 +12,5 @@ export default function() {`
`12`	`12`	`.then(() => expectFileToExist(join(classDir, 'test-class.spec.ts')))`
`13`	`13`
`14`	`14`	`// Try to run the unit tests.`
`15`		`- .then(() => ng('test', '--watch=false'));`
	`15`	`+ .then(() => ng('test', '--single-run'));`
`16`	`16`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,5 +14,5 @@ export default function() {`
`14`	`14`	`.then(() => expectFileToExist(join(componentDir, 'test-component.component.css')))`
`15`	`15`
`16`	`16`	`// Try to run the unit tests.`
`17`		`- .then(() => ng('test', '--watch=false'));`
	`17`	`+ .then(() => ng('test', '--single-run'));`
`18`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -11,5 +11,5 @@ export default function() {`
`11`	`11`	`.then(() => expectFileToExist(join(interfaceDir, 'test-interface.model.ts')))`
`12`	`12`
`13`	`13`	`// Try to run the unit tests.`
`14`		`- .then(() => ng('test', '--watch=false'));`
	`14`	`+ .then(() => ng('test', '--single-run'));`
`15`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,5 +17,5 @@ export default function() {`
`17`	`17`	`.then(() => expectFileToExist(join(moduleDir, 'test-module.component.css')))`
`18`	`18`
`19`	`19`	`// Try to run the unit tests.`
`20`		`- .then(() => ng('test', '--watch=false'));`
	`20`	`+ .then(() => ng('test', '--single-run'));`
`21`	`21`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,5 +16,5 @@ export default function() {`
`16`	`16`	`.then(() => expectFileToExist(join(moduleDir, 'test-module.component.css')))`
`17`	`17`
`18`	`18`	`// Try to run the unit tests.`
`19`		`- .then(() => ng('test', '--watch=false'));`
	`19`	`+ .then(() => ng('test', '--single-run'));`
`20`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,5 @@ export default function() {`
`13`	`13`	`.then(() => expectFileToExist(join(pipeDir, 'test-pipe.pipe.spec.ts')))`
`14`	`14`
`15`	`15`	`// Try to run the unit tests.`
`16`		`- .then(() => ng('test', '--watch=false'));`
	`16`	`+ .then(() => ng('test', '--single-run'));`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@ import {ng} from '../../utils/process';`
`3`	`3`
`4`	`4`
`5`	`5`	`export default function() {`
`6`		`- return ng('test', '--watch=false', '--code-coverage')`
	`6`	`+ return ng('test', '--single-run', '--code-coverage')`
`7`	`7`	`.then(() => expectFileToExist('coverage/src/app'))`
`8`	`8`	`.then(() => expectFileToExist('coverage/coverage.lcov'));`
`9`	`9`	`}`