@teambit/dependencies 1.0.784 → 1.0.786

package/dist/files-dependency-builder/dependency-tree/index.spec.js

@@ -80,10 +80,10 @@ function cleanUnitDir() {
 describe('dependencyTree', function () {
   this.timeout(8000);
   function testTreesForFormat(format, ext = '.js') {
-    it('returns an object form of the dependency tree for a file', () => {
+    it('returns an object form of the dependency tree for a file', async () => {
       const root = `${UNIT_TEST_DIR}/${format}`;
       const filename = _path().default.normalize(`${root}/a${ext}`);
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename,
         root
       });
@@ -179,20 +179,20 @@ describe('dependencyTree', function () {
     // mockfs.restore();
     cleanUnitDir();
   });
-  it('returns an empty object for a non-existent filename', () => {
+  it('returns an empty object for a non-existent filename', async () => {
     mockfs({
       imaginary: {}
     });
     const root = `${UNIT_TEST_DIR}/imaginary`;
     const filename = `${root}/notafile.js`;
-    const tree = dependencyTree({
+    const tree = await dependencyTree({
       filename,
       root
     });
     (0, _assert().default)(tree instanceof Object);
     (0, _assert().default)(!Object.keys(tree).length);
   });
-  it('handles nested tree structures', () => {
+  it('handles nested tree structures', async () => {
     const directory = `${UNIT_TEST_DIR}/extended`;
     mockfs({
       [directory]: {
@@ -212,7 +212,7 @@ describe('dependencyTree', function () {
       }
     });
     const filename = _path().default.normalize(`${directory}/a.js`);
-    const tree = dependencyTree({
+    const tree = await dependencyTree({
       filename,
       directory
     });
@@ -228,7 +228,7 @@ describe('dependencyTree', function () {
     // f ang g
     _assert().default.equal(cTree.length, 2);
   });
-  it('does not include files that are not real (#13)', () => {
+  it('does not include files that are not real (#13)', async () => {
     mockfs({
       [`${UNIT_TEST_DIR}/onlyRealDeps`]: {
         'a.js': 'var notReal = require("./notReal");'
@@ -236,14 +236,14 @@ describe('dependencyTree', function () {
     });
     const directory = `${UNIT_TEST_DIR}/onlyRealDeps`;
     const filename = _path().default.normalize(`${directory}/a.js`);
-    const tree = dependencyTree({
+    const tree = await dependencyTree({
       filename,
       directory
     });
     const subTree = tree[filename];
     _assert().default.ok(!Object.keys(subTree).some(dep => dep.indexOf('notReal') !== -1));
   });
-  it('does not choke on cyclic dependencies', () => {
+  it('does not choke on cyclic dependencies', async () => {
     mockfs({
       [`${UNIT_TEST_DIR}/cyclic`]: {
         'a.js': 'var b = require("./b");',
@@ -253,28 +253,28 @@ describe('dependencyTree', function () {
     const directory = `${UNIT_TEST_DIR}/cyclic`;
     const filename = _path().default.normalize(`${directory}/a.js`);
     const spy = _sinon().default.spy(dependencyTreeRewired, '_getDependencies');
-    const tree = dependencyTreeRewired.default({
+    const tree = await dependencyTreeRewired.default({
       filename,
       directory
     });
     (0, _assert().default)(spy.callCount === 2);
     (0, _assert().default)(Object.keys(tree[filename]).length);
-    dependencyTreeRewired._getDependencies.restore();
+    await dependencyTreeRewired._getDependencies.restore();
   });
-  it('excludes Nodejs core modules by default', () => {
+  it('excludes Nodejs core modules by default', async () => {
     const directory = `${fixtures}/commonjs`;
     const filename = _path().default.normalize(`${directory}/b.js`);
-    const tree = dependencyTree({
+    const tree = await dependencyTree({
       filename,
       directory
     });
     (0, _assert().default)(Object.keys(tree[filename]).length === 0);
     (0, _assert().default)(Object.keys(tree)[0].indexOf('b.js') !== -1);
   });
-  it('returns a list of absolutely pathed files', () => {
+  it('returns a list of absolutely pathed files', async () => {
     const directory = `${UNIT_TEST_DIR}/commonjs`;
     const filename = `${directory}/b.js`;
-    const tree = dependencyTree({
+    const tree = await dependencyTree({
       filename,
       directory
     });
@@ -284,7 +284,7 @@ describe('dependencyTree', function () {
     }
   });
   describe('when given a detective configuration', () => {
-    it('passes it through to precinct', () => {
+    it('passes it through to precinct', async () => {
       const spy = _sinon().default.spy(_precinct().default, 'paperwork');
       const directory = _path().default.normalize(`${fixtures}/onlyRealDeps`);
       const filename = _path().default.normalize(`${directory}/a.js`);
@@ -293,7 +293,7 @@ describe('dependencyTree', function () {
           skipLazyLoaded: true
         }
       };
-      dependencyTree({
+      await dependencyTree({
         filename,
         directory,
         detective: detectiveConfig
@@ -304,7 +304,7 @@ describe('dependencyTree', function () {
   });
   describe('when given a list to store non existent partials', () => {
     describe('and the file contains no valid partials', () => {
-      it('stores the invalid partials', () => {
+      it('stores the invalid partials', async () => {
         mockfs({
           [`${UNIT_TEST_DIR}/onlyRealDeps`]: {
             'a.js': 'var notReal = require("./notReal");'
@@ -313,7 +313,7 @@ describe('dependencyTree', function () {
         const directory = _path().default.normalize(`${UNIT_TEST_DIR}/onlyRealDeps`);
         const filename = _path().default.normalize(`${directory}/a.js`);
         const nonExistent = [];
-        dependencyTree({
+        await dependencyTree({
           filename,
           directory,
           nonExistent
@@ -323,7 +323,7 @@ describe('dependencyTree', function () {
       });
     });
     describe('and the file contains all valid partials', () => {
-      it('does not store anything', () => {
+      it('does not store anything', async () => {
         mockfs({
           [`${UNIT_TEST_DIR}/onlyRealDeps`]: {
             'a.js': 'var b = require("./b");',
@@ -333,7 +333,7 @@ describe('dependencyTree', function () {
         const directory = `${UNIT_TEST_DIR}/onlyRealDeps`;
         const filename = `${directory}/a.js`;
         const nonExistent = [];
-        dependencyTree({
+        await dependencyTree({
           filename,
           directory,
           nonExistent
@@ -342,7 +342,7 @@ describe('dependencyTree', function () {
       });
     });
     describe('and the file contains a mix of invalid and valid partials', () => {
-      it('stores the invalid ones', () => {
+      it('stores the invalid ones', async () => {
         mockfs({
           [`${UNIT_TEST_DIR}/onlyRealDeps`]: {
             'a.js': 'var b = require("./b");',
@@ -353,7 +353,7 @@ describe('dependencyTree', function () {
         const directory = _path().default.normalize(`${UNIT_TEST_DIR}/onlyRealDeps`);
         const filename = _path().default.normalize(`${directory}/a.js`);
         const nonExistent = [];
-        dependencyTree({
+        await dependencyTree({
           filename,
           directory,
           nonExistent
@@ -363,7 +363,7 @@ describe('dependencyTree', function () {
       });
     });
     describe('and there is more than one reference to the invalid partial', () => {
-      it('should include the non-existent partial per file', () => {
+      it('should include the non-existent partial per file', async () => {
         mockfs({
           [`${UNIT_TEST_DIR}/onlyRealDeps`]: {
             'a.js': 'var b = require("./b");\nvar crap = require("./notRealMan");',
@@ -374,7 +374,7 @@ describe('dependencyTree', function () {
         const directory = _path().default.normalize(`${UNIT_TEST_DIR}/onlyRealDeps`);
         const filename = _path().default.normalize(`${directory}/a.js`);
         const nonExistent = [];
-        dependencyTree({
+        await dependencyTree({
           filename,
           directory,
           nonExistent
@@ -397,8 +397,8 @@ describe('dependencyTree', function () {
       this._revert();
     });
     it('throws if the filename is missing', () => {
-      _assert().default.throws(() => {
-        dependencyTree({
+      _assert().default.rejects(async () => {
+        await dependencyTree({
           filename: undefined,
           // @ts-ignore
           directory: this._directory
@@ -406,17 +406,17 @@ describe('dependencyTree', function () {
       });
     });
     it('throws if the root is missing', () => {
-      _assert().default.throws(() => {
-        dependencyTree({
+      _assert().default.rejects(async () => {
+        await dependencyTree({
           filename: undefined
         });
       });
     });
     it('throws if a supplied filter is not a function', () => {
-      _assert().default.throws(() => {
+      _assert().default.rejects(async () => {
         const directory = `${UNIT_TEST_DIR}/onlyRealDeps`;
         const filename = `${directory}/a.js`;
-        dependencyTree({
+        await dependencyTree({
           filename,
           directory,
           filter: 'foobar'
@@ -424,10 +424,10 @@ describe('dependencyTree', function () {
       });
     });
     it('does not throw on the legacy `root` option', () => {
-      _assert().default.doesNotThrow(() => {
+      _assert().default.doesNotReject(async () => {
         const directory = `${UNIT_TEST_DIR}/onlyRealDeps`;
         const filename = `${directory}/a.js`;
-        dependencyTree({
+        await dependencyTree({
           filename,
           root: directory
         });
@@ -440,17 +440,17 @@ describe('dependencyTree', function () {
       this._directory = `${UNIT_TEST_DIR}/commonjs`;
     });
     it('does not throw', () => {
-      _assert().default.doesNotThrow(() => {
-        dependencyTree({
+      _assert().default.doesNotReject(async () => {
+        await dependencyTree({
           filename: 'foo',
           // @ts-ignore
           directory: this._directory
         });
       });
     });
-    it('returns no dependencies', () => {
+    it('returns no dependencies', async () => {
       // @ts-ignore
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename: 'foo',
         directory: this._directory
       });
@@ -462,10 +462,10 @@ describe('dependencyTree', function () {
       // @ts-ignore
       this._spy = _sinon().default.spy(dependencyTreeRewired, '_getDependencies');
     });
-    afterEach(() => {
-      dependencyTreeRewired._getDependencies.restore();
+    afterEach(async () => {
+      await dependencyTreeRewired._getDependencies.restore();
     });
-    it('returns the precomputed list of a cached entry point', () => {
+    it('returns the precomputed list of a cached entry point', async () => {
       const filename = `${UNIT_TEST_DIR}/amd/a.js`;
       const directory = `${UNIT_TEST_DIR}/amd`;
       const cache = {
@@ -476,7 +476,7 @@ describe('dependencyTree', function () {
           }
         }
       };
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename,
         directory,
         visited: cache
@@ -500,10 +500,10 @@ describe('dependencyTree', function () {
         mockes6();
       });
       testTreesForFormat('es6');
-      it('resolves files that have jsx', () => {
+      it('resolves files that have jsx', async () => {
         // @ts-ignore
         const filename = _path().default.normalize(`${this._directory}/jsx.js`);
-        const tree = dependencyTree({
+        const tree = await dependencyTree({
           filename,
           // @ts-ignore
           directory: this._directory
@@ -511,10 +511,10 @@ describe('dependencyTree', function () {
         // @ts-ignore
         _assert().default.ok(tree[filename].includes(_path().default.normalize(`${this._directory}/c.js`)));
       });
-      it('resolves files with a jsx extension', () => {
+      it('resolves files with a jsx extension', async () => {
         // @ts-ignore
         const filename = _path().default.normalize(`${this._directory}/foo.jsx`);
-        const tree = dependencyTree({
+        const tree = await dependencyTree({
           filename,
           // @ts-ignore
           directory: this._directory
@@ -522,10 +522,10 @@ describe('dependencyTree', function () {
         // @ts-ignore
         _assert().default.ok(tree[filename].includes(_path().default.normalize(`${this._directory}/b.js`)));
       });
-      it('resolves files that have es7', () => {
+      it('resolves files that have es7', async () => {
         // @ts-ignore
         const filename = _path().default.normalize(`${this._directory}/es7.js`);
-        const tree = dependencyTree({
+        const tree = await dependencyTree({
           filename,
           // @ts-ignore
           directory: this._directory
@@ -592,10 +592,10 @@ describe('dependencyTree', function () {
     });
   });
   describe('when a filter function is supplied', () => {
-    it('uses the filter to determine if a file should be included in the results', () => {
+    it('uses the filter to determine if a file should be included in the results', async () => {
       const directory = _path().default.normalize(`${fixtures}/onlyRealDeps`);
       const filename = _path().default.normalize(`${directory}/a.js`);
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename,
         directory,
         // Skip all 3rd party deps
@@ -620,10 +620,10 @@ describe('dependencyTree', function () {
         }
       });
     });
-    it('includes the lazy dependency', () => {
+    it('includes the lazy dependency', async () => {
       const directory = `${UNIT_TEST_DIR}/cjs`;
       const filename = _path().default.normalize(`${directory}/foo.js`);
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename,
         directory
       });
@@ -640,10 +640,10 @@ describe('dependencyTree', function () {
       });
     });
     describe('and mixedImport mode is turned on', () => {
-      it('includes the lazy dependency', () => {
+      it('includes the lazy dependency', async () => {
         const directory = `${UNIT_TEST_DIR}/es6`;
         const filename = _path().default.normalize(`${directory}/foo.js`);
-        const tree = dependencyTree({
+        const tree = await dependencyTree({
           filename,
           directory,
           detective: {
@@ -665,10 +665,10 @@ describe('dependencyTree', function () {
         }
       });
     });
-    it('includes the dynamic import', () => {
+    it('includes the dynamic import', async () => {
       const directory = _path().default.normalize(`${UNIT_TEST_DIR}/es6`);
       const filename = _path().default.normalize(`${directory}/foo.js`);
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename,
         directory
       });
@@ -685,14 +685,14 @@ describe('dependencyTree', function () {
         }
       });
     });
-    it('should include it as a dependency and not throw an error', () => {
+    it('should include it as a dependency and not throw an error', async () => {
       const directory = _path().default.normalize(`${UNIT_TEST_DIR}/baz`);
       const filename = _path().default.normalize(`${directory}/foo.js`);
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename,
         directory
       });
-      _assert().default.ok(`${directory}/bar.json` in tree);
+      _assert().default.ok(_path().default.normalize(_path().default.join(directory, 'bar.json')) in tree);
     });
   });
 
@@ -715,10 +715,10 @@ describe('dependencyTree', function () {
         }
       });
     });
-    it('it includes the module entry as dependency', () => {
+    it('it includes the module entry as dependency', async () => {
       const directory = `${UNIT_TEST_DIR}/es6`;
       const filename = `${directory}/module.entry.js`;
-      const tree = dependencyTree({
+      const tree = await dependencyTree({
         filename,
         directory,
         nodeModulesConfig: {
@@ -759,7 +759,7 @@ describe('dependencyTree', function () {
         }
       });
     });
-    it('should not override the cache with wrong packages', () => {
+    it('should not override the cache with wrong packages', async () => {
       const directory = _path().default.normalize(`${UNIT_TEST_DIR}/baz`);
       const fooFile = _path().default.normalize(`${directory}/foo.js`);
       const barFile = _path().default.normalize(`${directory}/bar.js`);
@@ -773,18 +773,18 @@ describe('dependencyTree', function () {
 
       // @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
       config.filename = fooFile;
-      dependencyTree(config);
+      await dependencyTree(config);
       expect(nonExistent[fooFile]).to.deep.equal(['non-exist-foo-pkg']);
 
       // @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
       config.filename = barFile;
-      dependencyTree(config);
+      await dependencyTree(config);
       expect(nonExistent[fooFile]).to.deep.equal(['non-exist-foo-pkg']);
       expect(nonExistent[barFile]).to.deep.equal(['non-exist-bar-pkg']);
 
       // @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
       config.filename = bazFile;
-      dependencyTree(config);
+      await dependencyTree(config);
       expect(nonExistent[fooFile]).to.deep.equal(['non-exist-foo-pkg']);
       expect(nonExistent[barFile]).to.deep.equal(['non-exist-bar-pkg']);
       expect(nonExistent[bazFile]).to.deep.equal(['non-exist-baz-pkg']);
@@ -800,7 +800,7 @@ describe('dependencyTree', function () {
         }
       });
     });
-    it('should not crash with "RangeError: Maximum call stack size exceeded" error', () => {
+    it('should not crash with "RangeError: Maximum call stack size exceeded" error', async () => {
       const directory = _path().default.normalize(`${UNIT_TEST_DIR}/baz`);
       const baseFile = _path().default.normalize(`${directory}/base.scss`);
       const indexFile = _path().default.normalize(`${directory}/index.jsx`);
@@ -810,16 +810,16 @@ describe('dependencyTree', function () {
 
       // @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
       config.filename = baseFile;
-      dependencyTree(config);
+      await dependencyTree(config);
 
       // @ts-ignore AUTO-ADDED-AFTER-MIGRATION-PLEASE-FIX!
       config.filename = indexFile;
-      const dependencies = dependencyTree(config);
+      const dependencies = await dependencyTree(config);
       expect(dependencies).to.be.ok;
     });
   });
   describe('files with dynamic import', () => {
-    it('should not show missing dependencies', () => {
+    it('should not show missing dependencies', async () => {
       mockfs({
         [`${UNIT_TEST_DIR}/dynamic`]: {
           'foo.js': 'const a = "./b"; import(a); require(a);'
@@ -828,7 +828,7 @@ describe('dependencyTree', function () {
       const directory = _path().default.normalize(`${UNIT_TEST_DIR}/dynamic`);
       const filename = _path().default.normalize(`${directory}/foo.js`);
       const visited = {};
-      dependencyTree({
+      await dependencyTree({
         filename,
         directory,
         visited
@@ -837,7 +837,7 @@ describe('dependencyTree', function () {
     });
   });
   describe('files with import from cdn (http, https)', () => {
-    it('should not show missing dependencies when importing from https', () => {
+    it('should not show missing dependencies when importing from https', async () => {
       mockfs({
         [`${UNIT_TEST_DIR}/cdn`]: {
           'foo.js': 'import { a } from "https://unpkg.com";'
@@ -846,14 +846,14 @@ describe('dependencyTree', function () {
       const directory = _path().default.normalize(`${UNIT_TEST_DIR}/cdn`);
       const filename = _path().default.normalize(`${directory}/foo.js`);
       const visited = {};
-      dependencyTree({
+      await dependencyTree({
         filename,
         directory,
         visited
       });
       expect(visited[filename].missing).to.be.undefined;
     });
-    it('should not show missing dependencies when importing from http', () => {
+    it('should not show missing dependencies when importing from http', async () => {
       mockfs({
         [`${UNIT_TEST_DIR}/cdn`]: {
           'bar.js': 'const b = require("http://pkg.com");'
@@ -862,7 +862,7 @@ describe('dependencyTree', function () {
       const directory = _path().default.normalize(`${UNIT_TEST_DIR}/cdn`);
       const filename = _path().default.normalize(`${directory}/bar.js`);
       const visited = {};
-      dependencyTree({
+      await dependencyTree({
         filename,
         directory,
         visited