Merge pull request #205 from zanchin/container-inspect-opts

Allow an optional opts argument to Container.inspect

lib/container.js

@@ -36,23 +36,26 @@ var Container = function(modem, id) {
 
 /**
  * Inspect
+ * @param  {Options}  opts     Options (optional)
  * @param  {Function} callback Callback, if supplied will query Docker.
  * @return {Object}            ID only and only if callback isn't supplied.
  */
-Container.prototype.inspect = function(callback) {
-  if (typeof callback === 'function') {
-    var optsf = {
-      path: '/containers/' + this.id + '/json',
-      method: 'GET',
-      statusCodes: {
-        200: true,
-        404: 'no such container',
-        500: 'server error'
-      }
-    };
+Container.prototype.inspect = function(opts, callback) {
+  var args = util.processArgs(opts, callback);
+  var optsf = {
+    path: '/containers/' + this.id + '/json?',
+    method: 'GET',
+    options: args.opts,
+    statusCodes: {
+      200: true,
+      404: 'no such container',
+      500: 'server error'
+    }
+  };
 
+  if (typeof args.callback === 'function') {
     this.modem.dial(optsf, function(err, data) {
-      callback(err, data);
+      args.callback(err, data);
     });
   } else {
     return JSON.stringify({

test/container.js

@@ -37,6 +37,18 @@ describe("#container", function() {
 
       container.inspect(handler);
     });
+
+    it("should inspect a container with opts", function(done) {
+      var container = docker.getContainer(testContainer);
+
+      function handler(err, data) {
+        expect(err).to.be.null;
+        expect(data).to.be.ok;
+        done();
+      }
+
+      container.inspect({}, handler);
+    });
   });
 
   describe("#archive", function() {