perf(cache): share config-sets for parallel test running

kulshekhar · Sep 1, 2018 · 090ca7b · 090ca7b
1 parent 5ef980f
commit 090ca7b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/e2e/__tests__/__snapshots__/logger.test.ts.snap b/e2e/__tests__/__snapshots__/logger.test.ts.snap
@@ -8,6 +8,7 @@ Array [
  "[level:20] checking version of jest: OK",
  "[level:20] created new transformer",
  "[level:20] computing cache key for <cwd>/Hello.spec.ts",
+ "[level:30] no matching config-set found, creating a new one",
  "[level:20] backporting config",
  "[level:20] normalized jest config",
  "[level:20] normalized ts-jest config",
@@ -40,6 +41,7 @@ Array [
  "[level:20] checking version of jest: OK",
  "[level:20] created new transformer",
  "[level:20] computing cache key for <cwd>/Hello.spec.ts",
+ "[level:30] no matching config-set found, creating a new one",
  "[level:20] backporting config",
  "[level:20] normalized jest config",
  "[level:20] normalized ts-jest config",
@@ -83,6 +85,7 @@ Array [
  "[level:20] checking version of jest: OK",
  "[level:20] created new transformer",
  "[level:20] computing cache key for <cwd>/Hello.spec.ts",
+ "[level:30] no matching config-set found, creating a new one",
  "[level:20] backporting config",
  "[level:20] normalized jest config",
  "[level:20] normalized ts-jest config",
@@ -124,6 +127,7 @@ Array [
  "[level:20] checking version of jest: OK",
  "[level:20] created new transformer",
  "[level:20] computing cache key for <cwd>/Hello.spec.ts",
+ "[level:30] no matching config-set found, creating a new one",
  "[level:20] backporting config",
  "[level:20] normalized jest config",
  "[level:20] normalized ts-jest config",
@@ -156,6 +160,7 @@ Array [
  "[level:20] checking version of jest: OK",
  "[level:20] created new transformer",
  "[level:20] computing cache key for <cwd>/Hello.spec.ts",
+ "[level:30] no matching config-set found, creating a new one",
  "[level:20] backporting config",
  "[level:20] normalized jest config",
  "[level:20] normalized ts-jest config",

diff --git a/src/ts-jest-transformer.ts b/src/ts-jest-transformer.ts
@@ -18,20 +18,19 @@ interface ConfigSetIndexItem {
 }
 
 export class TsJestTransformer implements jest.Transformer {
- protected static _lastTransformerId: number = 0
+ private static _configSetsIndex: ConfigSetIndexItem[] = []
+ private static _lastTransformerId: number = 0
  static get lastTransformerId() {
  return TsJestTransformer._lastTransformerId
  }
- protected static get _nextTransformerId() {
+ private static get _nextTransformerId() {
  return ++TsJestTransformer._lastTransformerId
  }
 
  readonly logger: Logger
  readonly id: number
  readonly options: TsJestGlobalOptions
 
- private _configSetsIndex: ConfigSetIndexItem[] = []
-
  constructor(baseOptions: TsJestGlobalOptions = {}) {
  this.options = { ...baseOptions }
  this.id = TsJestTransformer._nextTransformerId
@@ -50,17 +49,19 @@ export class TsJestTransformer implements jest.Transformer {
  let csi: ConfigSetIndexItem | undefined
  let jestConfigObj: jest.ProjectConfig
  if (typeof jestConfig === 'string') {
- csi = this._configSetsIndex.find(
+ csi = TsJestTransformer._configSetsIndex.find(
  cs => cs.jestConfig.serialized === jestConfig,
  )
  if (csi) return csi.configSet
  jestConfigObj = parse(jestConfig)
  } else {
- csi = this._configSetsIndex.find(cs => cs.jestConfig.value === jestConfig)
+ csi = TsJestTransformer._configSetsIndex.find(
+ cs => cs.jestConfig.value === jestConfig,
+ )
  if (csi) return csi.configSet
  // try to look-it up by stringified version
  const serialized = stringify(jestConfig)
- csi = this._configSetsIndex.find(
+ csi = TsJestTransformer._configSetsIndex.find(
  cs => cs.jestConfig.serialized === serialized,
  )
  if (csi) {
@@ -74,8 +75,9 @@ export class TsJestTransformer implements jest.Transformer {
  }
 
  // create the new record in the index
+ this.logger.info(`no matching config-set found, creating a new one`)
  const configSet = new ConfigSet(jestConfigObj, this.options, this.logger)
- this._configSetsIndex.push({
+ TsJestTransformer._configSetsIndex.push({
  jestConfig: new JsonableValue(jestConfigObj),
  configSet,
  })