added specs for the changes

Author: roshan04

bin/helpers/sync/failedSpecsDetails.js

@@ -83,8 +83,9 @@ let getSpecStatus = (specStatus) => {
     case 'passed_with_pending':
     case 'skipped':
     case 'passed_with_skipped': return chalk.blueBright(specStatus);
-    default: chalk.yellow(specStatus);
+    default: return chalk.yellow(specStatus);
   }
 }
 
 exports.failedSpecsDetails = failedSpecsDetails;
+exports.getSpecStatus = getSpecStatus;

test/unit/bin/helpers/packageInstaller.js

@@ -256,7 +256,7 @@ describe("packageInstaller", () => {
       })
       .catch((error) => {
         spawnStub.restore();
-        chai.assert.equal(error, "Packages were not installed successfully. Error code 1")
+        chai.assert.equal(error, "Packages were not installed successfully. Error code ${code}")
       });
     });
   });

test/unit/bin/helpers/sync/failedSpecDetails.js

@@ -1,7 +1,8 @@
 'use strict';
 const chai = require("chai"),
   expect = chai.expect,
-  chaiAsPromised = require("chai-as-promised");
+  chaiAsPromised = require("chai-as-promised"),
+  chalk = require('chalk');
 
 const sinon = require("sinon");
 chai.use(chaiAsPromised);
@@ -74,3 +75,42 @@ describe("failedSpecsDetails", () => {
     });
   });
 });
+
+describe("#getSpecStatus", () => {
+  
+  it("returns failed in red if specStatus is failed", () => {
+    let specStatus = "failed";
+    let response = chalk.red(specStatus);
+    expect(specDetails.getSpecStatus(specStatus)).to.eq(response);
+  });
+
+  it("returns passed_with_skipped in blueBright if specStatus is passed_with_skipped", () => {
+    let specStatus = "passed_with_skipped";
+    let response = chalk.blueBright(specStatus);
+    expect(specDetails.getSpecStatus(specStatus)).to.eq(response);
+  });
+
+  it("returns pending in blueBright if specStatus is pending", () => {
+    let specStatus = "pending";
+    let response = chalk.blueBright(specStatus);
+    expect(specDetails.getSpecStatus(specStatus)).to.eq(response);
+  });
+
+  it("returns passed_with_pending in blueBright if specStatus is passed_with_pending", () => {
+    let specStatus = "passed_with_pending";
+    let response = chalk.blueBright(specStatus);
+    expect(specDetails.getSpecStatus(specStatus)).to.eq(response);
+  });
+
+  it("returns skipped in blueBright if specStatus is skipped", () => {
+    let specStatus = "skipped";
+    let response = chalk.blueBright(specStatus);
+    expect(specDetails.getSpecStatus(specStatus)).to.eq(response);
+  });
+
+  it("returns other statuses in yellow if specStatus is other than something known", () => {
+    let specStatus = "xyz";
+    let response = chalk.yellow(specStatus);
+    expect(specDetails.getSpecStatus(specStatus)).to.eq(response);
+  });
+});

test/unit/bin/helpers/sync/specSummary.js

@@ -46,7 +46,7 @@ describe("printSpecsRunSummary", () => {
       var loggerInfoSpy = sinon.spy(logger, 'info');
 
       specSummary.printSpecsRunSummary(data, machines);
-      sinon.assert.calledWith(loggerInfoSpy, 'Total tests: 4, passed: 1, failed: 2, skipped: 1');
+      sinon.assert.calledWith(loggerInfoSpy, 'Total tests: 4, passed: 1, failed: 2, skipped: 1, pending: 0');
       sinon.assert.calledWith(loggerInfoSpy, `Done in ${time / 1000} seconds using ${machines} machines\n`);
 
       loggerInfoSpy.restore();

test/unit/bin/helpers/sync/syncSpecsLogs.js

@@ -136,9 +136,9 @@ describe("syncSpecsLogs", () => {
       const stream = sandbox.stub();
       stream.write = sandbox.stub();
       syncSpecsLogs.__set__('stream', stream);
-      let combination = "Windows 10", path = "path", status = "passed";
-      writeToTable(combination, path, status);
-      sinon.assert.calledOnceWithExactly(stream.write, [combination , ":", `${path} ${status}`]);
+      let combination = "Windows 10", path = "path", status = "passed", statusMark = "passed";
+      writeToTable(combination, path, status, statusMark);
+      sinon.assert.calledOnceWithExactly(stream.write, [combination , ":", `${path} ${statusMark} [${status}]`]);
     });
   });