add some flow types

jestjs · Dec 15, 2016 · 08e134f · 08e134f
1 parent 6471a7b
commit 08e134f
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 2 deletions.
diff --git a/packages/jest-cli/src/reporters/CoverageReporter.js b/packages/jest-cli/src/reporters/CoverageReporter.js
@@ -46,7 +46,6 @@ class CoverageReporter extends BaseReporter {
   ) {
     if (testResult.coverage) {
       this._coverageMap.merge(testResult.coverage);
-
       // Remove coverage data to free up some memory.
       delete testResult.coverage;
     }

diff --git a/packages/jest-runtime/src/transform.js b/packages/jest-runtime/src/transform.js
@@ -306,7 +306,7 @@ const generateEmptyCoverage = (
   const transform = getTransformer(filename, config);
   let source = content;
 
-  if (shouldTransform(filename, config)) {
+  if (transform && shouldTransform(filename, config)) {
     if (
       transform.canInstrument &&
       typeof transform.getEmptyCoverage === 'function'

diff --git a/types/Transform.js b/types/Transform.js
@@ -11,6 +11,28 @@
 
 import type {Config, Path} from 'types/Config';
 
+export type SourceMap = {|
+  file: string,
+  mappings: string,
+  names: string[],
+  sourceRoot: string,
+  sources: string[],
+  sourcesContent: string[],
+  version: number,
+|};
+
+export type Coverage = {|
+  path: string,
+  s: { [statementId: number]: number },
+  b: { [branchId: number]: number },
+  f: { [functionId: number]: number },
+  l: { [lineId: number]: number },
+  fnMap: { [functionId: number]: any },
+  statementMap: { [statementId: number]: any },
+  branchMap: { [branchId: number]: any },
+  inputSourceMap: ?SourceMap,
+|}
+
 export type TransformOptions = {|
   instrument: boolean,
   watch: boolean,
@@ -32,4 +54,11 @@ export type Transformer = {|
     config: Config,
     options?: TransformOptions,
   ) => string,
+
+  getEmptyCoverage: (
+    sourceText: string,
+    sourcePath: Path,
+    config: Config,
+    options?: TransformOptions,
+  ) => Coverage
 |};