chore: improve transform configs on sdk (#2643)

ScriptedAlchemy · ilteoood · web-flow · commit efe716fdb50e · 2024-06-28T21:05:49.000-07:00
Co-authored-by: ScriptedAlchemy &lt;zackaryjackson@bytedance.com&gt;
Co-authored-by: Matteo Pietro Dazzi &lt;matteopietro.dazzi@gmail.com&gt;
diff --git a/packages/bridge/bridge-react/vitest.config.ts b/packages/bridge/bridge-react/vitest.config.ts
@@ -6,7 +6,7 @@ export default defineConfig({
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
     __APP_VERSION__: '"0.0.0"',
   },
   plugins: [nxViteTsPaths()],
diff --git a/packages/chrome-devtools/vitest.config.ts b/packages/chrome-devtools/vitest.config.ts
@@ -5,7 +5,7 @@ export default defineConfig({
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
   },
   test: {
     environment: 'jsdom',
diff --git a/packages/managers/.swcrc b/packages/managers/.swcrc
@@ -0,0 +1,27 @@
+{
+  "jsc": {
+    "target": "es2017",
+    "parser": {
+      "syntax": "typescript",
+      "decorators": true,
+      "dynamicImport": true
+    },
+    "transform": {
+      "decoratorMetadata": true,
+      "legacyDecorator": true
+    },
+    "keepClassNames": true,
+    "externalHelpers": true,
+    "loose": true
+  },
+  "module": {
+    "type": "es6"
+  },
+  "sourceMaps": true,
+  "exclude": [
+    "jest.config.ts",
+    "./src/jest-setup.ts$",
+    "./**/jest-setup.ts$",
+    ".*.js$"
+  ]
+}
diff --git a/packages/managers/jest.config.js b/packages/managers/jest.config.js
@@ -1,25 +1,38 @@
-// For a detailed explanation regarding each configuration property, visit:
-// https://jestjs.io/docs/en/configuration.html
+/* eslint-disable */
+const { readFileSync } = require('fs');
+
+// Reading the SWC compilation config and remove the "exclude"
+// for the test files to be compiled by SWC
+const { exclude: _, ...swcJestConfig } = JSON.parse(
+  readFileSync(`${__dirname}/.swcrc`, 'utf-8'),
+);
+
+// disable .swcrc look-up by SWC core because we're passing in swcJestConfig ourselves.
+// If we do not disable this, SWC Core will read .swcrc and won't transform our test files due to "exclude"
+swcJestConfig.swcrc ??= false;
+// Uncomment if using global setup/teardown files being transformed via swc
+// https://nx.dev/packages/jest/documents/overview#global-setup/teardown-with-nx-libraries
+// jest needs EsModule Interop to find the default exported setup/teardown functions
+// swcJestConfig.module.noInterop = false;
 
 module.exports = {
   clearMocks: true,
+  cache: false,
   testEnvironment: 'jsdom',
   coveragePathIgnorePatterns: ['__tests__', '/node_modules/'],
-  coverageProvider: 'v8',
-  coverageReporters: ['cobertura', 'clover', 'json', 'lcov', 'text'],
   globals: {
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
   },
   preset: 'ts-jest',
   transformIgnorePatterns: [
     // Change MODULE_NAME_HERE to your module that isn't being compiled
     '/node_modules/(?!((@byted/garfish-)|(byted-tea-sdk))).+\\.js$',
   ],
   transform: {
-    '^.+\\.(t|j)sx?$': ['@swc/jest'],
+    '^.+\\.(t|j)sx?$': ['@swc/jest', swcJestConfig],
   },
   rootDir: __dirname,
   testMatch: ['<rootDir>__tests__/**/**.spec.[jt]s?(x)'],
diff --git a/packages/manifest/jest.config.js b/packages/manifest/jest.config.js
@@ -11,7 +11,7 @@ module.exports = {
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
   },
   preset: 'ts-jest',
   transformIgnorePatterns: [
diff --git a/packages/rspack/jest.config.js b/packages/rspack/jest.config.js
@@ -11,7 +11,7 @@ module.exports = {
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
   },
   preset: 'ts-jest',
   transformIgnorePatterns: [
diff --git a/packages/runtime/vitest.config.ts b/packages/runtime/vitest.config.ts
@@ -6,7 +6,7 @@ export default defineConfig({
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
   },
   plugins: [nxViteTsPaths()],
   test: {
diff --git a/packages/sdk/.swcrc b/packages/sdk/.swcrc
@@ -0,0 +1,27 @@
+{
+  "jsc": {
+    "target": "es2017",
+    "parser": {
+      "syntax": "typescript",
+      "decorators": true,
+      "dynamicImport": true
+    },
+    "transform": {
+      "decoratorMetadata": true,
+      "legacyDecorator": true
+    },
+    "keepClassNames": true,
+    "externalHelpers": true,
+    "loose": true
+  },
+  "module": {
+    "type": "es6"
+  },
+  "sourceMaps": true,
+  "exclude": [
+    "jest.config.ts",
+    "./src/jest-setup.ts$",
+    "./**/jest-setup.ts$",
+    ".*.js$"
+  ]
+}
diff --git a/packages/sdk/jest.config.js b/packages/sdk/jest.config.js
@@ -1,25 +1,38 @@
-// For a detailed explanation regarding each configuration property, visit:
-// https://jestjs.io/docs/en/configuration.html
+/* eslint-disable */
+const { readFileSync } = require('fs');
+
+// Reading the SWC compilation config and remove the "exclude"
+// for the test files to be compiled by SWC
+const { exclude: _, ...swcJestConfig } = JSON.parse(
+  readFileSync(`${__dirname}/.swcrc`, 'utf-8'),
+);
+
+// disable .swcrc look-up by SWC core because we're passing in swcJestConfig ourselves.
+// If we do not disable this, SWC Core will read .swcrc and won't transform our test files due to "exclude"
+swcJestConfig.swcrc ??= false;
+// Uncomment if using global setup/teardown files being transformed via swc
+// https://nx.dev/packages/jest/documents/overview#global-setup/teardown-with-nx-libraries
+// jest needs EsModule Interop to find the default exported setup/teardown functions
+// swcJestConfig.module.noInterop = false;
 
 module.exports = {
   clearMocks: true,
+  cache: false,
   testEnvironment: 'jsdom',
   coveragePathIgnorePatterns: ['__tests__', '/node_modules/'],
-  coverageProvider: 'v8',
-  coverageReporters: ['cobertura', 'clover', 'json', 'lcov', 'text'],
   globals: {
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
   },
   preset: 'ts-jest',
   transformIgnorePatterns: [
     // Change MODULE_NAME_HERE to your module that isn't being compiled
     '/node_modules/(?!((@byted/garfish-)|(byted-tea-sdk))).+\\.js$',
   ],
   transform: {
-    '^.+\\.(t|j)sx?$': ['@swc/jest'],
+    '^.+\\.(t|j)sx?$': ['@swc/jest', swcJestConfig],
   },
   rootDir: __dirname,
   testMatch: ['<rootDir>__tests__/**/**.spec.[jt]s?(x)'],
diff --git a/packages/webpack-bundler-runtime/jest.config.js b/packages/webpack-bundler-runtime/jest.config.js
@@ -11,7 +11,7 @@ module.exports = {
     __DEV__: true,
     __TEST__: true,
     __BROWSER__: false,
-    __VERSION__: '"unknow"',
+    __VERSION__: '"unknown"',
   },
   preset: 'ts-jest',
   transformIgnorePatterns: [
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
