added tests - now we have 100% coverage

dgarlitt · Jun 1, 2017 · bd5f73d · bd5f73d
1 parent dbfc1e8
commit bd5f73d
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 2 deletions.
diff --git a/lib/data/store.js b/lib/data/store.js
@@ -39,7 +39,7 @@ DataStore.prototype.saveResultToSuite = function(suite, browser, result) {
 
   if(result.log && result.log[0] !== null){
     if (result.log.length > 1) {
-      for (var i = 0; i < result.log.length; i++) {
+      for (var i = 1; i < result.log.length; i++) {
         result.log[0] = result.log[0] + '\n' + result.log[i];
       }
     }

diff --git a/test/data.store.test.js b/test/data.store.test.js
@@ -165,6 +165,15 @@ describe('data/store.js - test suite', function() {
 
       assert.deepEqual(expected, brwsr.errors);
     });
+
+    it('should create brwsr.errors from result.log when there are more errors', function() {
+      result.log = ['failure01\nfailure02','failure03\nfailure04'];
+      var expected = ['failure01', 'failure02', 'failure03', 'failure04'];
+
+      sut.saveResultToSuite(suite, browser, result);
+
+      assert.deepEqual(expected, brwsr.errors);
+    });
   });
 
   /**

diff --git a/test/printers.test.js b/test/printers.test.js
@@ -28,7 +28,8 @@ describe('printers.js test suite', function() {
       },
       'blackBright':  sinon.stub(),
       'white':        sinon.stub(),
-      'yellow':       sinon.stub()
+      'yellow':       sinon.stub(),
+      'xterm':        sinon.stub()
     };
 
     colorConsoleLogsFake = sinon.stub();
@@ -47,6 +48,14 @@ describe('printers.js test suite', function() {
     done();
   });
 
+
+  describe('setColorOptions()', function() {
+    it('should set color of console logs', function() {
+      sut.setColorOptions({"colorConsoleLogs": 99});
+      ok(sut.__get__('clc').xterm.callCount === 1);
+    });
+  });
+
   /**
    * printBrowserErrors() tests
    */