Revert "Reporter: Fix YAML output in TAP reporter (#110)"

This turns too many values into quoted JSON strings, which
makes the output difficult to read.

I am reverting this for 1.x-stable to allow other changes to be
released and adopted by consumers, such as QUnit.

I will improve on the solution in a different way in master
for js-reporters 2.0.

Ref #109.

Author: jeberger

lib/reporters/TapReporter.js

@@ -43,21 +43,21 @@ export default class TapReporter {
 
   logError (error, severity) {
     console.log('  ---')
-    console.log(`  message: "${(error.message || 'failed').replace(/"/g, '\\"')}"`)
+    console.log(`  message: "${error.message || 'failed'}"`)
     console.log(`  severity: ${severity || 'failed'}`)
 
     if (error.hasOwnProperty('actual')) {
-      var actualStr = error.actual !== undefined ? ('"' + JSON.stringify(error.actual, null, 2).replace(/"/g, '\\"').replace(/\n/g, '\\n') + '"') : 'undefined'
+      var actualStr = error.actual !== undefined ? JSON.stringify(error.actual, null, 2) : 'undefined'
       console.log(`  actual  : ${actualStr}`)
     }
 
     if (error.hasOwnProperty('expected')) {
-      var expectedStr = error.expected !== undefined ? ('"' + JSON.stringify(error.expected, null, 2).replace(/"/g, '\\"').replace(/\n/g, '\\n') + '"') : 'undefined'
+      var expectedStr = error.expected !== undefined ? JSON.stringify(error.expected, null, 2) : 'undefined'
       console.log(`  expected: ${expectedStr}`)
     }
 
     if (error.stack) {
-      console.log(`  stack: "${error.stack.replace(/"/g, '\\"').replace(/\n/g, '\\n')}"`)
+      console.log(`  stack: ${error.stack}`)
     }
 
     console.log('  ...')

test/unit/tap-reporter.js

@@ -68,13 +68,14 @@ describe('Tap reporter', function () {
 
     data.failingTest.errors.forEach(function (error) {
       expected.push('  ---')
-      expected.push('  message: "' + error.message.replace(/"/g, '\\"') + '"')
+      expected.push('  message: "' + error.message + '"')
       expected.push('  severity: failed')
-      expected.push('  stack: "' + error.stack.replace(/"/g, '\\"').replace(/\n/g, '\\n') + '"')
+      expected.push('  stack: ' + error.stack)
       expected.push('  ...')
     })
 
     emitter.emit('testEnd', data.failingTest)
+
     for (var i = 0; i < expected.length; i++) {
       expect(spy).to.have.been.calledWith(expected[i])
     }
@@ -93,7 +94,7 @@ describe('Tap reporter', function () {
 
     emitter.emit('testEnd', data.actualFalsyTest)
 
-    expect(spy).to.have.been.calledWith('  actual  : "0"')
+    expect(spy).to.have.been.calledWith('  actual  : 0')
   }))
 
   it('should output expected value for failed assertions even it was undefined', sinon.test(function () {
@@ -109,7 +110,7 @@ describe('Tap reporter', function () {
 
     emitter.emit('testEnd', data.expectedFalsyTest)
 
-    expect(spy).to.have.been.calledWith('  expected: "0"')
+    expect(spy).to.have.been.calledWith('  expected: 0')
   }))
 
   it('should output the total number of tests', sinon.test(function () {