Give package test modules access to Npm.depends dependencies.

Similar in spirit to bbac272.

Fixes meteor#7999.

History.md

@@ -38,6 +38,7 @@
   [#7154](https://github.com/meteor/meteor/issues/7154)
   [#7956](https://github.com/meteor/meteor/issues/7956)
   [#7974](https://github.com/meteor/meteor/issues/7974)
+  [#7999](https://github.com/meteor/meteor/issues/7999)
   [#8005](https://github.com/meteor/meteor/issues/8005)
   [#8007](https://github.com/meteor/meteor/issues/8007)
 

tools/isobuild/import-scanner.js

@@ -648,7 +648,12 @@ export default class ImportScanner {
     if (this.name) {
       // If we're bundling a package, prefix path with
       // node_modules/<package name>/.
-      path = pathJoin("node_modules", "meteor", this.name, path);
+      path = pathJoin(
+        "node_modules",
+        "meteor",
+        this.name.replace(/^local-test[:_]/, ""),
+        path,
+      );
     }
 
     // Install paths should always be delimited by /.

tools/tests/apps/modules/package.json

@@ -16,6 +16,7 @@
     "winston": "^2.2.0"
   },
   "scripts": {
-    "test": "METEOR_PROFILE=100 ../../../../meteor test --full-app --driver-package avital:mocha"
+    "test": "METEOR_PROFILE=100 ../../../../meteor test --full-app --driver-package avital:mocha",
+    "test-packages": "../../../../meteor test-packages --driver-package avital:mocha packages/modules-test-package"
   }
 }

tools/tests/apps/modules/packages/modules-test-package/package.js

@@ -7,7 +7,8 @@ Package.describe({
 
 Npm.depends({
   "os-browserify": "0.2.0",
-  "assert": "1.3.0"
+  "assert": "1.3.0",
+  "cheerio": "0.22.0"
 });
 
 Package.onUse(function(api) {
@@ -17,3 +18,10 @@ Package.onUse(function(api) {
   api.mainModule("server.js", "server");
   api.export("ModulesTestPackage");
 });
+
+Package.onTest(function (api) {
+  api.use("ecmascript");
+  api.use("tinytest");
+  api.use("modules-test-package");
+  api.mainModule("tests.js");
+});

tools/tests/apps/modules/packages/modules-test-package/tests.js

@@ -0,0 +1,9 @@
+import assert from "assert";
+
+describe("cheerio", () => {
+  it("should be importable", () => {
+    import cheerio from "cheerio";
+    assert.strictEqual(typeof cheerio, "object");
+    assert.deepEqual(Object.keys(cheerio), ["cheerio"]);
+  });
+});