Use ci-info to check for GHA

Author: ockham

packages/jest-reporters/package.json

@@ -19,6 +19,7 @@
     "@jest/types": "^28.0.0-alpha.5",
     "@types/node": "*",
     "chalk": "^4.0.0",
+    "ci-info": "^3.2.0",
     "collect-v8-coverage": "^1.0.0",
     "exit": "^0.1.2",
     "glob": "^7.1.2",

packages/jest-reporters/src/GithubActionsReporter.ts

@@ -5,6 +5,7 @@
  * LICENSE file in the root directory of this source tree.
  */
 
+import {GITHUB_ACTIONS} from 'ci-info';
 import type {AggregatedResult, TestResult} from '@jest/test-result';
 import BaseReporter from './BaseReporter';
 import type {Context} from './types';
@@ -18,7 +19,7 @@ export default class GithubActionsReporter extends BaseReporter {
     _contexts?: Set<Context>,
     aggregatedResults?: AggregatedResult,
   ): void {
-    if (!process.env.GITHUB_ACTIONS) {
+    if (!GITHUB_ACTIONS) {
       return;
     }
 

packages/jest-reporters/src/__tests__/GithubActionsReporter.test.js

@@ -8,7 +8,6 @@
 
 let GithubActionsReporter;
 
-const env = {...process.env};
 const write = process.stderr.write;
 const globalConfig = {
   rootDir: 'root',
@@ -29,7 +28,6 @@ beforeEach(() => {
 
 afterEach(() => {
   results = [];
-  process.env = env;
   process.stderr.write = write;
 });
 
@@ -120,7 +118,7 @@ test("reporter returns empty string if GITHUB_ACTIONS isn't set", () => {
 });
 
 test('reporter extracts the correct filename, line, and column', () => {
-  process.env.GITHUB_ACTIONS = true;
+  jest.doMock('ci-info', () => ({GITHUB_ACTIONS: true}));
 
   requireReporter();
   const testReporter = new GithubActionsReporter(globalConfig);

yarn.lock

@@ -2743,6 +2743,7 @@ __metadata:
     "@types/node": "*"
     "@types/node-notifier": ^8.0.0
     chalk: ^4.0.0
+    ci-info: ^3.2.0
     collect-v8-coverage: ^1.0.0
     exit: ^0.1.2
     glob: ^7.1.2