lint fixes

Author: nfriedly

speech-to-text/file-player.js

@@ -2,31 +2,6 @@
 
 var getContentTypeFromHeader = require('./content-type');
 
-
-
-/**
- * Reads the first few bytes of a binary file and resolves to the content-type if recognized & supported
- * @param {File|Blob} file
- * @returns {Promise}
- */
-function getContentTypeFromFile(file) {
-  return new Promise(function(resolve, reject) {
-    var blobToText = new Blob([file]).slice(0, 4);
-    var r = new FileReader();
-    r.readAsText(blobToText);
-    r.onload = function() {
-      var ct = getContentTypeFromHeader(r.result);
-      if (ct) {
-        resolve(ct);
-      } else {
-        var err = new Error('Unable to determine content type from file header; only wav, flac, and ogg/opus are supported.');
-        err.name = FilePlayer.ERROR_UNSUPPORTED_FORMAT;
-        reject(err);
-      }
-    };
-  });
-}
-
 /**
  * Plays audio from File/Blob instances
  * @param {File|Blob} file
@@ -57,6 +32,29 @@ function FilePlayer(file, contentType) {
 
 FilePlayer.ERROR_UNSUPPORTED_FORMAT = 'UNSUPPORTED_FORMAT';
 
+/**
+ * Reads the first few bytes of a binary file and resolves to the content-type if recognized & supported
+ * @param {File|Blob} file
+ * @returns {Promise}
+ */
+function getContentTypeFromFile(file) {
+  return new Promise(function(resolve, reject) {
+    var blobToText = new Blob([file]).slice(0, 4);
+    var r = new FileReader();
+    r.readAsText(blobToText);
+    r.onload = function() {
+      var ct = getContentTypeFromHeader(r.result);
+      if (ct) {
+        resolve(ct);
+      } else {
+        var err = new Error('Unable to determine content type from file header; only wav, flac, and ogg/opus are supported.');
+        err.name = FilePlayer.ERROR_UNSUPPORTED_FORMAT;
+        reject(err);
+      }
+    };
+  });
+}
+
 /**
  * Determines the file's content-type and then resolves to a FilePlayer instance
  * @param {File|Blob} file

speech-to-text/no-timestamps.js

@@ -1,3 +1,5 @@
+'use strict';
+
 /**
  * Returns true if the result is missing it's timestamps
  * @param {Object} data

speech-to-text/timing-stream.js

@@ -245,7 +245,8 @@ TimingStream.prototype.handleResult = function handleResult(data) {
   if (noTimestamps(data)) {
     var err = new Error('TimingStream requires timestamps');
     err.name = noTimestamps.ERROR_NO_TIMESTAMPS;
-    return this.emit('error', err);
+    this.emit('error', err);
+    return;
   }
 
   // http://www.ibm.com/watson/developercloud/speech-to-text/api/v1/#SpeechRecognitionEvent

test/format-stream-spec.js

@@ -24,18 +24,18 @@ describe('FormatStream', function() {
     stream.setEncoding('utf8');
     var source = {results: [
       {alternatives:
-        [{
-          confidence: 0.881,
-          transcript: 'foo bar ',
-          final: true}],
-      result_index: 0}]};
+      [{
+        confidence: 0.881,
+        transcript: 'foo bar ',
+        final: true}],
+        result_index: 0}]};
     var expected = {results: [
       {alternatives:
       [{
         confidence: 0.881,
         transcript: 'Foo bar. ',
         final: true}],
-      result_index: 0}]};
+        result_index: 0}]};
     stream.on('data', function(actual) {
       assert.equal(actual, expected);
       done();

test/result-stream-spec.js

@@ -7,7 +7,7 @@ var ResultStream = require('../speech-to-text/result-stream.js');
 describe('ResultStream', function() {
 
   it('should extract results', function(done) {
-    var stream = new ResultExtractor();
+    var stream = new ResultStream();
     var source = {results: [{alternatives: [{transcript: 'foo'}]}], result_index: 1};
     var expected = {alternatives: [{transcript: 'foo'}], index: 1};
     stream.on('data', function(actual) {
@@ -19,7 +19,7 @@ describe('ResultStream', function() {
   });
 
   it('should pass through speaker_labels messages', function(done) {
-    var stream = new ResultExtractor();
+    var stream = new ResultStream();
     var source = {speaker_labels: [{
       from: 28.92,
       to: 29.17,

test/timing-stream-spec.js

@@ -231,6 +231,8 @@ describe('TimingStream', function() {
         }
       });
 
+      assert.equal(speakerLabelsMessages, 3);
+
       done();
     });
   });
@@ -277,16 +279,16 @@ describe('TimingStream', function() {
     var noTimestamps = require('../speech-to-text/no-timestamps');
     assert(noTimestamps.ERROR_NO_TIMESTAMPS, 'noTimestamps.ERROR_NO_TIMESTAMPS should be defined');
     var stream = new TimingStream({objectMode: true});
-    var data = clone(require('./resources/results.json'));
-    delete data.results[0].alternatives[0].timestamps;
+    var message = clone(require('./resources/results.json'));
+    delete message.results[0].alternatives[0].timestamps;
     stream.on('data', function(data) {
-      assert.fail(data, undefined, "data emitted")
+      assert.fail(data, null, 'data emitted');
     });
     stream.on('error', function(err) {
       assert.equal(err.name, noTimestamps.ERROR_NO_TIMESTAMPS);
       done();
     });
-    stream.end(data);
+    stream.end(message);
   });
 
 });