Add callback support to bot::{reply,send}

package.json

@@ -25,7 +25,8 @@
   },
   "dependencies": {
     "@slack/client": "^3.4.0",
-    "lodash": "^3.10.1"
+    "lodash": "^3.10.1",
+    "es6-promise": "~4.1.1"
   },
   "devDependencies": {
     "coffee-coverage": "~1.0.1",

src/bot.coffee

@@ -3,6 +3,7 @@
 SlackClient = require './client'
 ReactionMessage = require './reaction-message'
 SlackTextMessage = require './slack-message'
+Promise = global.Promise || require('es6-promise');
 
 # Public: Adds a Listener for ReactionMessages with the provided matcher,
 # options, and callback
@@ -126,23 +127,32 @@ class SlackBot extends Adapter
   Hubot is sending a message to Slack
   ###
   send: (envelope, messages...) ->
+    callback = ->
+    if typeof(messages[messages.length - 1]) == 'function'
+      callback = messages.pop()
+
     sent_messages = []
     for message in messages
       if message isnt ''
         sent_messages.push @client.send(envelope, message)
+    Promise.all(messages).then(callback.bind(null, null), callback)
     return sent_messages
 
 
   ###
   Hubot is replying to a Slack message
   ###
   reply: (envelope, messages...) ->
+    callback = ->
+    if typeof(messages[messages.length - 1]) == 'function'
+      callback = messages.pop()
     sent_messages = []
     for message in messages
       if message isnt ''
         message = "<@#{envelope.user.id}>: #{message}" unless envelope.room[0] is 'D'
         @robot.logger.debug "Sending to #{envelope.room}: #{message}"
         sent_messages.push @client.send(envelope, message)
+    Promise.all(messages).then(callback.bind(null, null), callback)
     return sent_messages
 
 

test/bot.coffee

@@ -64,6 +64,10 @@ describe 'Send Messages', ->
     @stubs._dmmsg.should.eql 'message'
     @stubs._room.should.eql @stubs.user.id
 
+  it 'Should send a message with a callback', (done) ->
+    sentMessages = @slackbot.send {room: 'general'}, 'message', done
+    sentMessages.length.should.equal 1
+    sentMessages[0].should.equal 'message'
 
 describe 'Client sending message', ->
   it 'Should append as_user = true', ->
@@ -96,6 +100,11 @@ describe 'Reply to Messages', ->
     sentMessages.length.should.equal 1
     sentMessages[0].should.equal "message"
 
+  it 'Should call the callback', (done) ->
+    sentMessages = @slackbot.reply {user: @stubs.user, room: @stubs.channel.id}, 'message', done
+    sentMessages.length.should.equal 1
+    sentMessages[0].should.equal "<@#{@stubs.user.id}>: message"
+
 describe 'Setting the channel topic', ->
   it 'Should set the topic in channels', ->
     @slackbot.setTopic {room: @stubs.channel.id}, 'channel'