Small refactor of Fantom global setup (#52766)

Summary:
Pull Request resolved: #52766

Changelog: [internal]

Just a minor refactor so adding more logic that should work both at Meta and in OSS is easier in the next diff

Reviewed By: christophpurrer

Differential Revision: D78741904

fbshipit-source-id: 3abda5d5b7be157bf381e26dad2fd4b064a0f556

Author: rubennorte

private/react-native-fantom/config/jest.config.js

@@ -13,15 +13,6 @@
 const baseConfig = require('../../../jest.config');
 const path = require('path');
 
-const isCI =
-  Boolean(process.env.SANDCASTLE) || Boolean(process.env.GITHUB_ACTIONS);
-
-// In CI, we want to prewarm the caches/builds before running the tests so
-// that time isn't attributed to the first test that runs.
-const globalSetup /*: ?string */ = isCI
-  ? '<rootDir>/private/react-native-fantom/runner/warmup/index.js'
-  : null;
-
 module.exports = {
   displayName: 'fantom',
   rootDir: path.resolve(__dirname, '../../..') /*:: as string */,
@@ -42,5 +33,6 @@ module.exports = {
   transformIgnorePatterns: ['.*'],
   testRunner: '<rootDir>/private/react-native-fantom/runner/index.js',
   watchPathIgnorePatterns: ['<rootDir>/private/react-native-fantom/build/'],
-  globalSetup,
+  globalSetup:
+    '<rootDir>/private/react-native-fantom/runner/global-setup/setup.js',
 };

…ct-native-fantom/runner/warmup/warmup.js …tive-fantom/runner/global-setup/build.jsprivate/react-native-fantom/runner/warmup/warmup.js renamed to private/react-native-fantom/runner/global-setup/build.js private/react-native-fantom/runner/warmup/warmup.js renamed to private/react-native-fantom/runner/global-setup/build.js

@@ -8,6 +8,7 @@
  * @format
  */
 
+import {isCI} from '../EnvironmentOptions';
 import {
   getBuckModesForPlatform,
   getDebugInfoFromCommandResult,
@@ -37,28 +38,27 @@ async function tryOrLog(
   }
 }
 
-export default async function warmUp(
-  globalConfig: {...},
-  projectConfig: {...},
-): Promise<void> {
-  await tryOrLog(
-    () => warmUpHermesCompiler(false),
-    'Error warming up Hermes compiler (dev)',
-  );
-  await tryOrLog(
-    () => warmUpHermesCompiler(true),
-    'Error warming up Hermes compiler (opt)',
-  );
-  await tryOrLog(
-    () => warmUpRNTesterCLI(false),
-    'Error warming up RN Tester CLI (dev)',
-  );
-  await tryOrLog(
-    () => warmUpRNTesterCLI(true),
-    'Error warming up RN Tester CLI (opt)',
-  );
-  await tryOrLog(() => warmUpMetro(false), 'Error warming up Metro (dev)');
-  await tryOrLog(() => warmUpMetro(true), 'Error warming up Metro (opt)');
+export default async function build(): Promise<void> {
+  if (isCI) {
+    await tryOrLog(
+      () => warmUpHermesCompiler(false),
+      'Error warming up Hermes compiler (dev)',
+    );
+    await tryOrLog(
+      () => warmUpHermesCompiler(true),
+      'Error warming up Hermes compiler (opt)',
+    );
+    await tryOrLog(
+      () => warmUpRNTesterCLI(false),
+      'Error warming up RN Tester CLI (dev)',
+    );
+    await tryOrLog(
+      () => warmUpRNTesterCLI(true),
+      'Error warming up RN Tester CLI (opt)',
+    );
+    await tryOrLog(() => warmUpMetro(false), 'Error warming up Metro (dev)');
+    await tryOrLog(() => warmUpMetro(true), 'Error warming up Metro (opt)');
+  }
 }
 
 async function warmUpMetro(isOptimizedMode: boolean): Promise<void> {

private/react-native-fantom/runner/global-setup/globalSetup.js

@@ -0,0 +1,21 @@
+/**
+ * Copyright (c) Meta Platforms, Inc. and affiliates.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @flow strict-local
+ * @format
+ */
+
+import {isOSS} from '../EnvironmentOptions';
+import build from './build';
+
+export default async function globalSetup(
+  globalConfig: {...},
+  projectConfig: {...},
+): Promise<void> {
+  if (!isOSS) {
+    await build();
+  }
+}

…act-native-fantom/runner/warmup/index.js …tive-fantom/runner/global-setup/setup.jsprivate/react-native-fantom/runner/warmup/index.js renamed to private/react-native-fantom/runner/global-setup/setup.js private/react-native-fantom/runner/warmup/index.js renamed to private/react-native-fantom/runner/global-setup/setup.js

@@ -10,4 +10,4 @@
 
 require('../../../../scripts/shared/babelRegister').registerForMonorepo();
 
-module.exports = require('./warmup');
+module.exports = require('./globalSetup');