amqplib 0.5.0 → 0.5.1

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Change log for amqplib
 
+## Changes in v0.5.1
+
+    git log v0.5.0..v0.5.1
+
+ * Fix mistake in closeBecause
+   ([PR 298](https://github.com/squaremo/amqp.node/pull/298); thanks
+   to @lholznagel and others who reported the issue, and to @nfantone
+   for the rapid fix)
+
 ## Changes in v0.5.0
 
     git log v0.4.2..v0.5.0

package/lib/channel_model.js

@@ -73,8 +73,9 @@ C.open = function() {
 };
 
 C.close = function() {
+  var self = this;
   return Promise.fromCallback(function(cb) {
-    return this.closeBecause("Goodbye", defs.constants.REPLY_SUCCESS,
+    return self.closeBecause("Goodbye", defs.constants.REPLY_SUCCESS,
                     cb);
   });
 };

package/package.json

@@ -2,7 +2,7 @@
   "name": "amqplib",
   "homepage": "http://squaremo.github.io/amqp.node/",
   "main": "./channel_api.js",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "An AMQP 0-9-1 (e.g., RabbitMQ) library and client.",
   "repository": {
     "type": "git",

package/test/channel_api.js

@@ -40,7 +40,7 @@ function channel_test(chmethod, name, chfun) {
       c[chmethod]().then(ignoreErrors).then(chfun)
         .then(succeed(done), fail(done))
       // close the connection regardless of what happens with the test
-        .then(function() {c.close();});
+        .finally(function() {c.close();});
     });
   });
 }
@@ -509,6 +509,12 @@ chtest("prefetch", function(ch) {
     });
 });
 
+chtest('close', function(ch) {
+  // Resolving promise guarantees
+  // channel is closed
+  return ch.close();
+});
+
 });
 
 var confirmtest = channel_test.bind(null, 'createConfirmChannel');