@heroku/skynet 1.6.7 → 1.6.9

package/README.md

@@ -36,8 +36,54 @@ heroku login # login to the staging cloud via the cli
 heroku skynet:<cmd>
 ```
 
+### Unit test
+```
+# $SKYNET_CLI_HOME points to the home dir of heroku-skynet-cli 
+cd $SKYNET_CLI_HOME
+unset SKYNET_HOST
+npm test
+```
+
+### Integration test
+```
+#
+# Start local skynet
+#
+
+# start postgres DB and redis
+$ pg_ctl -D /usr/local/var/postgres start
+$ redis-server
+
+# Update URL and KEY in .env. $REAL_KEY_IN_PRODUCTION refers to the real key in production.
+$ sed -i -e 's/HEROKU_API_URL=.*/HEROKU_API_URL=https:\/\/api.heroku.com/g' .env  
+$ sed -i -e 's/HEROKU_API_KEY=.*/HEROKU_API_KEY=$REAL_KEY_IN_PRODUCTION/g'  .env
+
+# Start skynet instance
+$ cd $SKYNET_HOME
+$ heroku local web,worker
+
+# 
+# Config heroku-skynet-cli
+#
+
+# Pointing CLI to the Heroku Production
+$ cd $HEROKU_SKYNET_CLI_HOME
+$ cloud production
+
+# Pointing to the local Skynet instance
+$ export SKYNET_HOST=http://localhost:5000
+
+#
+# Run Tests
+#
+
+# This unsuspend request will be sent to the local Skynet instance
+$ heroku skynet::unsuspend::user -u dzhuo@heroku.com 
+
+```
+
 ### Publishing to `npm`
-1. Send an email to `heroku-help@salesforce.com` requesting access to @heroku on npm. Include your npm username (https://www.npmjs.com/)
+1. Request access to NPM using [this form](https://securityorg.force.com/IntakeApp/s/?entityCode=HerokuAccess&si=a1i3A0000007euiQAA)
 
 1. Publish the package:
 

package/commands/suspend/user.js

@@ -42,7 +42,7 @@ async function run (context) {
 
   if (file) {
     let users = await readlines(file)
-    let response = await skynet.bulkSuspendUsers(users.join(), notes, category)
+    let response = await skynet.bulkSuspendUsers(users.join(), notes, category, notify, force)
     cli.log(response)
   } else {
     if (!user) {

package/commands/unsuspend/apps.js

@@ -6,9 +6,8 @@ let SkynetAPI = require('../../lib/skynet')
 async function run (context) {
   sudo()
   const skynet = new SkynetAPI(context.auth.password)
-  let response = await skynet.unsuspendApp(context.flags.app, context.flags.notes)
-  response = JSON.parse(response)
-  cli.log(`suspending...${cli.color.cyan(response.status)}.\n${response.message}`)
+  const response = await cli.action(`unsuspending ${cli.color.cyan(context.flags.app)}`, skynet.unsuspendApp(context.flags.app))
+  cli.log(`unsuspending...${cli.color.cyan(response.status)}.\n${response.message}`)
 }
 
 module.exports = {

package/commands/unsuspend/user.js

@@ -7,9 +7,7 @@ async function run (context) {
   sudo()
   const skynet = new SkynetAPI(context.auth.password)
   let user = context.flags.user || process.env.HEROKU_USER
-
-  let response = await cli.action(`unsuspending ${cli.color.cyan(user)}`, skynet.unsuspendUser(user))
-  response = JSON.parse(response)
+  const response = await cli.action(`unsuspending ${cli.color.cyan(user)}`, skynet.unsuspendUser(user))
   cli.log(`${response.status}. ${response.message}`)
 }
 

package/lib/skynet.js

@@ -9,7 +9,7 @@ const SKYNET_BASE_URL = `${skynetHost}/api-h`
 
 module.exports = class SkynetAPI {
   constructor (token) {
-    let packageJSON = require('../package.json')
+    const packageJSON = require('../package.json')
     this.version = `skynet-${packageJSON.version}`
     this.token = token
   }
@@ -44,7 +44,9 @@ module.exports = class SkynetAPI {
 
   removeWhitelist (appOrUserEmail) {
     return this.request(`/whitelist/${appOrUserEmail}`, {
-      method: 'DELETE'
+      method: 'DELETE',
+      body: {},
+      json: true
     })
   }
 
@@ -108,7 +110,9 @@ module.exports = class SkynetAPI {
 
   unsuspendApp (app) {
     return this.request(`/suspensions/app/${app}`, {
-      method: 'DELETE'
+      method: 'DELETE',
+      body: {},
+      json: true
     })
   }
 
@@ -131,17 +135,21 @@ module.exports = class SkynetAPI {
 
   unsuspendUser (user) {
     return this.request(`/suspensions/user/${user}`, {
-      method: 'DELETE'
+      method: 'DELETE',
+      body: {},
+      json: true
     })
   }
 
-  bulkSuspendUsers (users, notes, category) {
+  bulkSuspendUsers (users, notes, category, notify, force = false) {
     var body = {
       value: users,
       reason: notes,
       method: 'skynet-cli',
       category: category,
-      bulk: 'true'
+      bulk: 'true',
+      force: force,
+      notify: notify
     }
 
     return this.request(`/suspend/user`, {
@@ -184,7 +192,7 @@ module.exports = class SkynetAPI {
   }
 
   addCategory (category, description) {
-    let body = {
+    const body = {
       category: category,
       description: description
     }
@@ -198,7 +206,9 @@ module.exports = class SkynetAPI {
 
   removeCategory (category) {
     return this.request(`/categories/${category}`, {
-      method: 'DELETE'
+      method: 'DELETE',
+      body: {},
+      json: true
     })
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heroku/skynet",
-  "version": "1.6.7",
+  "version": "1.6.9",
   "description": "use Skynet from Heroku CLI",
   "main": "index.js",
   "author": "Bob Argenbright @byt3smith",