amqplib 0.4.1 → 0.5.2

package/test/connect.js

@@ -1,21 +1,65 @@
 'use strict';
 
 var connect = require('../lib/connect').connect;
+var Buffer = require('safe-buffer').Buffer
+var credentialsFromUrl = require('../lib/connect').credentialsFromUrl;
 var assert = require('assert');
 var util = require('./util');
 var net = require('net');
 var fail = util.fail, succeed = util.succeed,
-    kCallback = util.kCallback;
+    kCallback = util.kCallback,
+    succeedIfAttributeEquals = util.succeedIfAttributeEquals;
+var format = require('util').format;
 
 var URL = process.env.URL || 'amqp://localhost';
 
+suite("Credentials", function() {
+
+  function checkCreds(creds, user, pass, done) {
+    if (creds.mechanism != 'PLAIN') {
+      return done('expected mechanism PLAIN');
+    }
+    if (creds.username != user || creds.password != pass) {
+      return done(format("expected '%s', '%s'; got '%s', '%s'",
+                         user, pass, creds.username, creds.password));
+    }
+    done();
+  }
+
+  test("no creds", function(done) {
+    var parts = {auth: ''};
+    var creds = credentialsFromUrl(parts);
+    checkCreds(creds, 'guest', 'guest', done);
+  });
+  test("usual user:pass", function(done) {
+    var parts = {auth: 'user:pass'};
+    var creds = credentialsFromUrl(parts);
+    checkCreds(creds, 'user', 'pass', done);
+  });
+  test("missing user", function(done) {
+    var parts = {auth: ':password'};
+    var creds = credentialsFromUrl(parts);
+    checkCreds(creds, '', 'password', done);
+  });
+  test("missing password", function(done) {
+    var parts = {auth: 'username'};
+    var creds = credentialsFromUrl(parts);
+    checkCreds(creds, 'username', '', done);
+  });
+  test("colon in password", function(done) {
+    var parts = {auth: 'username:pass:word'};
+    var creds = credentialsFromUrl(parts);
+    checkCreds(creds, 'username', 'pass:word', done);
+  });
+});
+
 suite("Connect API", function() {
 
   test("Connection refused", function(done) {
     connect('amqp://localhost:23450', {},
             kCallback(fail(done), succeed(done)));
   });
-  
+
   // %% this ought to fail the promise, rather than throwing an error
   test("bad URL", function() {
     assert.throws(function() {
@@ -33,6 +77,22 @@ suite("Connect API", function() {
     connect(u, {}, kCallback(fail(done), succeed(done)));
   });
 
+  test("using custom heartbeat option", function(done) {
+    var url = require('url');
+    var parts = url.parse(URL, true);
+    var config = parts.query || {};
+    config.heartbeat = 20;
+    connect(config, {}, kCallback(succeedIfAttributeEquals('heartbeat', 20, done), fail(done)));
+  });
+
+  test("wrongly typed heartbeat option", function(done) {
+    var url = require('url');
+    var parts = url.parse(URL, true);
+    var config = parts.query || {};
+    config.heartbeat = 'NOT A NUMBER';
+    connect(config, {}, kCallback(fail(done), succeed(done)));
+  });
+
   test("using plain credentials", function(done) {
     var url = require('url');
     var parts = url.parse(URL, true);
@@ -48,7 +108,7 @@ suite("Connect API", function() {
   test("using unsupported mechanism", function(done) {
     var creds = {
       mechanism: 'UNSUPPORTED',
-      response: function() { return new Buffer(''); }
+      response: function() { return Buffer.from(''); }
     };
     connect(URL, {credentials: creds},
             kCallback(fail(done), succeed(done)));

package/test/connection.js

@@ -2,6 +2,7 @@
 
 var assert = require('assert');
 var defs = require('../lib/defs');
+var Buffer = require('safe-buffer').Buffer;
 var Connection = require('../lib/connection').Connection;
 var HEARTBEAT = require('../lib/frame').HEARTBEAT;
 var HB_BUF = require('../lib/frame').HEARTBEAT_BUF;
@@ -16,7 +17,7 @@ var OPEN_OPTS = {
   // start-ok
   'clientProperties': {},
   'mechanism': 'PLAIN',
-  'response': new Buffer(['', 'guest', 'guest'].join(String.fromCharCode(0))),
+  'response': Buffer.from(['', 'guest', 'guest'].join(String.fromCharCode(0))),
   'locale': 'en_US',
   
   // tune-ok
@@ -37,8 +38,8 @@ function happy_open(send, await) {
        {versionMajor: 0,
         versionMinor: 9,
         serverProperties: {},
-        mechanisms: new Buffer('PLAIN'),
-        locales: new Buffer('en_US')});
+        mechanisms: Buffer.from('PLAIN'),
+        locales: Buffer.from('en_US')});
   return await(defs.ConnectionStartOk)()
     .then(function(f) {
       send(defs.ConnectionTune,
@@ -65,7 +66,7 @@ function connectionTest(client, server) {
 
     // NB only not a race here because the writes are synchronous
     var protocolHeader = pair.server.read(8);
-    assert.deepEqual(new Buffer("AMQP" + String.fromCharCode(0,0,9,1)),
+    assert.deepEqual(Buffer.from("AMQP" + String.fromCharCode(0,0,9,1)),
                      protocolHeader);
 
     var s = util.runServer(pair.server, function(send, await) {
@@ -92,7 +93,7 @@ suite("Connection errors", function() {
     var ss = util.socketPair();
     var conn = new (require('../lib/connection').Connection)(ss.client);
     ss.server.on('readable', function() {
-      ss.server.write(new Buffer([0, 0, 0, 0, 0, 0, 0, 0, 0, 0]));
+      ss.server.write(Buffer.from([0, 0, 0, 0, 0, 0, 0, 0, 0, 0]));
     });
     conn.open({}, kCallback(fail(done), succeed(done)));
   });
@@ -132,8 +133,8 @@ test("unexpected socket close", connectionTest(
          {versionMajor: 0,
           versionMinor: 9,
           serverProperties: {},
-          mechanisms: new Buffer('PLAIN'),
-          locales: new Buffer('en_US')});
+          mechanisms: Buffer.from('PLAIN'),
+          locales: Buffer.from('en_US')});
     return await(defs.ConnectionStartOk)()
       .then(function() {
         socket.end();
@@ -156,7 +157,7 @@ test("wrong frame on channel 0", connectionTest(
         // there's actually nothing that would plausibly be sent to a
         // just opened connection, so this is violating more than one
         // rule. Nonetheless.
-        send(defs.ChannelOpenOk, {channelId: new Buffer('')}, 0);
+        send(defs.ChannelOpenOk, {channelId: Buffer.from('')}, 0);
       })
       .then(await(defs.ConnectionClose))
       .then(function(close) {
@@ -175,7 +176,7 @@ test("unopened channel",  connectionTest(
         // there's actually nothing that would plausibly be sent to a
         // just opened connection, so this is violating more than one
         // rule. Nonetheless.
-        send(defs.ChannelOpenOk, {channelId: new Buffer('')}, 3);
+        send(defs.ChannelOpenOk, {channelId: Buffer.from('')}, 3);
       })
       .then(await(defs.ConnectionClose))
       .then(function(close) {

package/test/data.js

@@ -3,7 +3,7 @@
 'use strict';
 
 var C = require('claire');
-
+var Buffer = require('safe-buffer').Buffer;
 var forAll = C.forAll;
 var arb = C.data;
 var transform = C.transform;
@@ -18,7 +18,7 @@ var Undefined = C.Undefined;
 
 // Stub these out so we can use outside tests
 // if (!suite) var suite = function() {}
-// if (!test) var test = function() {} 
+// if (!test) var test = function() {}
 
 // These aren't exported in claire/index. so I could have to reproduce
 // them I guess.
@@ -36,7 +36,7 @@ function rangeInt(name, a, b) {
 }
 
 function toFloat32(i) {
-  var b = new Buffer(4);
+  var b = Buffer.alloc(4);
   b.writeFloatBE(i, 0);
   return b.readFloatBE(0);
 }
@@ -53,6 +53,12 @@ function floatChooser(maxExp) {
   }
 }
 
+function explicitType(t, underlying) {
+    return label(t, transform(function(n) {
+        return {'!': t, value: n};
+    }, underlying));
+}
+
 // FIXME null, byte array, others?
 
 var Octet = rangeInt('octet', 0, 255);
@@ -63,7 +69,7 @@ var ShortStr = label('shortstr',
 
 var LongStr = label('longstr',
                     transform(
-                      function(bytes) { return new Buffer(bytes); },
+                      function(bytes) { return Buffer.from(bytes); },
                       repeat(Octet)));
 
 var UShort = rangeInt('short-uint', 0, 0xffff);
@@ -85,12 +91,27 @@ var Decimal = label('decimal', transform(
     return {'!': 'decimal', value: {places: args[1], digits: args[0]}};
   }, sequence(arb.UInt, Octet)));
 
+// Signed 8 bit int
+var Byte = rangeInt('byte', -128, 127);
+
+// Explicitly typed values
+var ExByte = explicitType('byte', Byte);
+var ExInt8 = explicitType('int8', Byte);
+var ExShort = explicitType('short', Short);
+var ExInt16 = explicitType('int16', Short);
+var ExInt = explicitType('int', Long);
+var ExInt32 = explicitType('int32', Long);
+var ExLong = explicitType('long', LongLong);
+var ExInt64 = explicitType('int64', LongLong);
+
 var FieldArray = label('field-array', recursive(function() {
   return arb.Array(
     arb.Null,
-    LongStr, ShortStr, Octet,
-    UShort, ULong, ULongLong,
-    Short, Long, LongLong,
+    LongStr, ShortStr,
+    Octet, UShort, ULong, ULongLong,
+    Byte, Short, Long, LongLong,
+    ExByte, ExInt8, ExShort, ExInt16,
+    ExInt, ExInt32, ExLong, ExInt64,
     Bit, Float, Double, FieldTable, FieldArray)
 }));
 
@@ -100,7 +121,9 @@ var FieldTable = label('table', recursive(function() {
                  arb.Null,
                  LongStr, ShortStr, Octet,
                  UShort, ULong, ULongLong,
-                 Short, Long, LongLong,
+                 Byte, Short, Long, LongLong,
+                 ExByte, ExInt8, ExShort, ExInt16,
+                 ExInt, ExInt32, ExLong, ExInt64,
                  Bit, Float, Double, FieldArray, FieldTable))
 }));
 
@@ -184,7 +207,7 @@ function method(info) {
 function properties(info) {
   var types = info.args.map(argtype);
   types.unshift(ULongLong); // size
-  var domain = sequence.apply(null, types); 
+  var domain = sequence.apply(null, types);
   var names = info.args.map(name);
   return label(info.name, transform(function(fieldVals) {
     return {id: info.id,

package/test/frame.js

@@ -3,7 +3,7 @@
 var assert = require('assert');
 var succeed = require('./util').succeed;
 var fail = require('./util').fail;
-
+var Buffer = require('safe-buffer').Buffer;
 var connection = require('../lib/connection');
 var Frames = connection.Connection;
 var HEARTBEAT = require('../lib/frame').HEARTBEAT;
@@ -21,7 +21,7 @@ function inputs() {
   return new PassThrough({objectMode: true});
 }
 
-var HB = new Buffer([defs.constants.FRAME_HEARTBEAT,
+var HB = Buffer.from([defs.constants.FRAME_HEARTBEAT,
                      0, 0, // channel 0
                      0, 0, 0, 0, // zero size
                      defs.constants.FRAME_END]);
@@ -51,7 +51,7 @@ suite("Explicit parsing", function() {
       var input = inputs();
       var frames = new Frames(input);
       frames.frameMax = 5; //for the max frame test
-      input.write(new Buffer(bytes));
+      input.write(Buffer.from(bytes));
       frames.step(function(err, frame) {
         if (err != null) done();
         else done(new Error('Was a bogus frame!'));
@@ -166,7 +166,7 @@ var Content = transform(function(args) {
   return {
     method: args[0].fields,
     header: args[1].fields,
-    body: new Buffer(args[2])
+    body: Buffer.from(args[2])
   }
 }, sequence(amqp.methods['BasicDeliver'],
             amqp.properties['BasicProperties'], Body));

package/test/util.js

@@ -1,11 +1,11 @@
 'use strict';
 
+var Promise = require('bluebird');
 var crypto = require('crypto');
 var Connection = require('../lib/connection').Connection;
 var PassThrough =
   require('stream').PassThrough ||
   require('readable-stream/passthrough');
-var defer = require('when').defer;
 var defs = require('../lib/defs');
 var assert = require('assert');
 
@@ -78,24 +78,24 @@ function runServer(socket, run) {
 
   function await(method) {
     return function() {
-      var d = defer();
-      if (method) {
-        frames.step(function(e, f) {
-          if (e !== null) return d.reject(e);
-          if (f.id === method)
-            d.resolve(f);
-          else
-            d.reject(new Error("Expected method: " + method +
-                               ", got " + f.id));
-        });
-      }
-      else {
-        frames.step(function(e, f) {
-          if (e !== null) return d.reject(e);
-          else d.resolve(f);
-        });
-      }
-      return d.promise;
+      return new Promise(function(resolve, reject) {
+        if (method) {
+          frames.step(function(e, f) {
+            if (e !== null) return reject(e);
+            if (f.id === method)
+              resolve(f);
+            else
+              reject(new Error("Expected method: " + method +
+                                 ", got " + f.id));
+          });
+        }
+        else {
+          frames.step(function(e, f) {
+            if (e !== null) return reject(e);
+            else resolve(f);
+          });
+        }
+      });
     };
   }
   run(send, await);
@@ -107,6 +107,19 @@ function succeed(done) {
   return function() { done(); }
 }
 
+// Produce a callback that will complete the test successfully
+// only if the value is an object, it has the specified
+// attribute, and its value is equals to the expected value
+function succeedIfAttributeEquals(attribute, value, done) {
+  return function(object) {
+    if (object && !(object instanceof Error) && value === object[attribute]) {
+      return done();
+    }
+
+    done(new Error(attribute + " is not equal to " + value));
+  };
+}
+
 // Produce a callback that will fail the test, given either an error
 // (to be used as a failure continuation) or any other value (to be
 // used as a success continuation when failure is expected)
@@ -196,6 +209,7 @@ module.exports = {
   socketPair: socketPair,
   runServer: runServer,
   succeed: succeed,
+  succeedIfAttributeEquals: succeedIfAttributeEquals,
   fail: fail,
   latch: latch,
   completes: completes,

package/.npmignore

@@ -1,9 +0,0 @@
-*~
-scratch
-# do not ignore lib/defs.js, we need that
-# node_modules is ignored anyway
-.travis.yml
-bin/amqp-rabbitmq-0.9.1.json
-bin/generate-defs.js
-etc/
-coverage/