Export CI=true does not execute tests in batch mode

[stefaneg]

If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.

Description

Trying to execute test in CI environment.

Expected behavior

Tests should run and immediately terminate.

Actual behavior

Tests run, but wait for changes/user input.

Environment

npm ls react-scripts
client@0.1.0 /Users/gulli/src/ru/reference-tictactoe-student/client
└── react-scripts@0.7.0

node -v
v6.9.1

npm -v
4.0.3

Then, specify:

OSX Sierra
Browser irrelevant.

Reproducible Demo

clone https://github.com/hgop/reference-tictactoe
cd client
npm install
export CI=true
npm test

Should run tests and exit immediately since CI variable is true according to docs here:

https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/template/README.md#continuous-integration

[existentialism]

@stefaneg thanks for the issue!

The CI variable was introduced in react-scripts v0.8.0, so please try upgrading.

[gaearon]

Hmm I think CI variable itself was supported since we added the test runner.

[stefaneg]

Still does not work for me. Tried 0.8.0 and 0.8.4.

client@0.1.0 /Users/gulli/src/ru/reference-tictactoe-student/client
└── react-scripts@0.8.4

I'm getting the information that this is working for some of my students. I'll post an update when I know more details.

[gaearon]

I see you have --watchAll in your test command, it probably overrides CI=true.

[existentialism]

Yeah, my mistake.

[stefaneg]

Yeah, I just realized that. Thank you guys.

[existentialism]

Glad it's solved, sorry again for leading you slightly astray :)