Merge pull request #369 from fs-webdev/handleSplitIoJest

use browser-resolve for splitio imports in jest config

Author: joeycozza

CHANGELOG-FRONTIER.md

@@ -1,3 +1,7 @@
+## 8.7.10
+
+- Have jest configuration use browser-resolve for requires in splitio packages. Only affects jest tests, nothing else.
+
 ## 8.7.9
 
 - Add additional check for unsupported browser banner to avoid errors

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@fs/react-scripts",
-  "version": "8.7.9",
+  "version": "8.7.10",
   "upstreamVersion": "5.0.1",
   "description": "Configuration and scripts for Create React App.",
   "repository": {
@@ -60,6 +60,7 @@
     "babel-plugin-bundled-import-meta": "^0.3.2",
     "babel-plugin-named-asset-import": "^0.3.8",
     "bfj": "^7.0.2",
+    "browser-resolve": "^2.0.0",
     "browserslist": "^4.18.1",
     "camelcase": "^6.2.1",
     "case-sensitive-paths-webpack-plugin": "^2.4.0",

packages/react-scripts/package.json

@@ -25,6 +25,7 @@ module.exports = (resolve, rootDir, isEjecting) => {
 
   const config = {
     roots: ['<rootDir>/src'],
+    resolver: `${__dirname}/jestResolver.js`,
 
     collectCoverageFrom:[
       "src/**/*.{js,jsx,ts,tsx}",

packages/react-scripts/scripts/utils/createJestConfig.js

@@ -0,0 +1,9 @@
+'use strict'
+const browserResolve = require('browser-resolve')
+
+module.exports = (path, options) => {
+  if (options.basedir.includes('@splitsoftware/splitio')) {
+    return browserResolve.sync(path, options)
+  }
+  return options.defaultResolver(path, options)
+}