Add unit tests and handle NaN

Author: Karan Nagpal

bin/helpers/sync/syncSpecsLogs.js

@@ -16,7 +16,7 @@ let specSummary = {
 let noWrap = false;
 let terminalWidth = (process.stdout.columns) * 0.9;
 let lineSeparator = Constants.syncCLI.DEFAULT_LINE_SEP;
-if (!utils.isUndefined(terminalWidth)) lineSeparator = "\n" + "-".repeat(terminalWidth);
+if (!isNaN(terminalWidth)) lineSeparator = "\n" + "-".repeat(terminalWidth);
 
 let  getOptions = (auth, build_id) => {
   return {
@@ -33,13 +33,13 @@ let  getOptions = (auth, build_id) => {
   };
 }
 
-let getTableConfig = () => {
-  let centerWidth = Math.ceil(terminalWidth * 0.01),
-      leftWidth = Math.floor(terminalWidth * 0.75),
-      colWidth = Math.floor(terminalWidth * 0.2);
+let getTableConfig = (termWidth) => {
+  let centerWidth = Math.ceil(termWidth * 0.01),
+      leftWidth = Math.floor(termWidth * 0.75),
+      colWidth = Math.floor(termWidth * 0.2);
 
   // Do not autosize on terminal's width if no-wrap provided
-  if (noWrap || utils.isUndefined(terminalWidth)) {
+  if (noWrap || isNaN(termWidth)) {
     centerWidth = 1;
     leftWidth = 100;
     colWidth = 30;
@@ -93,7 +93,7 @@ let printSpecsStatus = (bsConfig, buildDetails) => {
   setNoWrapParams();
   return new Promise((resolve, reject) => {
     options = getOptions(bsConfig.auth, buildDetails.build_id)
-    tableConfig = getTableConfig();
+    tableConfig = getTableConfig(terminalWidth);
     stream = tableStream(tableConfig);
 
     async.whilst(

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

@@ -94,7 +94,7 @@ describe("syncSpecsLogs", () => {
       var getBorderConfigStub = sandbox.stub();
       syncSpecsLogs.__set__('getBorderConfig', getBorderConfigStub);
 
-      let options = getTableConfig();
+      let options = getTableConfig((process.stdout.columns) * 0.9);
       expect(options.columnDefault.width).to.equal(Math.floor(((process.stdout.columns) * 0.9) * 0.2));
       expect(options.columns[1].alignment).to.equal('center');
       expect(options.columns[2].alignment).to.equal('left');
@@ -103,6 +103,20 @@ describe("syncSpecsLogs", () => {
       expect(options.columnCount).to.equal(3);
       expect(getBorderConfigStub.calledOnce).to.be.true;
     });
+
+    it('should return proper table config option for spec table if process.stdout.columns is not defined', () => {
+      var getBorderConfigStub = sandbox.stub();
+      syncSpecsLogs.__set__('getBorderConfig', getBorderConfigStub);
+
+      let options = getTableConfig(NaN);
+      expect(options.columnDefault.width).to.equal(30);
+      expect(options.columns[1].alignment).to.equal('center');
+      expect(options.columns[2].alignment).to.equal('left');
+      expect(options.columns[1].width).to.equal(1);
+      expect(options.columns[2].width).to.equal(100);
+      expect(options.columnCount).to.equal(3);
+      expect(getBorderConfigStub.calledOnce).to.be.true;
+    });
   });
 
   context("getBorderConfig", () => {