status-im
/
react-native
mirror of https://github.com/status-im/react-native.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

packager: DependencyGraph-test: add broken use case 

Summary:
This changeset adds a test that verifies that the duplicate modules use case is broken. The goal is to acknowledge the problem for now, and when we update `jest-haste`, we'll simply fix what needs to be fixed in that test.

See also, https://github.com/facebook/jest/pull/3107

Reviewed By: davidaurelio

Differential Revision: D4673431

fbshipit-source-id: 05e09bdf61a2eddf2e9d1e32a33d32065c9051f1
This commit is contained in:
Jean Lauliac 2017-03-16 13:49:51 -07:00 committed by Facebook Github Bot
parent 3bb9512484
commit a1694ba86b
1 changed files with 63 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

63
packager/src/node-haste/__tests__/DependencyGraph-test.js
View File

@ -5181,6 +5181,69 @@ describe('DependencyGraph', function() {
expect(deps).toBeDefined();
});
});
it('should recover from multiple modules with the same name (but this is broken right now)', async () => {
const root = '/root';
console.warn = jest.fn();
const filesystem = setMockFileSystem({
'root': {
'index.js': [
'/**',
' * @providesModule index',
' */',
'require(\'a\')',
'require(\'b\')',
].join('\n'),
'a.js': [
'/**',
' * @providesModule a',
' */',
].join('\n'),
'b.js': [
'/**',
' * @providesModule b',
' */',
].join('\n'),
},
});
const dgraph = DependencyGraph.load({...defaults, roots: [root]});
await getOrderedDependenciesAsJSON(dgraph, root + '/index.js');
filesystem.root['b.js'] = [
'/**',
' * @providesModule a',
' */',
].join('\n');
await triggerAndProcessWatchEvent(dgraph, 'change', root + '/b.js');
try {
await getOrderedDependenciesAsJSON(dgraph, root + '/index.js');
throw new Error('expected `getOrderedDependenciesAsJSON` to fail');
} catch (error) {
if (error.type !== 'UnableToResolveError') {
throw error;
}
expect(console.warn).toBeCalled();
filesystem.root['b.js'] = [
'/**',
' * @providesModule b',
' */',
].join('\n');
await triggerAndProcessWatchEvent(dgraph, 'change', root + '/b.js');
}
// This verifies that it is broken right now. Instead of throwing it should
// return correct results. Once this is fixed in `jest-haste`, remove
// the whole try catch and verify results are matching a snapshot.
try {
await getOrderedDependenciesAsJSON(dgraph, root + '/index.js');
throw new Error('expected `getOrderedDependenciesAsJSON` to fail');
} catch (error) {
if (error.type !== 'UnableToResolveError') {
throw error;
}
}
});
});
describe('Extensions', () => {