feat(pack): add workspace support

PR-URL: https://github.com/npm/cli/pull/3033
Credit: @wraithgar
Close: #3033
Reviewed-by: @darcyclarke

2 files changed, 117 insertions, 0 deletions

diff --git a/test/fixtures/mock-npm.js b/test/fixtures/mock-npm.js
index c47758111..01f482bde 100644
--- a/test/fixtures/mock-npm.js
+++ b/test/fixtures/mock-npm.js
@@ -2,10 +2,24 @@
 // npm.config You still need a separate flatOptions but this is the first step
 // to eventually just using npm itself
 
+const mockLog = {
+  clearProgress: () => {},
+  disableProgress: () => {},
+  enableProgress: () => {},
+  http: () => {},
+  info: () => {},
+  levels: [],
+  notice: () => {},
+  pause: () => {},
+  silly: () => {},
+  verbose: () => {},
+  warn: () => {},
+}
 const mockNpm = (base = {}) => {
   const config = base.config || {}
   const flatOptions = base.flatOptions || {}
   return {
+    log: mockLog,
     ...base,
     flatOptions,
     config: {
diff --git a/test/lib/pack.js b/test/lib/pack.js
index a5163acfd..4bcfe8ae3 100644
--- a/test/lib/pack.js
+++ b/test/lib/pack.js
@@ -1,6 +1,7 @@
 const t = require('tap')
 const requireInject = require('require-inject')
 const mockNpm = require('../fixtures/mock-npm')
+const pacote = require('pacote')
 
 const OUTPUT = []
 const output = (...msg) => OUTPUT.push(msg)
@@ -11,6 +12,16 @@ const libnpmpack = async (spec, opts) => {
 
   return ''
 }
+const mockPacote = {
+  manifest: (spec) => {
+    if (spec.type === 'directory')
+      return pacote.manifest(spec)
+    return {
+      name: spec.name || 'test-package',
+      version: spec.version || '1.0.0-test',
+    }
+  },
+}
 
 t.afterEach(cb => {
   OUTPUT.length = 0
@@ -152,3 +163,95 @@ t.test('should log pack contents', (t) => {
     t.end()
   })
 })
+
+t.test('workspaces', (t) => {
+  const testDir = t.testdir({
+    'package.json': JSON.stringify({
+      name: 'workspaces-test',
+      version: '1.0.0',
+      workspaces: ['workspace-a', 'workspace-b'],
+    }, null, 2),
+    'workspace-a': {
+      'package.json': JSON.stringify({
+        name: 'workspace-a',
+        version: '1.0.0',
+      }),
+    },
+    'workspace-b': {
+      'package.json': JSON.stringify({
+        name: 'workspace-b',
+        version: '1.0.0',
+      }),
+    },
+  })
+  const Pack = requireInject('../../lib/pack.js', {
+    libnpmpack,
+    pacote: mockPacote,
+    npmlog: {
+      notice: () => {},
+      showProgress: () => {},
+      clearProgress: () => {},
+    },
+  })
+  const npm = mockNpm({
+    localPrefix: testDir,
+    config: {
+      unicode: false,
+      json: false,
+      'dry-run': false,
+    },
+    output,
+  })
+  const pack = new Pack(npm)
+
+  t.test('all workspaces', (t) => {
+    pack.execWorkspaces([], [], er => {
+      if (er)
+        throw er
+
+      t.strictSame(OUTPUT, [
+        ['workspace-a-1.0.0.tgz'],
+        ['workspace-b-1.0.0.tgz'],
+      ])
+      t.end()
+    })
+  })
+
+  t.test('all workspaces, `.` first arg', (t) => {
+    pack.execWorkspaces(['.'], [], er => {
+      if (er)
+        throw er
+
+      t.strictSame(OUTPUT, [
+        ['workspace-a-1.0.0.tgz'],
+        ['workspace-b-1.0.0.tgz'],
+      ])
+      t.end()
+    })
+  })
+
+  t.test('one workspace', (t) => {
+    pack.execWorkspaces([], ['workspace-a'], er => {
+      if (er)
+        throw er
+
+      t.strictSame(OUTPUT, [
+        ['workspace-a-1.0.0.tgz'],
+      ])
+      t.end()
+    })
+  })
+
+  t.test('specific package', (t) => {
+    pack.execWorkspaces(['abbrev'], [], er => {
+      if (er)
+        throw er
+
+      t.strictSame(OUTPUT, [
+        ['abbrev-1.0.0-test.tgz'],
+      ])
+      t.end()
+    })
+  })
+  t.end()
+})