mirror of https://github.com/status-im/metro.git
re-enable and fix tests
Summary: When bringing back `node-haste` to React Native, I left an `fdescribe` in a test that led to ~70 tests being skipped. This re-enables these tests, and fixes test failures Reviewed By: cpojer Differential Revision: D3811225 fbshipit-source-id: 67a16f385759bb829f1f3f559862eab7e78f2097
This commit is contained in:
parent
dd4722161e
commit
9889e7d5ca
|
@ -1273,8 +1273,8 @@ describe('DependencyGraph', function() {
|
|||
`Failed to build DependencyGraph: @providesModule naming collision:\n` +
|
||||
` Duplicate module name: index\n` +
|
||||
` Paths: /root/b.js collides with /root/index.js\n\n` +
|
||||
`This error is caused by a @providesModule declaration ` +
|
||||
`with the same name across two different files.`
|
||||
'This error is caused by a @providesModule declaration ' +
|
||||
'with the same name across two different files.'
|
||||
);
|
||||
expect(err.type).toEqual('DependencyGraphError');
|
||||
});
|
||||
|
@ -2608,15 +2608,17 @@ describe('DependencyGraph', function() {
|
|||
let DependencyGraph;
|
||||
beforeEach(function() {
|
||||
process.platform = 'win32';
|
||||
DependencyGraph = require('../index'); // force reload with fastpath
|
||||
|
||||
// force reload with fastpath
|
||||
jest.resetModules();
|
||||
DependencyGraph = require('../index');
|
||||
});
|
||||
|
||||
afterEach(function() {
|
||||
process.platform = realPlatform;
|
||||
});
|
||||
|
||||
pit('should get dependencies', function() {
|
||||
process.platform = 'win32';
|
||||
it('should get dependencies', function() {
|
||||
const root = 'C:\\root';
|
||||
setMockFileSystem({
|
||||
'root': {
|
||||
|
@ -6080,7 +6082,7 @@ describe('DependencyGraph', function() {
|
|||
});
|
||||
});
|
||||
|
||||
fdescribe('Deterministic order of dependencies', () => {
|
||||
describe('Deterministic order of dependencies', () => {
|
||||
let callDeferreds, dependencyGraph, moduleReadDeferreds;
|
||||
let moduleRead;
|
||||
let DependencyGraph;
|
||||
|
|
Loading…
Reference in New Issue