test: coverage for resolving prod/dev, direct/deep deps with/without lock files

Author: dominykas

lib/deps.js

@@ -21,7 +21,7 @@ internals.walk = (node, callback) => {
 };
 
 
-exports.resolve = async ({ packageJson, lockfile }, options) => {
+internals.resolve = async ({ packageJson, lockfile }, options) => {
 
     const { deep, dev } = options;
 
@@ -103,11 +103,11 @@ exports.detect = async ({ packageJson, loadFile }, options) => {
 
     const lockfile = (await internals.tryLoad(loadFile, 'package-lock.json')) || (await internals.tryLoad(loadFile, 'npm-shrinkwrap.json'));
 
-    const versions = await exports.resolve({ packageJson, lockfile }, options);
+    const versions = await internals.resolve({ packageJson, lockfile }, options);
 
     const support = [];
 
-    for (const packageName of Object.keys(versions)) {
+    for (const packageName of Object.keys(versions).sort()) {
         try {
             const { result } = await Package.detect({ packageName });
             support.push(result);