From 2d03a34bbc1403327f3f95226f06f3faf0e9bcbc Mon Sep 17 00:00:00 2001 From: Rafael Oleza Date: Fri, 24 Nov 2017 12:58:12 -0800 Subject: [PATCH] Remove isJSON() method from Module Reviewed By: cpojer Differential Revision: D6408739 fbshipit-source-id: 3f1b76ca54937fbc433d4e7a804a9703a3a2ccb7 --- .../__tests__/DeltaCalculator-test.js | 3 - ...verseDependencies-integration-test.js.snap | 3 - .../traverseDependencies-integration-test.js | 211 ------------------ .../__tests__/traverseDependencies-test.js | 5 +- .../src/node-haste/AssetModule.js | 4 - .../metro-bundler/src/node-haste/Module.js | 4 - .../metro-bundler/src/node-haste/Polyfill.js | 4 - 7 files changed, 1 insertion(+), 233 deletions(-) diff --git a/packages/metro-bundler/src/DeltaBundler/__tests__/DeltaCalculator-test.js b/packages/metro-bundler/src/DeltaBundler/__tests__/DeltaCalculator-test.js index 01e5202c..3234072b 100644 --- a/packages/metro-bundler/src/DeltaBundler/__tests__/DeltaCalculator-test.js +++ b/packages/metro-bundler/src/DeltaBundler/__tests__/DeltaCalculator-test.js @@ -65,9 +65,6 @@ describe('DeltaCalculator', () => { isAsset() { return !!isAsset; }, - isJSON() { - return !!isAsset; - }, }; } diff --git a/packages/metro-bundler/src/DeltaBundler/__tests__/__snapshots__/traverseDependencies-integration-test.js.snap b/packages/metro-bundler/src/DeltaBundler/__tests__/__snapshots__/traverseDependencies-integration-test.js.snap index a35c9d53..a2910b84 100644 --- a/packages/metro-bundler/src/DeltaBundler/__tests__/__snapshots__/traverseDependencies-integration-test.js.snap +++ b/packages/metro-bundler/src/DeltaBundler/__tests__/__snapshots__/traverseDependencies-integration-test.js.snap @@ -78,7 +78,6 @@ Array [ ], "id": "index", "isAsset": false, - "isJSON": false, "isPolyfill": false, "path": "/root/index.js", "resolution": undefined, @@ -87,7 +86,6 @@ Array [ "dependencies": Array [], "id": "a", "isAsset": false, - "isJSON": false, "isPolyfill": false, "path": "/root/a.js", "resolution": undefined, @@ -96,7 +94,6 @@ Array [ "dependencies": Array [], "id": "b", "isAsset": false, - "isJSON": false, "isPolyfill": false, "path": "/root/b.js", "resolution": undefined, diff --git a/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-integration-test.js b/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-integration-test.js index bf6d2244..95ca2ecd 100644 --- a/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-integration-test.js +++ b/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-integration-test.js @@ -80,7 +80,6 @@ describe('traverseDependencies', function() { return { path: dep.path, - isJSON: dep.isJSON(), isAsset: dep.isAsset(), isPolyfill: dep.isPolyfill(), resolution: dep.resolution, @@ -175,7 +174,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -185,7 +183,6 @@ describe('traverseDependencies', function() { path: '/root/a.js', dependencies: ['b'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -195,7 +192,6 @@ describe('traverseDependencies', function() { path: '/root/b.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -235,7 +231,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -244,7 +239,6 @@ describe('traverseDependencies', function() { path: '/root/a.js', dependencies: ['b'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -279,7 +273,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -288,7 +281,6 @@ describe('traverseDependencies', function() { path: '/root/a.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -327,13 +319,11 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./a.json', './b'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, { id: 'package/a.json', - isJSON: true, path: '/root/a.json', dependencies: [], isAsset: false, @@ -342,7 +332,6 @@ describe('traverseDependencies', function() { }, { id: 'package/b.json', - isJSON: true, path: '/root/b.json', dependencies: [], isAsset: false, @@ -381,13 +370,11 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./package.json'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, { id: 'package/package.json', - isJSON: true, path: '/root/package.json', dependencies: [], isAsset: false, @@ -429,7 +416,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./imgs/a.png'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -439,7 +425,6 @@ describe('traverseDependencies', function() { dependencies: [], isAsset: true, resolution: 1, - isJSON: false, isPolyfill: false, }, ]); @@ -482,7 +467,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./imgs/a.png', './imgs/b.png', './imgs/c.png'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -492,7 +476,6 @@ describe('traverseDependencies', function() { resolution: 1.5, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, { @@ -501,7 +484,6 @@ describe('traverseDependencies', function() { resolution: 0.7, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, { @@ -510,7 +492,6 @@ describe('traverseDependencies', function() { resolution: 1, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, ]); @@ -554,7 +535,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./imgs/a.png', './imgs/b.png', './imgs/c.png'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -564,7 +544,6 @@ describe('traverseDependencies', function() { resolution: 1.5, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, { @@ -573,7 +552,6 @@ describe('traverseDependencies', function() { resolution: 0.7, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, { @@ -582,7 +560,6 @@ describe('traverseDependencies', function() { resolution: 1, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, ]); @@ -620,7 +597,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -629,7 +605,6 @@ describe('traverseDependencies', function() { path: '/root/a.js', dependencies: ['index'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -669,7 +644,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -678,7 +652,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -718,7 +691,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage/'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -727,7 +699,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -775,7 +746,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['sha.js', 'x.y.z'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -784,7 +754,6 @@ describe('traverseDependencies', function() { path: '/root/sha.js/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -793,7 +762,6 @@ describe('traverseDependencies', function() { path: '/root/x.y.z/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -827,7 +795,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -836,7 +803,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -872,7 +838,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -881,7 +846,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -918,7 +882,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -927,7 +890,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/lib/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -961,7 +923,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./lib/'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -970,7 +931,6 @@ describe('traverseDependencies', function() { path: '/root/lib/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1008,7 +968,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./lib/'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1017,7 +976,6 @@ describe('traverseDependencies', function() { path: '/root/lib/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1049,7 +1007,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1148,7 +1105,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage/subdir/lolynot'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1158,7 +1114,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/subdir/lolynot.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1203,7 +1158,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1213,7 +1167,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/main.js', dependencies: ['./subdir/lolynot'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1223,7 +1176,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/subdir/lolynot.js', dependencies: ['../other'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1233,7 +1185,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/other.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1296,7 +1247,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1306,7 +1256,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/client.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1359,7 +1308,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1368,7 +1316,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/client.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1422,7 +1369,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1431,7 +1377,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/client.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1482,7 +1427,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1491,7 +1435,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/client.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1545,7 +1488,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1555,7 +1497,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/client.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -1621,7 +1562,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1630,7 +1570,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/client.js', dependencies: ['./node', './dir/server.js'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1639,7 +1578,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/not-node.js', dependencies: ['./not-browser'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1648,7 +1586,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/browser.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1657,7 +1594,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/dir/client.js', dependencies: ['../hello'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1666,7 +1602,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/bye.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1728,7 +1663,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1737,7 +1671,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: ['node-package'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1746,7 +1679,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/browser-package/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1808,7 +1740,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1817,7 +1748,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: ['./dir/ooga'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1826,7 +1756,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/dir/ooga.js', dependencies: ['node-package'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1835,7 +1764,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/dir/browser.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1897,7 +1825,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1906,7 +1833,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: ['node-package'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1915,7 +1841,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/browser-package/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1974,7 +1899,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1983,7 +1907,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: ['booga'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -1991,7 +1914,6 @@ describe('traverseDependencies', function() { dependencies: [], id: '/root/emptyModule.js', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/emptyModule.js', resolution: undefined, @@ -2047,7 +1969,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2056,7 +1977,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: ['./booga'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2064,7 +1984,6 @@ describe('traverseDependencies', function() { dependencies: [], id: '/root/emptyModule.js', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/emptyModule.js', resolution: undefined, @@ -2132,7 +2051,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2141,7 +2059,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/index.js', dependencies: ['node-package'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2150,7 +2067,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/node_modules/rn-package/index.js', dependencies: ['nested-package'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2159,7 +2075,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/node_modules/nested-browser-package/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2188,7 +2103,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['/root/apple.js'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2197,7 +2111,6 @@ describe('traverseDependencies', function() { path: '/root/apple.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2299,7 +2212,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2312,7 +2224,6 @@ describe('traverseDependencies', function() { 'node-package-c', ], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2321,7 +2232,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/node_modules/rn-package-a/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2330,7 +2240,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/node_modules/rn-package-b/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2339,7 +2248,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/node_modules/rn-package-d/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2382,7 +2290,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2391,7 +2298,6 @@ describe('traverseDependencies', function() { path: '/root/foo/index.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2400,7 +2306,6 @@ describe('traverseDependencies', function() { path: '/root/provides-bar/lib/bar.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2436,7 +2341,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2445,7 +2349,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/bar.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2483,7 +2386,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['bar/lib/foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2492,7 +2394,6 @@ describe('traverseDependencies', function() { path: '/root/provides-bar/lib/foo.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2548,7 +2449,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -2558,7 +2458,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\a.js', dependencies: ['b'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -2568,7 +2467,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\b.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -2598,7 +2496,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['C:/root/apple.js'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2607,7 +2504,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\apple.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2651,7 +2547,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['./imgs/a.png', './imgs/b.png', './imgs/c.png'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2661,7 +2556,6 @@ describe('traverseDependencies', function() { resolution: 1.5, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, { @@ -2670,7 +2564,6 @@ describe('traverseDependencies', function() { resolution: 0.7, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, { @@ -2679,7 +2572,6 @@ describe('traverseDependencies', function() { resolution: 1, dependencies: [], isAsset: true, - isJSON: false, isPolyfill: false, }, ]); @@ -2753,7 +2645,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['foo', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2762,7 +2653,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/main.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2771,7 +2661,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/node_modules/bar/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2780,7 +2669,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/bar/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2830,7 +2718,6 @@ describe('traverseDependencies', function() { path: '/root/index.ios.js', dependencies: ['foo', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2839,7 +2726,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/index.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2848,7 +2734,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/bar/main.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2908,7 +2793,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['foo', 'bar/'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2917,7 +2801,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/main.js', dependencies: ['bar/lol'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2926,7 +2809,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/node_modules/bar/lol.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2935,7 +2817,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/bar/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -2999,7 +2880,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['foo', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3008,7 +2888,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/main.js', dependencies: ['bar/lol'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3017,7 +2896,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/node_modules/bar/lol.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3026,7 +2904,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/bar/main2.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3079,7 +2956,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3088,7 +2964,6 @@ describe('traverseDependencies', function() { path: '/root/path/to/bar.js', dependencies: ['foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3097,7 +2972,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3227,7 +3101,6 @@ describe('traverseDependencies', function() { 'anotherIndex', ], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3236,7 +3109,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/react-haste/main.js', dependencies: ['submodule'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3246,7 +3118,6 @@ describe('traverseDependencies', function() { '/root/node_modules/react-haste/node_modules/submodule/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3255,7 +3126,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/ember/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3264,7 +3134,6 @@ describe('traverseDependencies', function() { path: '/root/vendored_modules/a-vendored-package/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3273,7 +3142,6 @@ describe('traverseDependencies', function() { path: '/anotherRoot/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3318,7 +3186,6 @@ describe('traverseDependencies', function() { path: '/react-haste/index.js', dependencies: ['shouldWork'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3327,7 +3194,6 @@ describe('traverseDependencies', function() { path: '/react-haste/node_modules/react-haste/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3369,7 +3235,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['sha.js'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3378,7 +3243,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/sha.js/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3427,7 +3291,6 @@ describe('traverseDependencies', function() { path: '/root/index.ios.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3436,7 +3299,6 @@ describe('traverseDependencies', function() { path: '/root/a.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3488,7 +3350,6 @@ describe('traverseDependencies', function() { path: '/root/index.ios.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3497,7 +3358,6 @@ describe('traverseDependencies', function() { path: '/root/a.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3534,7 +3394,6 @@ describe('traverseDependencies', function() { path: '/root/index.ios.js', dependencies: ['./a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3543,7 +3402,6 @@ describe('traverseDependencies', function() { path: '/root/a.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3592,7 +3450,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['foo/package.json', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3601,7 +3458,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/package.json', dependencies: [], isAsset: false, - isJSON: true, isPolyfill: false, resolution: undefined, }, @@ -3610,7 +3466,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/bar/main.js', dependencies: ['./package.json'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3619,7 +3474,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/bar/package.json', dependencies: [], isAsset: false, - isJSON: true, isPolyfill: false, resolution: undefined, }, @@ -3653,7 +3507,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['a/index.js'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3662,7 +3515,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/a/index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3739,7 +3591,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['foo', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3748,7 +3599,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\main.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3757,7 +3607,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\node_modules\\bar\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3766,7 +3615,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\bar\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3816,7 +3664,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.ios.js', dependencies: ['foo', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3825,7 +3672,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\index.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3834,7 +3680,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\bar\\main.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3894,7 +3739,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['foo', 'bar/'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3903,7 +3747,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\main.js', dependencies: ['bar/lol'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3912,7 +3755,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\node_modules\\bar\\lol.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3921,7 +3763,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\bar\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3985,7 +3826,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['foo', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -3994,7 +3834,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\main.js', dependencies: ['bar/lol'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4003,7 +3842,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\node_modules\\bar\\lol.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4012,7 +3850,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\bar\\main2.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4065,7 +3902,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4074,7 +3910,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\path\\to\\bar.js', dependencies: ['foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4083,7 +3918,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4213,7 +4047,6 @@ describe('traverseDependencies', function() { 'anotherIndex', ], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4222,7 +4055,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\react-haste\\main.js', dependencies: ['submodule'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4232,7 +4064,6 @@ describe('traverseDependencies', function() { 'C:\\root\\node_modules\\react-haste\\node_modules\\submodule\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4241,7 +4072,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\ember\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4250,7 +4080,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\vendored_modules\\a-vendored-package\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4259,7 +4088,6 @@ describe('traverseDependencies', function() { path: 'C:\\anotherRoot\\index.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4303,7 +4131,6 @@ describe('traverseDependencies', function() { path: 'C:\\react-haste\\index.js', dependencies: ['shouldWork'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4312,7 +4139,6 @@ describe('traverseDependencies', function() { path: 'C:\\react-haste\\node_modules\\react-haste\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4354,7 +4180,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['sha.js'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4363,7 +4188,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\sha.js\\main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4412,7 +4236,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.ios.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4421,7 +4244,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\a.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4469,7 +4291,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.ios.js', dependencies: ['a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4478,7 +4299,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\a.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4515,7 +4335,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.ios.js', dependencies: ['./a'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4524,7 +4343,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\a.ios.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4573,7 +4391,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\index.js', dependencies: ['foo/package.json', 'bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4582,7 +4399,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\foo\\package.json', dependencies: [], isAsset: false, - isJSON: true, isPolyfill: false, resolution: undefined, }, @@ -4591,7 +4407,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\bar\\main.js', dependencies: ['./package.json'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4600,7 +4415,6 @@ describe('traverseDependencies', function() { path: 'C:\\root\\node_modules\\bar\\package.json', dependencies: [], isAsset: false, - isJSON: true, isPolyfill: false, resolution: undefined, }, @@ -4667,7 +4481,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4676,7 +4489,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4727,7 +4539,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4736,7 +4547,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4839,7 +4649,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage', 'foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4848,7 +4657,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/main.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, }, @@ -4857,7 +4665,6 @@ describe('traverseDependencies', function() { path: '/root/bar.js', dependencies: ['foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -4867,7 +4674,6 @@ describe('traverseDependencies', function() { path: '/root/foo.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -4910,7 +4716,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['./foo.png'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -4921,7 +4726,6 @@ describe('traverseDependencies', function() { dependencies: [], isAsset: true, resolution: 1, - isJSON: false, isPolyfill: false, resolveDependency: undefined, }, @@ -4972,7 +4776,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['aPackage'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -4982,7 +4785,6 @@ describe('traverseDependencies', function() { path: '/root/aPackage/browser.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5037,7 +4839,6 @@ describe('traverseDependencies', function() { dependencies: ['bPackage'], id: 'index', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/index.js', resolution: undefined, @@ -5047,7 +4848,6 @@ describe('traverseDependencies', function() { dependencies: [], id: 'bPackage/main.js', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/aPackage/main.js', resolution: undefined, @@ -5098,7 +4898,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5108,7 +4907,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/main.js', dependencies: ['bar'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5118,7 +4916,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/node_modules/bar/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5138,7 +4935,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5148,7 +4944,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/main.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5202,7 +4997,6 @@ describe('traverseDependencies', function() { path: '/root/index.js', dependencies: ['foo'], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5212,7 +5006,6 @@ describe('traverseDependencies', function() { path: '/root/node_modules/foo/browser.js', dependencies: [], isAsset: false, - isJSON: false, isPolyfill: false, resolution: undefined, resolveDependency: undefined, @@ -5337,7 +5130,6 @@ describe('traverseDependencies', function() { dependencies: ['a'], id: 'index', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/index.jsx', resolution: undefined, @@ -5346,7 +5138,6 @@ describe('traverseDependencies', function() { dependencies: [], id: 'a', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/a.coffee', resolution: undefined, @@ -5376,7 +5167,6 @@ describe('traverseDependencies', function() { dependencies: ['./a'], id: '/root/index.jsx', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/index.jsx', resolution: undefined, @@ -5385,7 +5175,6 @@ describe('traverseDependencies', function() { dependencies: [], id: '/root/a.coffee', isAsset: false, - isJSON: false, isPolyfill: false, path: '/root/a.coffee', resolution: undefined, diff --git a/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-test.js b/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-test.js index b61149bb..5fb4f684 100644 --- a/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-test.js +++ b/packages/metro-bundler/src/DeltaBundler/__tests__/traverseDependencies-test.js @@ -33,7 +33,7 @@ function deferred(value) { return {promise, resolve: () => resolve(value)}; } -function createModule({path, name, isAsset, isJSON}) { +function createModule({path, name, isAsset}) { return { path, name, @@ -43,9 +43,6 @@ function createModule({path, name, isAsset, isJSON}) { isAsset() { return !!isAsset; }, - isJSON() { - return !!isAsset; - }, }; } diff --git a/packages/metro-bundler/src/node-haste/AssetModule.js b/packages/metro-bundler/src/node-haste/AssetModule.js index 63428b98..1786193d 100644 --- a/packages/metro-bundler/src/node-haste/AssetModule.js +++ b/packages/metro-bundler/src/node-haste/AssetModule.js @@ -61,10 +61,6 @@ class AssetModule extends Module { return `AssetModule : ${this.path}`; } - isJSON() { - return false; - } - isAsset() { return true; } diff --git a/packages/metro-bundler/src/node-haste/Module.js b/packages/metro-bundler/src/node-haste/Module.js index ce85a706..4eb9c929 100644 --- a/packages/metro-bundler/src/node-haste/Module.js +++ b/packages/metro-bundler/src/node-haste/Module.js @@ -407,10 +407,6 @@ class Module { return `Module : ${this.path}`; } - isJSON() { - return extname(this.path) === '.json'; - } - isAsset() { return false; } diff --git a/packages/metro-bundler/src/node-haste/Polyfill.js b/packages/metro-bundler/src/node-haste/Polyfill.js index 84ec03be..5193e187 100644 --- a/packages/metro-bundler/src/node-haste/Polyfill.js +++ b/packages/metro-bundler/src/node-haste/Polyfill.js @@ -47,10 +47,6 @@ class Polyfill extends Module { return this._dependencies; } - isJSON() { - return false; - } - isPolyfill() { return true; }