feat: adds examples; refactors for auth response

examples/.gitignore

@@ -0,0 +1 @@
+package-lock.json

examples/mq/subscribe.js

@@ -0,0 +1,17 @@
+const Tillhub = require('@tillhub/node-sdk')
+
+const th = new Tillhub({
+  base: process.env.TILLHUB_BASE,
+  credentials: {
+    id: process.env.CLIENT_ACCOUNT,
+    apiKey: process.env.API_KEY
+  }
+})
+
+function login () {
+  th.init((err, authResponse, client, authInstance) => {
+    if (err) throw err
+  })
+}
+
+login()

examples/package.json

@@ -0,0 +1,18 @@
+{
+  "name": "examples",
+  "version": "0.1.0",
+  "description": "",
+  "main": "index.js",
+  "scripts": {
+    "run": "node -r dotenv/config",
+    "mq:subscribe": "npm run run -- mq/subscribe.js"
+  },
+  "author": "",
+  "license": "ISC",
+  "dependencies": {
+    "@tillhub/node-sdk": "*"
+  },
+  "devDependencies": {
+    "node": "^10.10.0"
+  }
+}

lib/index.js

@@ -48,15 +48,15 @@ class Tillhub extends EventEmitter {
   /**
    * Initialise the SDK instance by authenticating the client
    *
-   * @param {Function?} callback optional callback. I fnot specified a Promise will be returned
+   * @param {Function?} callback optional callback. If not specified and credentials are set we return a Promise. When called synchronously, we will only prepare the SDK
    */
   init (callback) {
-    if (!this.options.credentials) {
+    this.auth = new Auth({ credentials: this.options.credentials })
+
+    if (!this.options.credentials && callback) {
       return callback(errors.generate(errors.constants.INSUFFICIENT_CREDENTIALS))
     }
 
-    this.auth = new Auth({ credentials: this.options.credentials })
-
     this.auth.authenticate((err, authResponse) => {
       if (err) return callback(err)
 
@@ -69,7 +69,7 @@ class Tillhub extends EventEmitter {
         }
       })
 
-      return callback(null, this, this.auth)
+      return callback(null, authResponse, this, this.auth)
     })
   }
 

lib/v0/auth.js

@@ -21,6 +21,9 @@ const errors = require('../errors')
 class Auth extends EventEmitter {
   constructor (options = {}) {
     super()
+
+    this.subUser = null
+
     this.options = {
       credentials: {},
       base: process.env.TILLHUB_BASE || 'https://api.tillhub.com',

lib/v1/auth.js

@@ -23,6 +23,9 @@ const errors = require('../errors')
 class Auth extends AuthV0 {
   constructor (options = {}) {
     super()
+
+    this.subUser = null
+
     this.options = {
       credentials: {},
       base: process.env.TILLHUB_BASE || 'https://api.tillhub.com',
@@ -80,8 +83,9 @@ class Auth extends AuthV0 {
       if (resp.statusCode === 200) {
         this.token = body.token
         this.user = body.user.legacy_id || body.user.id
+        this.subUser = body.sub_user
 
-        return callback(null, { token: this.token, user: this.user })
+        return callback(null, { token: this.token, user: this.user, subUser: this.subUser, key: this.subUser ? this.subUser.key : null })
       }
 
       return callback(new Error('tillhub: could not get serviceaccount login info'), body, resp)

test/auth/class-auth.spec.js

@@ -57,7 +57,7 @@ test('root: Can init with auth credentials', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     t.ok(th.auth)

test/carts/create.spec.js

@@ -68,7 +68,7 @@ test('v1: carts: can create cart from existing products', function (t) {
       })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const carts = th.carts()

test/carts/get.spec.js

@@ -56,7 +56,7 @@ test('v1: carts: can fetch some carts', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const carts = th.carts()

test/products/get.spec.js

@@ -56,7 +56,7 @@ test('v1: products: can fetch some products', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const products = th.products()
@@ -126,7 +126,7 @@ test('v1: Products: can fetch paged products', function (t) {
       })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const products = th.products()

test/templates/create.spec.js

@@ -56,7 +56,7 @@ test('v1: Templates: can create template', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const templates = th.templates()

test/templates/get.spec.js

@@ -56,7 +56,7 @@ test('v1: Templates: can fetch some templates', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const templates = th.templates()

test/transactions/get.spec.js

@@ -56,7 +56,7 @@ test('v0: Transactions: can fetch some transactions', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const transactions = th.transactions()
@@ -126,7 +126,7 @@ test('v0: Transactions: can fetch paged transactions', function (t) {
       })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const transactions = th.transactions()

test/vouchers/create.spec.js

@@ -56,7 +56,7 @@ test('v0: Vouchers: can create voucher', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const vouchers = th.vouchers()

test/vouchers/get.logs.spec.js

@@ -65,7 +65,7 @@ test('v1: Vouchers: can fetch some vouchers', function (t) {
       })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const vouchers = th.vouchers()

test/vouchers/get.spec.js

@@ -56,7 +56,7 @@ test('v0: Vouchers: can fetch some vouchers', function (t) {
     })
   }
 
-  th.init((err, client, authInstance) => {
+  th.init((err, authResponse, client, authInstance) => {
     t.error(err)
 
     const vouchers = th.vouchers()