Merge pull request #16 from boybundit/reply-validation

Validate missing param in reply method

examples/demo.js

@@ -57,6 +57,9 @@ bot.on('message', function (event) {
 				case 'Multiple':
 					return event.reply(['Line 1', 'Line 2', 'Line 3', 'Line 4', 'Line 5']);
 					break;
+				case 'Version':
+					event.reply('linebot@' + process.env.npm_package_version);
+					break;
 				default:
 					event.reply(event.message.text).then(function (data) {
 						console.log('Success', data);

lib/linebot.js

@@ -60,7 +60,8 @@ class LineBot extends EventEmitter {
 	static createMessages(message) {
 		if (typeof message === 'string') {
 			return [{ type: 'text', text: message }];
-		} else if (message.constructor === Array) {
+		}
+		if (Array.isArray(message)) {
 			return message.map(function (m) {
 				if (typeof m === 'string') {
 					return { type: 'text', text: m };

package.json

@@ -1,6 +1,6 @@
 {
   "name": "linebot",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "Node.js SDK for the LINE Messaging API",
   "main": "index.js",
   "files": [

test/linebot.test.js

@@ -75,6 +75,10 @@ describe('linebot', function () {
 			const res = bot.reply('reply token', 'message');
 			assert.equal(Promise, res.constructor);
 		});
+		it('should not crash if message is missing.', function () {
+			const res = bot.reply('reply token');
+			assert.equal(Promise, res.constructor);
+		});
 	});
 	describe('#push()', function () {
 		it('should return a promise.', function () {