Exit with error if --watch is used in CI

(Rather than ignoring --watch we’ve decided to exit with an error instead: https://github.com/avajs/ava/issues/1186#issuecomment-274246060 — @novemberborn)

---

Description

This is a feature request. Skip ‘–watch’ arg if isCi

Since we have known our test running on CI server. Ava should more clever to ignore --watch arg.

Config

Copy the relevant section from package.json:

{
  "scripts": {
    "test": "ava -w"
  }
}

Now I write my test command like this:

{
  "scripts": {
    "test": "[ -z $CI ] && (ava -w); [ $CI ] && (ava)"
  }
}

[forresst]

To summarize the discussion, the solution would be to raise an error (exiting with 1 to allow stop the process in CI) with the following explanatory message : AVA will not run with the --watch (-w) option in CI, because CI processes should terminate, and with the --watch option, AVA will never terminate.

Is that the right solution?

I can do the PR if you want (Code, test and explanation in docs)

[mightyiam]

Yes. “Skipping” watching, with a warning, seems reasonable. But, if anyone is running with -w in CI it most likely means that his test script has ava -w. Shouldn’t we “educate” by not letting that pass, and instead provide a kind message?

“Error: AVA will not run with the --watch (-w) option in CI, because CI processes should terminate, and with the --watch option, AVA will never terminate.”