Added Test option to package add

README.md

@@ -124,6 +124,12 @@ Turn on debug logging
 openupm --verbose ...
 ```
 
+Add [testables](https://docs.unity3d.com/Manual/cus-tests.html) when importing
+
+```
+openupm --test ...
+```
+
 ## Work with Unity official (upstream) registry
 
 Most commands can fallback to Unity upstream registry if necessary, to make it easier to mix the official registry with a 3rd-party registry. i.e.

lib/cli.js

@@ -18,7 +18,8 @@ program
   .option("-c, --chdir <path>", "change the working directory")
   .option("-r, --registry <url>", "specify registry url")
   .option("-v, --verbose", "output extra debugging")
-  .option("--no-upstream", "don't use upstream unity registry");
+  .option("--no-upstream", "don't use upstream unity registry")
+  .option("-t", "--test", "add package as testable");
 
 program
   .command("add <pkg> [otherPkgs...]")

lib/cmd-add.js

@@ -129,6 +129,14 @@ const _add = async function(pkg) {
     });
     entry.scopes = Array.from(scopesSet).sort();
   }
+  if (env.testables) {
+    if (!manifest.testables) {
+      manifest.testables = [];
+    }
+    if (manifest.testables.indexOf(name) === -1) {
+      manifest.testables.push(name);
+    }
+  }
   // save manifest
   if (dirty) {
     if (!saveManifest(manifest)) return { code: 1, dirty };

lib/core.js

@@ -26,6 +26,7 @@ const parseEnv = function(options, { checkPath }) {
   env.manifestPath = "";
   env.upstream = true;
   env.upstreamRegistry = "https://packages.unity.com";
+  env.testables = false;
   // log level
   const logLevel = options.parent.verbose ? "debug" : "info";
   log.setLevel(logLevel);
@@ -63,6 +64,9 @@ const parseEnv = function(options, { checkPath }) {
     log.error(`can not locate manifest.json at path ${manifestPath}`);
     return false;
   } else env.manifestPath = manifestPath;
+  if (options.parent.test) {
+    env.testables = true;
+  }
   // return
   return true;
 };

test/test-cmd-add.js

@@ -28,6 +28,14 @@ describe("cmd-add.js", function() {
       chdir: getWorkDir("test-openupm-cli")
     }
   };
+  const testableOptions = {
+    parent: {
+      registry: "http://example.com",
+      upstream: false,
+      chdir: getWorkDir("test-openupm-cli"),
+      test: true
+    }
+  };
   describe("add", function() {
     let stdout;
     let stderr;
@@ -151,6 +159,19 @@ describe("cmd-add.js", function() {
         "com.upstream.package-up": "1.0.0"
       }
     };
+    const expectedManifestTestable = {
+      dependencies: {
+        "com.base.package-a": "1.0.0"
+      },
+      scopedRegistries: [
+        {
+          name: "example.com",
+          scopes: ["com.base.package-a", "com.example"],
+          url: "http://example.com"
+        }
+      ],
+      testables: ["com.base.package-a"]
+    };
     beforeEach(function() {
       removeWorkDir("test-openupm-cli");
       createWorkDir("test-openupm-cli", { manifest: true });
@@ -400,5 +421,19 @@ describe("cmd-add.js", function() {
         .includes("manifest updated")
         .should.be.ok();
     });
+    it("add pkg with tests", async function() {
+      const retCode = await add("com.base.package-a", testableOptions);
+      retCode.should.equal(0);
+      const manifest = await loadManifest();
+      manifest.should.be.deepEqual(expectedManifestTestable);
+      stdout
+        .captured()
+        .includes("added: ")
+        .should.be.ok();
+      stdout
+        .captured()
+        .includes("manifest updated")
+        .should.be.ok();
+    });
   });
 });