Treat dumb terminals as noninteractive (#5237)

* Treat dumb terminals as noninteractive

* Add basic changelog entry

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ### Fixes
 
+* `[jest-cli]` Treat dumb terminals as noninteractive ([#5237](https://github.com/facebook/jest/pull/5237))
 * `[jest-cli]` `jest --onlyChanged --changedFilesWithAncestor` now also works
   with git. ([#5189](https://github.com/facebook/jest/pull/5189))
 * `[jest-config]` fix unexpected condition to avoid infinite recursion in

packages/jest-util/src/__tests__/is_interactive.test.js

@@ -1,17 +1,21 @@
 let oldIsTTY;
+let oldTERM;
 
 beforeEach(() => {
   oldIsTTY = process.stdout.isTTY;
+  oldTERM = process.env.TERM;
 });
 
 afterEach(() => {
   process.stdout.isTTY = oldIsTTY;
+  process.env.TERM = oldTERM;
   jest.resetModules();
 });
 
 it('Returns true when running on interactive environment', () => {
   jest.doMock('is-ci', () => false);
   process.stdout.isTTY = true;
+  process.env.TERM = 'xterm-256color';
 
   const isInteractive = require('../is_interative').default;
   expect(isInteractive).toBe(true);
@@ -24,20 +28,31 @@ it('Returns false when running on a non-interactive environment', () => {
   // Test with is-ci being true and isTTY false
   jest.doMock('is-ci', () => true);
   process.stdout.isTTY = false;
+  process.env.TERM = 'xterm-256color';
   isInteractive = require('../is_interative').default;
   expect(isInteractive).toBe(expectedResult);
 
   // Test with is-ci being false and isTTY false
   jest.resetModules();
   jest.doMock('is-ci', () => false);
   process.stdout.isTTY = false;
+  process.env.TERM = 'xterm-256color';
   isInteractive = require('../is_interative').default;
   expect(isInteractive).toBe(expectedResult);
 
   // Test with is-ci being true and isTTY true
   jest.resetModules();
   jest.doMock('is-ci', () => true);
   process.stdout.isTTY = true;
+  process.env.TERM = 'xterm-256color';
+  isInteractive = require('../is_interative').default;
+  expect(isInteractive).toBe(expectedResult);
+
+  // Test with dumb terminal
+  jest.resetModules();
+  jest.doMock('is-ci', () => false);
+  process.stdout.isTTY = false;
+  process.env.TERM = 'dumb';
   isInteractive = require('../is_interative').default;
   expect(isInteractive).toBe(expectedResult);
 });

packages/jest-util/src/is_interative.js

@@ -1,3 +1,3 @@
 import isCI from 'is-ci';
 
-export default process.stdout.isTTY && !isCI;
+export default process.stdout.isTTY && process.env.TERM !== 'dumb' && !isCI;