Merge 1aa6dc9 into d06894f

.gitignore

@@ -2,5 +2,5 @@ node_modules/
 package-lock.json
 test.js
 .idea/
-.nyc_output
+.nyc_output/
 build/

factory.js

@@ -2,11 +2,32 @@
 var trainingSet
 var responseSet
 
-function process (query) {
-  return global.responseSet[query]
+var users = {
+  idmap: {},
+
+  addUser: function (token, name) {
+    this.idmap[token] = name
+  },
+
+  get: function (token) {
+    return this.idmap[token]
+  }
 }
 
 function train () {
   global.responseSet = global.trainingSet
 }
+
+function process (query, token) {
+  if (users.get(token)) {
+    return { action: 'response', message: responseSet[query] }
+  } else {
+    return { action: 'unrecognized', message: 'user does not exist' }
+  }
+}
+
+function currentUser (token) {
+  return users.idmap[token]
+}
+
 /* eslint-enable no-unused-vars */

index.js

@@ -9,19 +9,24 @@ class Bot {
     this.factory = new VM()
     this.factory.run(ctx)
     this.factory.run(`trainingSet=${trainingSet}`)
-    this.responses = null
   }
 
   greet () {
     return this.render(this.greeting)
   }
 
-  render (statement) {
-    return statement.replace(/<bot-name>/g, this.name)
+  addUser (token, name) {
+    this.factory.run(`users.addUser("${token}", "${name}")`)
   }
 
-  respond (query) {
-    return this.render(this.factory.run(`process("${query}")`))
+  render (statement, token) {
+    return statement.replace(/<bot-name>/g, this.name).replace(/<customer-name>/g, this.factory.run(`currentUser(${token})`))
+  }
+
+  respond (query, token) {
+    const response = this.factory.run(`process("${query}", "${token}")`)
+    response.message = this.render(response.message, token)
+    return response
   }
 
   train () {

test/factory.js

@@ -2,11 +2,32 @@
 var trainingSet
 var responseSet
 
-function process (query) {
-  return global.responseSet[query]
+var users = {
+  idmap: {},
+
+  addUser: function (token, name) {
+    this.idmap[token] = name
+  },
+
+  get: function (token) {
+    return this.idmap[token]
+  }
 }
 
 function train () {
   global.responseSet = global.trainingSet
 }
+
+function process (query, token) {
+  if (users.get(token)) {
+    return { action: 'response', message: responseSet[query] }
+  } else {
+    return { action: 'unrecognized', message: 'user does not exist' }
+  }
+}
+
+function currentUser (token) {
+  return users.idmap[token]
+}
+
 /* eslint-enable no-unused-vars */

test/initialize-spec.js

@@ -1,13 +1,14 @@
 const chai = require('chai')
 const expect = chai.expect
 const juice = require('../index')
-const trainingSet = { 'hi bot': 'hello', "what's your name?": '<bot-name>' }
+const trainingSet = { 'hi bot': 'hello <customer-name>', "what's your name?": '<bot-name>' }
 
 describe('Initialize', () => {
   let bot
 
   beforeEach(() => {
     bot = new juice.Bot('Jeff', 'Ma Nemma <bot-name>', JSON.stringify(trainingSet))
+    expect(() => bot.addUser('123', 'test-user')).to.not.throw()
   })
 
   it('should set up greeting for bot', () => {
@@ -23,8 +24,22 @@ describe('Initialize', () => {
     expect(bot.factory.run('responseSet')).to.deep.equal(trainingSet)
   })
 
+  it('should recognize registered user from token', () => {
+    expect(() => bot.train()).to.not.throw()
+    expect(bot.respond('hi bot', '123').action).to.equal('response')
+    expect(bot.respond('hi bot', '123').message).to.equal('hello test-user')
+  })
+
+  it('should register new user with corresponsing token', () => {
+    expect(() => bot.train()).to.not.throw()
+    expect(() => bot.addUser('1234', 'user2')).to.not.throw()
+    expect(bot.respond('hi bot', '1234').action).to.equal('response')
+    expect(bot.respond('hi bot', '1234').message).to.equal('hello user2')
+  })
+
   it('should respond to registered query after training', () => {
     expect(() => bot.train()).to.not.throw()
-    expect(bot.respond("what's your name?")).to.equal('Jeff')
+    expect(bot.respond("what's your name?", '123').action).to.equal('response')
+    expect(bot.respond("what's your name?", '123').message).to.equal('Jeff')
   })
 })