Add test case for detectEncoding_ for better coverage.

jmuk · jmuk · commit d602ee1e0cb2 · 2016-09-23T16:10:23.000-07:00
diff --git a/packages/speech/src/index.js b/packages/speech/src/index.js
@@ -137,7 +137,6 @@ Speech.prototype.endpointerTypes = {
  */
 Speech.detectEncoding_ = function(filename) {
   if (!is.string(filename)) {
-    // istanbul ignore next
     return;
   }
 
diff --git a/packages/speech/test/index.js b/packages/speech/test/index.js
@@ -158,6 +158,10 @@ describe('Speech', function() {
         Speech.detectEncoding_('blah.mp3');
       }, /Encoding could not be determined for file: blah\.mp3/);
     });
+
+    it('should return nothing if the argument is not a string', function() {
+      assert.equal(Speech.detectEncoding_({}), undefined);
+    });
   });
 
   describe('findFile_', function() {

Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,6 @@ Speech.prototype.endpointerTypes = {`
`137`	`137`	`*/`
`138`	`138`	`Speech.detectEncoding_ = function(filename) {`
`139`	`139`	`if (!is.string(filename)) {`
`140`		`- // istanbul ignore next`
`141`	`140`	`return;`
`142`	`141`	`}`
`143`	`142`