fix: undo workaround for color output on windows; test passes now

Author: fluffynuts

test/index.spec.js

@@ -59,7 +59,7 @@ var windowsIcons = {
 var formatError = function (a, b) {
   return a + b;
 };
-function noop() {}
+function noop() { }
 var baseReporterDecorator = function (context) {
   context.renderBrowser = sinon.spy();
   context.writeCommonMsg = sinon.spy();
@@ -130,28 +130,15 @@ describe('SpecReporter', function () {
       });
 
       it('should set the BaseReporter function\'s colors', function () {
-        //NOTE: this is temporary as the colors module was updated
-        if (os.platform() === 'win32') {
-            newSpecReporter.SPEC_FAILURE.should.equal('%s %s FAILED\n');
-            newSpecReporter.SPEC_SLOW.should.equal('%s SLOW %s: %s\n');
-            newSpecReporter.ERROR.should.equal('%s ERROR\n');
-            newSpecReporter.FINISHED_ERROR.should.equal(' ERROR');
-            newSpecReporter.FINISHED_SUCCESS.should.equal(' SUCCESS');
-            newSpecReporter.FINISHED_DISCONNECTED.should.equal(' DISCONNECTED');
-            newSpecReporter.X_FAILED.should.equal(' (%d FAILED)');
-            newSpecReporter.TOTAL_SUCCESS.should.equal('TOTAL: %d SUCCESS\n');
-            newSpecReporter.TOTAL_FAILED.should.equal('TOTAL: %d FAILED, %d SUCCESS\n');
-        } else {
-            newSpecReporter.SPEC_FAILURE.should.equal(ansiColors.red + '%s %s FAILED' + ansiColors.reset + '\n');
-            newSpecReporter.SPEC_SLOW.should.equal(ansiColors.yellow + '%s SLOW %s: %s' + ansiColors.reset + '\n');
-            newSpecReporter.ERROR.should.equal(ansiColors.red + '%s ERROR' + ansiColors.reset + '\n');
-            newSpecReporter.FINISHED_ERROR.should.equal(ansiColors.red + ' ERROR' + ansiColors.reset);
-            newSpecReporter.FINISHED_SUCCESS.should.equal(ansiColors.green + ' SUCCESS' + ansiColors.reset);
-            newSpecReporter.FINISHED_DISCONNECTED.should.equal(ansiColors.red + ' DISCONNECTED' + ansiColors.reset);
-            newSpecReporter.X_FAILED.should.equal(ansiColors.red + ' (%d FAILED)' + ansiColors.reset);
-            newSpecReporter.TOTAL_SUCCESS.should.equal(ansiColors.green + 'TOTAL: %d SUCCESS' + ansiColors.reset + '\n');
-            newSpecReporter.TOTAL_FAILED.should.equal(ansiColors.red + 'TOTAL: %d FAILED, %d SUCCESS' + ansiColors.reset + '\n');
-        }
+        newSpecReporter.SPEC_FAILURE.should.equal(ansiColors.red + '%s %s FAILED' + ansiColors.reset + '\n');
+        newSpecReporter.SPEC_SLOW.should.equal(ansiColors.yellow + '%s SLOW %s: %s' + ansiColors.reset + '\n');
+        newSpecReporter.ERROR.should.equal(ansiColors.red + '%s ERROR' + ansiColors.reset + '\n');
+        newSpecReporter.FINISHED_ERROR.should.equal(ansiColors.red + ' ERROR' + ansiColors.reset);
+        newSpecReporter.FINISHED_SUCCESS.should.equal(ansiColors.green + ' SUCCESS' + ansiColors.reset);
+        newSpecReporter.FINISHED_DISCONNECTED.should.equal(ansiColors.red + ' DISCONNECTED' + ansiColors.reset);
+        newSpecReporter.X_FAILED.should.equal(ansiColors.red + ' (%d FAILED)' + ansiColors.reset);
+        newSpecReporter.TOTAL_SUCCESS.should.equal(ansiColors.green + 'TOTAL: %d SUCCESS' + ansiColors.reset + '\n');
+        newSpecReporter.TOTAL_FAILED.should.equal(ansiColors.red + 'TOTAL: %d FAILED, %d SUCCESS' + ansiColors.reset + '\n');
       });
     });
 
@@ -376,7 +363,7 @@ describe('SpecReporter', function () {
         });
         it('should throw an error', function () {
           expect(function () {
-            newSpecReporter.onSpecFailure([],{
+            newSpecReporter.onSpecFailure([], {
               suite: [],
               log: []
             })