Added fix for test failuers

vijay-qlogic · vijay-qlogic · commit f9f67ca8ab08 · 2018-11-07T18:04:27.000+05:30
diff --git a/samples/quickstart.js b/samples/quickstart.js
@@ -26,7 +26,10 @@ const client = new vision.ImageAnnotatorClient();
 async function main() {
   const [result] = await client.labelDetection('./resources/wakeupcat.jpg');
   const labels = result.labelAnnotations;
+  console.log('Labels:');
   labels.forEach(label => console.log(label.description));
 }
-main().catch(console.error);
+main().catch(err => {
+  console.error('ERROR:', err);
+});
 // [END vision_quickstart]
diff --git a/samples/system-test/quickstart.test.js b/samples/system-test/quickstart.test.js
@@ -23,16 +23,12 @@ const cmd = `node quickstart.js`;
 const cwd = path.join(__dirname, `..`);
 
 describe(`quickstart`, () => {
-  before(async () => {
-    tools.stubConsole;
-  });
-
-  after(async () => {
-    tools.restoreConsole;
-  });
+  before(tools.stubConsole);
+  after(tools.restoreConsole);
 
   it(`should detect labels in a remote file`, async () => {
     const output = await tools.runAsync(`${cmd}`, cwd);
+    assert.ok(output.includes(`Labels:`));
     assert.ok(output.includes(`cat`));
   });
 });
diff --git a/samples/system-test/textDetection.test.js b/samples/system-test/textDetection.test.js
@@ -19,12 +19,12 @@ const path = require(`path`);
 const assert = require('assert');
 const tools = require(`@google-cloud/nodejs-repo-tools`);
 
-describe(`text detection`, () => {
+describe(`Text Detection`, () => {
   before(async () => {
     tools.checkCredentials;
   });
 
-  it(`should detect texts`, async () => {
+  it(`should detect texts`, done => {
     const redis = require('redis');
     const client = redis.createClient();
 
@@ -34,23 +34,29 @@ describe(`text detection`, () => {
           console.error(
             'Redis is unavailable. Skipping vision textDetection test.'
           );
-          assert.ifError(err);
+          client.end(true);
+          done();
         } else {
-          assert.ifError(err);
+          client.end(true);
+          done(err);
         }
       })
       .on('ready', async () => {
         const inputDir = path.join(__dirname, `../resources`);
         const textDetectionSample = require(`../textDetection`);
 
-        const textResponse = await textDetectionSample.main(inputDir);
+        const textResponse = await textDetectionSample
+          .main(inputDir)
+          .catch(err => {
+            console.log(`Error at 46: ${err}`);
+          });
         assert.ok(Object.keys(textResponse).length > 0);
 
-        const hits = await textDetectionSample.lookup([
-          'the',
-          'sunbeams',
-          'in',
-        ]);
+        const hits = await textDetectionSample
+          .lookup(['the', 'sunbeams', 'in'])
+          .catch(err => {
+            console.log(`Error at 51: ${err}`);
+          });
         assert.ok(hits.length > 0);
         assert.ok(hits.length > 0);
         assert.ok(hits[0].length > 0);
