fix: display multiple versions when --json on npm view

PR-URL: https://github.com/npm/cli/pull/1657
Credit: @claudiahdz
Close: #1657
Reviewed-by: @isaacs

1 files changed, 110 insertions, 128 deletions

diff --git a/tap-snapshots/test-lib-view.js-TAP.test.js b/tap-snapshots/test-lib-view.js-TAP.test.js
index 6bccea684..f8a9fe464 100644
--- a/tap-snapshots/test-lib-view.js-TAP.test.js
+++ b/tap-snapshots/test-lib-view.js-TAP.test.js
@@ -38,231 +38,215 @@ exports[`test/lib/view.js TAP should log info by field name maintainers with ema
 exports[`test/lib/view.js TAP should log info by field name maintainers with url > must match snapshot 1`] = `
 
 [
-  "claudia (http://c.yellow.com)",
-  "isaacs (http://i.yellow.com)"
+  "claudia (http://c.pink.com)",
+  "isaacs (http://i.pink.com)"
 ]
 `
 
 exports[`test/lib/view.js TAP should log info by field name nested field with brackets > must match snapshot 1`] = `
 
-"Jeremy Ashkenas"
+"123"
 `
 
 exports[`test/lib/view.js TAP should log info by field name readme > must match snapshot 1`] = `
 
-
+a very useful readme
 `
 
 exports[`test/lib/view.js TAP should log info by field name several fields > must match snapshot 1`] = `
 
 {
-  "name": "underscore",
-  "version": "1.3.1"
+  "name": "yellow",
+  "version": "1.0.0"
 }
 `
 
 exports[`test/lib/view.js TAP should log info by field name several fields with several versions > must match snapshot 1`] = `
 
-underscore@1.0.3 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.0.4 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.0 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.1 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.2 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.3 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.4 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.5 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.6 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.1.7 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.2.0 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.2.1 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.2.2 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.2.3 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.2.4 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.3.0 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.3.1 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.3.2 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.3.3 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.4.0 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.4.1 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.4.2 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.4.3 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.4.4 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.5.0 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-underscore@1.5.1 'Jeremy Ashkenas <jeremy@documentcloud.org>'
-`
-
-exports[`test/lib/view.js TAP should log info by field name unknown nested field > must match snapshot 1`] = `
-
-
+yellow@1.0.0 'claudia'
+yellow@1.0.1 'claudia'
+yellow@1.0.2 'claudia'
 `
 
 exports[`test/lib/view.js TAP should log info of package in current working dir non-specific version > must match snapshot 1`] = `
 
 
-underscore@1.3.1 | Proprietary | deps: none | versions: 26
-JavaScript's functional programming helper library.
-http://documentcloud.github.com/underscore/
+blue@1.0.0 | Proprietary | deps: none | versions: 2
 
 dist
-.tarball:http://localhost:1331/underscore/-/underscore-1.3.1.tgz
-.shasum:6cb8aad0e77eb5dbbfb54b22bcd8697309cf9641
-
-maintainers:
--jashkenas <jashkenas@gmail.com>
+.tarball:http://hm.blue.com/1.0.0.tgz
+.shasum:123
+.integrity:---
+.unpackedSize:1 B
 
 dist-tags:
-latest: 1.5.1
-stable: 1.5.1
 
-published over a year ago by jashkenas <jashkenas@gmail.com>
+
+published a year ago
 `
 
 exports[`test/lib/view.js TAP should log info of package in current working dir specific version > must match snapshot 1`] = `
 
 
-underscore@1.3.1 | Proprietary | deps: none | versions: 26
-JavaScript's functional programming helper library.
-http://documentcloud.github.com/underscore/
+blue@1.0.0 | Proprietary | deps: none | versions: 2
 
 dist
-.tarball:http://localhost:1331/underscore/-/underscore-1.3.1.tgz
-.shasum:6cb8aad0e77eb5dbbfb54b22bcd8697309cf9641
-
-maintainers:
--jashkenas <jashkenas@gmail.com>
+.tarball:http://hm.blue.com/1.0.0.tgz
+.shasum:123
+.integrity:---
+.unpackedSize:1 B
 
 dist-tags:
-latest: 1.5.1
-stable: 1.5.1
-
-published over a year ago by jashkenas <jashkenas@gmail.com>
-`
-
-exports[`test/lib/view.js TAP should log package info mkdirp@0.3.5 > must match snapshot 1`] = `
-
 
-mkdirp@0.3.5 | MIT | deps: none | versions: 17
-Recursively mkdir, like \`mkdir -p\`
 
-dist
-.tarball:http://localhost:1331/mkdirp/-/mkdirp-0.3.5.tgz
-.shasum:de3e5f8961c88c787ee1368df849ac4413eca8d7
+published a year ago
+`
 
-maintainers:
--substack <mail@substack.net>
+exports[`test/lib/view.js TAP should log package info package with --json and semver range > must match snapshot 1`] = `
 
-dist-tags:
-latest: 0.3.5
-
-published over a year ago by substack <mail@substack.net>
+[
+  {
+    "_npmUser": "claudia <claudia@cyan.com>",
+    "name": "cyan",
+    "dist-tags": {},
+    "versions": [
+      "1.0.0",
+      "1.0.1"
+    ],
+    "version": "1.0.0",
+    "dist": {
+      "shasum": "123",
+      "tarball": "http://hm.cyan.com/1.0.0.tgz",
+      "integrity": "---",
+      "fileCount": 1,
+      "unpackedSize": 1
+    }
+  },
+  {
+    "_npmUser": "claudia <claudia@cyan.com>",
+    "name": "cyan",
+    "dist-tags": {},
+    "versions": [
+      "1.0.0",
+      "1.0.1"
+    ]
+  }
+]
 `
 
 exports[`test/lib/view.js TAP should log package info package with homepage > must match snapshot 1`] = `
 
 
-orange@undefined | Proprietary | deps: none | versions: 2
+orange@1.0.0 | Proprietary | deps: none | versions: 2
 http://hm.orange.com
 
 dist
-.tarball:undefined
-.shasum:undefined
+.tarball:http://hm.orange.com/1.0.0.tgz
+.shasum:123
 .integrity:---
 .unpackedSize:1 B
 
 dist-tags:
-1.0.1: [object Object]
+latest: 1.0.0
 `
 
 exports[`test/lib/view.js TAP should log package info package with license, bugs, repository and other fields > must match snapshot 1`] = `
 
 
-green@undefined | ACME | deps: 2 | versions: 2
+green@1.0.0 | ACME | deps: 2 | versions: 2
+green is a very important color
 
-DEPRECATED ⚠️  - true
+DEPRECATED!! - true
 
 keywords:colors, green, crayola
 
 bin:green
 
 dist
-.tarball:undefined
-.shasum:undefined
+.tarball:http://hm.green.com/1.0.0.tgz
+.shasum:123
 .integrity:---
 .unpackedSize:1 B
 
 dependencies:
-red: [object Object]
-yellow: [object Object]
+red: 1.0.0
+yellow: 1.0.0
+
+maintainers:
+-claudia <c@yellow.com>
+-isaacs <i@yellow.com>
 
 dist-tags:
-1.0.1: [object Object]
+latest: 1.0.0
 `
 
 exports[`test/lib/view.js TAP should log package info package with maintainers info as object > must match snapshot 1`] = `
 
 
-pink@undefined | Proprietary | deps: none | versions: 2
+pink@1.0.0 | Proprietary | deps: none | versions: 2
 
 dist
-.tarball:undefined
-.shasum:undefined
+.tarball:http://hm.pink.com/1.0.0.tgz
+.shasum:123
 .integrity:---
 .unpackedSize:1 B
 
 dist-tags:
-1.0.1: [object Object]
+latest: 1.0.0
 `
 
 exports[`test/lib/view.js TAP should log package info package with more than 25 deps > must match snapshot 1`] = `
 
 
-black@undefined | Proprietary | deps: 25 | versions: 2
+black@1.0.0 | Proprietary | deps: 25 | versions: 2
 
 dist
-.tarball:undefined
-.shasum:undefined
+.tarball:http://hm.black.com/1.0.0.tgz
+.shasum:123
 .integrity:---
 .unpackedSize:1 B
 
 dependencies:
-0: [object Object]
-10: [object Object]
-11: [object Object]
-12: [object Object]
-13: [object Object]
-14: [object Object]
-15: [object Object]
-16: [object Object]
-17: [object Object]
-18: [object Object]
-19: [object Object]
-1: [object Object]
-20: [object Object]
-21: [object Object]
-22: [object Object]
-23: [object Object]
-2: [object Object]
-3: [object Object]
-4: [object Object]
-5: [object Object]
-6: [object Object]
-7: [object Object]
-8: [object Object]
-9: [object Object]
+0: 1.0.0
+10: 1.0.0
+11: 1.0.0
+12: 1.0.0
+13: 1.0.0
+14: 1.0.0
+15: 1.0.0
+16: 1.0.0
+17: 1.0.0
+18: 1.0.0
+19: 1.0.0
+1: 1.0.0
+20: 1.0.0
+21: 1.0.0
+22: 1.0.0
+23: 1.0.0
+2: 1.0.0
+3: 1.0.0
+4: 1.0.0
+5: 1.0.0
+6: 1.0.0
+7: 1.0.0
+8: 1.0.0
+9: 1.0.0
 (...and 1 more.)
 
 dist-tags:
-1.0.1: [object Object]
+latest: 1.0.0
 `
 
 exports[`test/lib/view.js TAP should log package info package with no modified time > must match snapshot 1`] = `
 
 
-cyan@undefined | Proprietary | deps: none | versions: 2
+cyan@1.0.0 | Proprietary | deps: none | versions: 2
 
 dist
-.tarball:undefined
-.shasum:undefined
+.tarball:http://hm.cyan.com/1.0.0.tgz
+.shasum:123
+.integrity:---
+.unpackedSize:1 B
 
 dist-tags:
 
@@ -273,18 +257,16 @@ published by claudia <claudia@cyan.com>
 exports[`test/lib/view.js TAP should log package info package with no repo or homepage > must match snapshot 1`] = `
 
 
-blue@undefined | Proprietary | deps: none | versions: 2
+blue@1.0.0 | Proprietary | deps: none | versions: 2
 
 dist
-.tarball:undefined
-.shasum:undefined
+.tarball:http://hm.blue.com/1.0.0.tgz
+.shasum:123
+.integrity:---
+.unpackedSize:1 B
 
 dist-tags:
 
 
 published a year ago
 `
-
-exports[`test/lib/view.js TAP should log package info package with no versions > must match snapshot 1`] = `
-
-`