fix: update return value; add tests

Author: Michael Perrotte

index.js

@@ -38,7 +38,7 @@ function setAccess (spec, access, opts) {
       body: { access },
       spec
     })
-  }).then(res => res.body.resume() && true)
+  }).then(res => true)
 }
 
 cmd.grant = (spec, entity, permissions, opts) => {

test/index.js

@@ -19,6 +19,16 @@ test('access public', t => {
   })
 })
 
+test('access public - failure', t => {
+  tnock(t, REG).post(
+    '/-/package/%40foo%2Fbar/access', { access: 'public' }
+  ).reply(418)
+  return access.public('@foo/bar', OPTS)
+    .catch(err => {
+      t.equals(err.statusCode, 418, 'fails with code from registry')
+    })
+})
+
 test('access restricted', t => {
   tnock(t, REG).post(
     '/-/package/%40foo%2Fbar/access', { access: 'restricted' }
@@ -28,6 +38,16 @@ test('access restricted', t => {
   })
 })
 
+test('access restricted - failure', t => {
+  tnock(t, REG).post(
+    '/-/package/%40foo%2Fbar/access', { access: 'restricted' }
+  ).reply(418)
+  return access.restricted('@foo/bar', OPTS)
+    .catch(err => {
+      t.equals(err.statusCode, 418, 'fails with code from registry')
+    })
+})
+
 test('access 2fa-required', t => {
   tnock(t, REG).post('/-/package/%40foo%2Fbar/access', {
     publish_requires_tfa: true