Merge pull request #30 from dpatekar/master

fixed unsubscribe handler exception

lib/handlers/unsubscribe.js

@@ -37,14 +37,16 @@ function actualUnsubscribe (client, packet, done) {
 function doUnsubscribe (sub, done) {
   var client = this.client
   var broker = client.broker
-  var func = client.subscriptions[sub].func
-
-  delete client.subscriptions[sub]
-
-  broker.unsubscribe(
-    sub,
-    func,
-    done)
+  if (client.subscriptions[sub]) {
+    var func = client.subscriptions[sub].func
+    delete client.subscriptions[sub]
+    broker.unsubscribe(
+      sub,
+      func,
+      done)
+  } else {
+    done()
+  }
 }
 
 function completeUnsubscribe (err) {

test/basic.js

@@ -156,6 +156,29 @@ test('unsubscribe', function (t) {
   })
 })
 
+test('unsubscribe without subscribe', function (t) {
+  t.plan(1)
+
+  var s = noError(connect(setup()), t)
+
+  s.inStream.write({
+    cmd: 'unsubscribe',
+    messageId: 43,
+    unsubscriptions: ['hello']
+  })
+
+  s.outStream.once('data', function (packet) {
+    t.deepEqual(packet, {
+      cmd: 'unsuback',
+      messageId: 43,
+      dup: false,
+      length: 2,
+      qos: 0,
+      retain: false
+    }, 'packet matches')
+  })
+})
+
 test('unsubscribe on disconnect', function (t) {
   var s = noError(connect(setup()), t)