fix tests

Author: sonnyp

Makefile

@@ -2,10 +2,10 @@
 
 REMOTE=origin
 
-TESTS = test/*.coffee
+TESTS = test/*.js
 
 test:
-	./node_modules/.bin/mocha --compilers coffee:coffee-script $(TESTS)
+	./node_modules/.bin/mocha $(TESTS)
 
 release: bump-version
 	@echo "Tagging $(VERSION)"

README.md

@@ -65,17 +65,19 @@ hubot-xmpp will then automatically resolve the JID to a private
 chat JID, and private message the sender.
 
 If you need to get the private chat JID, you can use
-'msg.envelope.user.privateChatJid' where msg is the parameter of hubot's
-"route" callback.
+`msg.envelope.user.privateChatJid` where `msg` is the parameter of hubot's
+`route` callback.
 
 Example:
 
-```coffeescript
-robot.respond /talk to me$/i, ( msg ) ->
-  # Simply reply
-  msg.reply "Hello #{msg.envelope.user.name}. Your private JID is #{msg.envelope.user.privateChatJID}"
+```js
+robot.respond(/talk to me$/i, msg => {
+  // Simply reply
+  msg.reply(`Hello ${msg.envelope.user.name}. Your private JID is ${msg.envelope.user.privateChatJID}`);
+});
 
-robot.respond /talk to me in private$/i, ( msg ) ->
-  msg.envelope.user.type = 'direct'
-  msg.send "Hey #{msg.envelope.user.name}! You told me in room #{msg.envelope.user.room} to talk to you."
+robot.respond(/talk to me in private$/i, msg => {
+  msg.envelope.user.type = 'direct';
+  msg.send(`Hey ${msg.envelope.user.name}! You told me in room ${msg.envelope.user.room} to talk to you.`);
+});
 ```

package.json

@@ -32,10 +32,9 @@
     "uuid": "2.0.2"
   },
   "devDependencies": {
-    "coffee-script": "1.1.3",
-    "sinon": "1.9.0",
-    "mocha": "1.15.1",
-    "hubot": ">=2.4.1"
+    "hubot": ">=3.0.0",
+    "mocha": "^6.2.2",
+    "sinon": "1.9.0"
   },
   "volta": {
     "node": "8.15.1",

src/xmpp.js

@@ -9,48 +9,21 @@
  * DS207: Consider shorter variations of null checks
  * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
  */
-const {Adapter,Robot,TextMessage,EnterMessage,LeaveMessage} = require('hubot');
+const {Adapter,Robot,TextMessage,EnterMessage,LeaveMessage} = require('hubot/es2015');
 const {JID, Stanza, Client, parse, Element} = require('node-xmpp-client');
 const uuid = require('uuid');
 const util = require('util');
 
 class XmppBot extends Adapter {
-  static initClass() {
-  
-    this.prototype.reconnectTryCount = 0;
-    this.prototype.currentIqId = 1001;
-    this.prototype.joining = [];
-    this.prototype.joined = [];
-  }
-
   constructor( robot ) {
-    {
-      // Hack: trick Babel/TypeScript into allowing this before super.
-      if (false) { super(); }
-      let thisFn = (() => { return this; }).toString();
-      let thisName = thisFn.match(/return (?:_assertThisInitialized\()*(\w+)\)*;/)[1];
-      eval(`${thisName} = this;`);
-    }
-    this.error = this.error.bind(this);
-    this.online = this.online.bind(this);
-    this.ping = this.ping.bind(this);
-    this.read = this.read.bind(this);
-    this.readIq = this.readIq.bind(this);
-    this.readMessage = this.readMessage.bind(this);
-    this.readPresence = this.readPresence.bind(this);
-    this.offline = this.offline.bind(this);
-    this.checkCanStart = this.checkCanStart.bind(this);
-    this.robot = robot;
+    super(robot)
 
     // Flag to log a warning message about group chat configuration only once
     this.anonymousGroupChatWarningLogged = false;
 
     // Store the room JID to private JID map.
     // Key is the room JID, value is the private JID
     this.roomToPrivateJID = {};
-
-    // http://stackoverflow.com/a/646643
-    if (String.prototype.startsWith == null) { String.prototype.startsWith = function(s) { return this.slice(0, s.length) === s; }; }
   }
 
   run() {
@@ -592,7 +565,11 @@ class XmppBot extends Adapter {
     }
   }
 }
-XmppBot.initClass();
+
+XmppBot.prototype.reconnectTryCount = 0;
+XmppBot.prototype.currentIqId = 1001;
+XmppBot.prototype.joining = [];
+XmppBot.prototype.joined = [];
 
 exports.use = robot => new XmppBot(robot);
 
@@ -605,4 +582,4 @@ function __guardMethod__(obj, methodName, transform) {
 }
 function __guard__(value, transform) {
   return (typeof value !== 'undefined' && value !== null) ? transform(value) : undefined;
-}
+}

test/adapter-test.js

@@ -1317,7 +1317,7 @@ describe('XmppBot', function() {
         name: 'message',
         flag: 'ignore_me'
       };
-      const proxied = bot.readMessage;
+      const proxied = bot.readMessage.bind(bot);
       bot.readMessage = function(message) {
         proxied(message);
         if (message.flag === 'ignore_me') {
@@ -1347,7 +1347,7 @@ describe('XmppBot', function() {
           };
         }
       };
-      const proxied = bot.readMessage;
+      const proxied = bot.readMessage.bind(bot);
       bot.readMessage = function(message) {
         proxied(message);
         assert.equal(true, Array.from(bot.joined).includes('test@example.com'));