From b096c165185a34813264711f85fbf7ecdb567be9 Mon Sep 17 00:00:00 2001
From: Jo Pearce <jo@jopearce.co.uk>
Date: Sat, 26 Oct 2019 08:47:59 +0100
Subject: [PATCH] feat(jest-config): merge preset globals with project globals
 (#9027)

---
 CHANGELOG.md                                  |  1 +
 .../src/__tests__/normalize.test.js           | 56 +++++++++++++++++++
 packages/jest-config/src/normalize.ts         | 15 +++++
 3 files changed, 72 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 507920b30aed..5cb084c4a8bf 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -9,6 +9,7 @@
 - `[expect, jest-matcher-utils]` Display change counts in annotation lines ([#9035](https://github.com/facebook/jest/pull/9035))
 - `[jest-config]` Throw the full error message and stack when a Jest preset is missing a dependency ([#8924](https://github.com/facebook/jest/pull/8924))
 - `[jest-config]` [**BREAKING**] Set default display name color based on runner ([#8689](https://github.com/facebook/jest/pull/8689))
+- `[jest-config]` Merge preset globals with project globals ([#9027](https://github.com/facebook/jest/pull/9027))
 - `[jest-diff]` Add options for colors and symbols ([#8841](https://github.com/facebook/jest/pull/8841))
 - `[jest-diff]` [**BREAKING**] Export as ECMAScript module ([#8873](https://github.com/facebook/jest/pull/8873))
 - `[jest-diff]` Add `includeChangeCounts` and rename `Indicator` options ([#8881](https://github.com/facebook/jest/pull/8881))
diff --git a/packages/jest-config/src/__tests__/normalize.test.js b/packages/jest-config/src/__tests__/normalize.test.js
index 9f085294e276..b1c035f0ce91 100644
--- a/packages/jest-config/src/__tests__/normalize.test.js
+++ b/packages/jest-config/src/__tests__/normalize.test.js
@@ -1219,6 +1219,62 @@ describe('preset', () => {
   });
 });
 
+describe('preset with globals', () => {
+  beforeEach(() => {
+    const Resolver = require('jest-resolve');
+    Resolver.findNodeModule = jest.fn(name => {
+      if (name === 'global-foo/jest-preset') {
+        return '/node_modules/global-foo/jest-preset.json';
+      }
+
+      return '/node_modules/' + name;
+    });
+    jest.doMock(
+      '/node_modules/global-foo/jest-preset.json',
+      () => ({
+        globals: {
+          config: {
+            hereToStay: 'This should stay here',
+          },
+        },
+      }),
+      {virtual: true},
+    );
+  });
+
+  afterEach(() => {
+    jest.dontMock('/node_modules/global-foo/jest-preset.json');
+  });
+
+  test('should merge the globals preset correctly', () => {
+    const {options} = normalize(
+      {
+        preset: 'global-foo',
+        rootDir: '/root/path/foo',
+        globals: {
+          textValue: 'This is just text',
+          config: {
+            sideBySide: 'This should also live another day',
+          },
+        },
+      },
+      {},
+    );
+
+    expect(options).toEqual(
+      expect.objectContaining({
+        globals: {
+          textValue: 'This is just text',
+          config: {
+            hereToStay: 'This should stay here',
+            sideBySide: 'This should also live another day',
+          },
+        },
+      }),
+    );
+  });
+});
+
 describe('preset without setupFiles', () => {
   let Resolver;
   beforeEach(() => {
diff --git a/packages/jest-config/src/normalize.ts b/packages/jest-config/src/normalize.ts
index fea4d8da24f1..391a78e22385 100644
--- a/packages/jest-config/src/normalize.ts
+++ b/packages/jest-config/src/normalize.ts
@@ -73,6 +73,20 @@ const mergeTransformWithPreset = (
   }
 };
 
+const mergeGlobalsWithPreset = (
+  options: Config.InitialOptions,
+  preset: Config.InitialOptions,
+) => {
+  if (options['globals'] && preset['globals']) {
+    for (const p in preset['globals']) {
+      options['globals'][p] = {
+        ...preset['globals'][p],
+        ...options['globals'][p],
+      };
+    }
+  }
+};
+
 const setupPreset = (
   options: Config.InitialOptions,
   optionsPreset: string,
@@ -149,6 +163,7 @@ const setupPreset = (
   }
   mergeModuleNameMapperWithPreset(options, preset);
   mergeTransformWithPreset(options, preset);
+  mergeGlobalsWithPreset(options, preset);
 
   return {...preset, ...options};
 };