Revert D4149694: [RN] Update Jest + jest-haste-map

Differential Revision: D4149694

fbshipit-source-id: 65a39bc1d0a0f9e7094ebc8dec9233e432a5e14d
This commit is contained in:
Konstantin Raev 2016-11-14 12:34:40 -08:00 committed by Facebook Github Bot
parent 475a0c20f6
commit b64f892210
2 changed files with 92 additions and 7 deletions

View File

@ -8,13 +8,104 @@
*/ */
'use strict'; 'use strict';
jest.disableAutomock(); jest.autoMockOff();
jest.useRealTimers(); jest.useRealTimers();
jest jest
.mock('fs') .mock('fs')
.mock('../../Logger') .mock('../../Logger')
.mock('../../lib/TransformCache'); .mock('../../lib/TransformCache');
// This is an ugly hack:
// * jest-haste-map uses `find` for fast file system crawling which won't work
// when we mock the file system in node. This mock copies the node crawler's
// implementation and always falls back to the node crawling mechanism.
// Ideally we'll make this an option in jest-haste-map to force it to use
// the node crawler.
jest.mock('jest-haste-map/build/crawlers/node', () => {
const H = require('jest-haste-map/build/constants');
const fs = require('fs');
const path = require('path');
function find(
roots,
extensions,
ignore,
callback)
{
const result = [];
let activeCalls = 0;
function search(directory) {
activeCalls++;
fs.readdir(directory, (err, names) => {
activeCalls--;
names.forEach(file => {
file = path.join(directory, file);
if (ignore(file)) {
return;
}
activeCalls++;
fs.lstat(file, (err, stat) => {
activeCalls--;
if (!err && stat && !stat.isSymbolicLink()) {
if (stat.isDirectory()) {
search(file);
} else {
const ext = path.extname(file).substr(1);
if (extensions.indexOf(ext) !== -1) {
result.push([file, stat.mtime.getTime()]);
}
}
}
if (activeCalls === 0) {
callback(result);
}
});
});
if (activeCalls === 0) {
callback(result);
}
});
}
roots.forEach(search);
}
return function nodeCrawl(
roots,
extensions,
ignore,
data)
{
return new Promise(resolve => {
const callback = list => {
const files = Object.create(null);
list.forEach(fileData => {
const name = fileData[0];
const mtime = fileData[1];
const existingFile = data.files[name];
if (existingFile && existingFile[H.MTIME] === mtime) {
files[name] = existingFile;
} else {
// See ../constants.js
files[name] = ['', mtime, 0, []];
}
});
data.files = files;
resolve(data);
};
find(roots, extensions, ignore, callback);
});
};
});
const mocksPattern = /(?:[\\/]|^)__mocks__[\\/]([^\/]+)\.js$/; const mocksPattern = /(?:[\\/]|^)__mocks__[\\/]([^\/]+)\.js$/;
jasmine.DEFAULT_TIMEOUT_INTERVAL = 10000; jasmine.DEFAULT_TIMEOUT_INTERVAL = 10000;
@ -105,7 +196,6 @@ describe('DependencyGraph', function() {
assetExts: ['png', 'jpg'], assetExts: ['png', 'jpg'],
cache: new Cache(), cache: new Cache(),
fileWatcher, fileWatcher,
forceNodeFilesystemAPI: true,
providesModuleNodeModules: [ providesModuleNodeModules: [
'haste-fbjs', 'haste-fbjs',
'react-haste', 'react-haste',

View File

@ -53,7 +53,6 @@ class DependencyGraph {
roots: Array<string>, roots: Array<string>,
ignoreFilePath: (filePath: string) => boolean, ignoreFilePath: (filePath: string) => boolean,
fileWatcher: FileWatcher, fileWatcher: FileWatcher,
forceNodeFilesystemAPI: boolean,
assetRoots_DEPRECATED: Array<string>, assetRoots_DEPRECATED: Array<string>,
assetExts: Array<string>, assetExts: Array<string>,
providesModuleNodeModules: mixed, providesModuleNodeModules: mixed,
@ -86,7 +85,6 @@ class DependencyGraph {
roots, roots,
ignoreFilePath, ignoreFilePath,
fileWatcher, fileWatcher,
forceNodeFilesystemAPI,
assetRoots_DEPRECATED, assetRoots_DEPRECATED,
assetExts, assetExts,
providesModuleNodeModules, providesModuleNodeModules,
@ -110,7 +108,6 @@ class DependencyGraph {
roots: Array<string>, roots: Array<string>,
ignoreFilePath: (filePath: string) => boolean, ignoreFilePath: (filePath: string) => boolean,
fileWatcher: FileWatcher, fileWatcher: FileWatcher,
forceNodeFilesystemAPI?: boolean,
assetRoots_DEPRECATED: Array<string>, assetRoots_DEPRECATED: Array<string>,
assetExts: Array<string>, assetExts: Array<string>,
providesModuleNodeModules: mixed, providesModuleNodeModules: mixed,
@ -134,7 +131,6 @@ class DependencyGraph {
roots, roots,
ignoreFilePath: ignoreFilePath || (() => {}), ignoreFilePath: ignoreFilePath || (() => {}),
fileWatcher, fileWatcher,
forceNodeFilesystemAPI: !!forceNodeFilesystemAPI,
assetRoots_DEPRECATED: assetRoots_DEPRECATED || [], assetRoots_DEPRECATED: assetRoots_DEPRECATED || [],
assetExts: assetExts || [], assetExts: assetExts || [],
providesModuleNodeModules, providesModuleNodeModules,
@ -169,7 +165,6 @@ class DependencyGraph {
const mw = this._opts.maxWorkers; const mw = this._opts.maxWorkers;
const haste = new JestHasteMap({ const haste = new JestHasteMap({
extensions: this._opts.extensions.concat(this._opts.assetExts), extensions: this._opts.extensions.concat(this._opts.assetExts),
forceNodeFilesystemAPI: this._opts.forceNodeFilesystemAPI,
ignorePattern: {test: this._opts.ignoreFilePath}, ignorePattern: {test: this._opts.ignoreFilePath},
maxWorkers: typeof mw === 'number' && mw >= 1 ? mw : getMaxWorkers(), maxWorkers: typeof mw === 'number' && mw >= 1 ? mw : getMaxWorkers(),
mocksPattern: '', mocksPattern: '',