Merge pull request #490 from mamoonraja/master

Update Discovery
[semver minor]

Author: nfriedly

discovery/v1.js

@@ -32,7 +32,7 @@ function DiscoveryV1(options) {
 
   // Check if 'version_date' was provided
   if (typeof this._options.version_date === 'undefined') {
-    throw new Error('Argument error: version_date was not specified, use DiscoveryV1.VERSION_DATE_2017_04_27');
+    throw new Error('Argument error: version_date was not specified, use DiscoveryV1.VERSION_DATE_2017_08_01');
   }
   this._options.qs.version = options.version_date;
 }
@@ -52,6 +52,11 @@ DiscoveryV1.VERSION_DATE_2016_12_15 = '2016-12-15';
  * @type {string}
  */
 DiscoveryV1.VERSION_DATE_2017_04_27 = '2017-04-27';
+/**
+ * Release migrating from Watson Discovery News Original to Watson Discovery News
+ * @type {string}
+ */
+DiscoveryV1.VERSION_DATE_2017_08_01 = '2017-08-01';
 
 /**
  * Return the list of environments
@@ -285,7 +290,7 @@ DiscoveryV1.prototype.getConfiguration = function(params, callback) {
  *
  * @param {Object} params
  * @param {String} params.environment_id
-  */
+ */
 DiscoveryV1.prototype.getCollections = function(params, callback) {
   params = params || {};
 

speech-to-text/recognize_stream.js

@@ -217,7 +217,7 @@ RecognizeStream.prototype.initialize = function() {
   this.initialized = true;
 };
 
-RecognizeStream.prototype._read = function(/* size*/) {
+RecognizeStream.prototype._read = function() /* size*/ {
   // there's no easy way to control reads from the underlying library
   // so, the best we can do here is a no-op
 };

test/unit/test.discovery.v1.js

@@ -13,6 +13,14 @@ describe('discovery-v1', function() {
 
   // Test params
   const service = {
+    username: 'batman',
+    password: 'bruce-wayne',
+    url: 'http://ibm.com:80',
+    version: 'v1',
+    version_date: DiscoveryV1.VERSION_DATE_2017_08_01
+  };
+
+  const service_v2016_04_27 = {
     username: 'batman',
     password: 'bruce-wayne',
     url: 'http://ibm.com:80',
@@ -57,7 +65,7 @@ describe('discovery-v1', function() {
   });
 
   describe('discovery versions', function() {
-    [service, service_v2016_12_15].forEach(service => {
+    [service, service_v2016_04_27, service_v2016_12_15].forEach(service => {
       beforeEach(function() {
         nock.disableNetConnect();
         // grr! these should be in the individual tests where they are needed!