From 012caf0e22d2f8ab6322d2e424af47ca14f89d20 Mon Sep 17 00:00:00 2001 From: David Aurelio Date: Mon, 15 Feb 2016 06:32:19 -0800 Subject: [PATCH] Remove test for missing feature Reviewed By: cpojer Differential Revision: D2937175 fb-gh-sync-id: 7c348018918a509e1cd681dfa8df9151b94ce765 shipit-source-id: 7c348018918a509e1cd681dfa8df9151b94ce765 --- .../__tests__/DependencyGraph-test.js | 66 ------------------- 1 file changed, 66 deletions(-) diff --git a/react-packager/src/DependencyResolver/DependencyGraph/__tests__/DependencyGraph-test.js b/react-packager/src/DependencyResolver/DependencyGraph/__tests__/DependencyGraph-test.js index c5b0b754..fa823286 100644 --- a/react-packager/src/DependencyResolver/DependencyGraph/__tests__/DependencyGraph-test.js +++ b/react-packager/src/DependencyResolver/DependencyGraph/__tests__/DependencyGraph-test.js @@ -3718,72 +3718,6 @@ describe('DependencyGraph', function() { }); }); - //TODO(davidaurelio) Make this actually worked. The test only passed because - // the mocked cache didn't cache. In reality, it didn't work. I tried it. - xpit('updates package.json', function() { - var root = '/root'; - var filesystem = fs.__setMockFilesystem({ - 'root': { - 'index.js': [ - '/**', - ' * @providesModule index', - ' */', - 'require("aPackage")', - ].join('\n'), - 'aPackage': { - 'package.json': JSON.stringify({ - name: 'aPackage', - main: 'main.js', - }), - 'main.js': 'main', - }, - }, - }); - - var dgraph = new DependencyGraph({ - ...defaults, - roots: [root], - }); - return getOrderedDependenciesAsJSON(dgraph, '/root/index.js').then(function() { - filesystem.root['index.js'] = filesystem.root['index.js'].replace(/aPackage/, 'bPackage'); - triggerFileChange('change', 'index.js', root, mockStat); - - filesystem.root.aPackage['package.json'] = JSON.stringify({ - name: 'bPackage', - main: 'main.js', - }); - triggerFileChange('change', 'package.json', '/root/aPackage', mockStat); - - return getOrderedDependenciesAsJSON(dgraph, '/root/index.js').then(function(deps) { - expect(deps) - .toEqual([ - { - id: 'index', - path: '/root/index.js', - dependencies: ['bPackage'], - isAsset: false, - isAsset_DEPRECATED: false, - isJSON: false, - isPolyfill: false, - resolution: undefined, - resolveDependency: undefined, - }, - { - id: 'bPackage/main.js', - path: '/root/aPackage/main.js', - dependencies: [], - isAsset: false, - isAsset_DEPRECATED: false, - isJSON: false, - isPolyfill: false, - resolution: undefined, - resolveDependency: undefined, - }, - ]); - }); - }); - }); - pit('changes to browser field', function() { var root = '/root'; var filesystem = fs.__setMockFilesystem({