100% test coverage

snappy.js

@@ -32,15 +32,15 @@ exports.isValidCompressedSync = binding.isValidCompressedSync;
  * A parser can be attached. If no parser is attached, return buffer.
  */
 exports.uncompress = function (compressed, opts, callback) {
-  if (!Buffer.isBuffer(compressed)) {
-    return callback(new Error('input must be a Buffer'));
-  }
-
   if (!callback) {
     callback = opts;
     opts = {};
   }
 
+  if (!Buffer.isBuffer(compressed)) {
+    return callback(new Error('input must be a Buffer'));
+  }
+
   if (typeof (opts.asBuffer) !== 'boolean') {
     opts.asBuffer = true;
   }

test.js

@@ -32,6 +32,13 @@ test('compress() buffer', function (t) {
   });
 });
 
+test('compress() bad input', function (t) {
+  snappy.compress(123, function (err) {
+    t.equal(err && err.message, 'input must be a String or a Buffer');
+    t.end();
+  });
+})
+
 test('compressSync() buffer', function (t) {
   var buffer = snappy.compressSync(inputBuffer);
   t.ok(Buffer.isBuffer(buffer), 'should return a Buffer');
@@ -98,6 +105,13 @@ test('uncompress() on bad input', function (t) {
   });
 });
 
+test('uncompress() on not a Buffer', function (t) {
+  snappy.uncompress('beep boop OMG OMG OMG', function (err) {
+    t.equal(err.message, 'input must be a Buffer');
+    t.end();
+  })
+});
+
 test('uncompressSync() defaults to Buffer', function (t) {
   var results = snappy.uncompressSync(compressed);
   t.deepEqual(results, inputBuffer);