fix: fixed function return type; added tests to catch a change like this

Author: Michael Perrotte

index.js

@@ -63,5 +63,5 @@ cmd.ls.stream = (org, opts = {}) => {
     mapJSON: (value, [key]) => {
       return [key, value]
     }
-  }).collect()
+  })
 }

package.json

@@ -23,6 +23,7 @@
     "check-coverage": true
   },
   "devDependencies": {
+    "minipass": "^3.1.1",
     "nock": "^12.0.1",
     "standard": "^14.3.1",
     "standard-version": "^7.1.0",

test/index.js

@@ -1,5 +1,6 @@
 'use strict'
 
+const Minipass = require('minipass')
 const test = require('tap').test
 const tnock = require('./fixtures/tnock.js')
 
@@ -94,27 +95,35 @@ test('ls', t => {
 })
 
 test('ls stream with no options', t => {
+  t.plan(2)
   const roster = {
     zkat: 'developer',
     iarna: 'admin',
     isaacs: 'owner'
   }
   const rosterArr = Object.keys(roster).map(k => [k, roster[k]])
   tnock(t, REG).get('/-/org/myorg/user').reply(200, roster)
-  return org.ls.stream('myorg').then(res => {
-    t.deepEqual(res, rosterArr, 'got back a roster, in entries format')
-  })
+  const result = org.ls.stream('myorg')
+  t.ok(Minipass.isStream(result), 'returns a stream')
+  return result.collect()
+    .then(res => {
+      t.deepEqual(res, rosterArr, 'got back a roster, in entries format')
+    })
 })
 
 test('ls stream', t => {
+  t.plan(2)
   const roster = {
     zkat: 'developer',
     iarna: 'admin',
     isaacs: 'owner'
   }
   const rosterArr = Object.keys(roster).map(k => [k, roster[k]])
   tnock(t, OPTS.registry).get('/-/org/myorg/user').reply(200, roster)
-  return org.ls.stream('myorg', OPTS).then(res => {
-    t.deepEqual(res, rosterArr, 'got back a roster, in entries format')
-  })
+  const result = org.ls.stream('myorg', OPTS)
+  t.ok(Minipass.isStream(result), 'returns a stream')
+  return result.collect()
+    .then(res => {
+      t.deepEqual(res, rosterArr, 'got back a roster, in entries format')
+    })
 })