js-reporters - npm Package Compare versions

Comparing version 1.2.2 to 1.2.3

CHANGELOG.md

@@ -0,5 +1,13 @@
+
+1.2.3 / 2020-09-07
+==================
+
+  * Helpers: Correct spelling in `autoRegister()` error message. (P. Roebuck) [#108](https://github.com/js-reporters/js-reporters/issues/108)
+  * Reporter: Align `actual` with `expected` in TapReporter. (Robert Jackson) [#107](https://github.com/js-reporters/js-reporters/pull/107)
+  * Reporter: Revert "Fix YAML output in TAP reporter". [#110](https://github.com/js-reporters/js-reporters/issues/110)
+
 1.2.2 / 2019-05-13
 ==================
 
-  * Reporter: Fix YAML output in TAP reporter (#110)
+  * Reporter: Fix YAML output in TAP reporter. [#110](https://github.com/js-reporters/js-reporters/issues/110)
 
@@ -6,0 +14,0 @@ 1.2.1 / 2017-07-04

dist/js-reporters.js

 /**
- * JsReporters 1.2.2
+ * JsReporters 1.2.3
  * https://github.com/js-reporters
@@ -9,3 +9,3 @@ *
  *
- * Date: Mon May 13 2019
+ * Date: Thu Sep 10 2020
  */
@@ -992,3 +992,3 @@
   } else {
-    throw new Error('Failed to register js-reporters adapater. Supported ' + 'frameworks are: QUnit, Mocha, Jasmine');
+    throw new Error('Failed to register js-reporters adapter. Supported ' + 'frameworks are: QUnit, Mocha, Jasmine');
   }
@@ -1578,12 +1578,12 @@
       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';
-        console.log('  actual: ' + actualStr);
+        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);
@@ -1593,3 +1593,3 @@ }
       if (error.stack) {
-        console.log('  stack: "' + error.stack.replace(/"/g, '\\"').replace(/\n/g, '\\n') + '"');
+        console.log('  stack: ' + error.stack);
       }
@@ -1596,0 +1596,0 @@

lib/helpers.js

@@ -22,3 +22,3 @@ /* global QUnit, mocha, jasmine */
   } else {
-    throw new Error('Failed to register js-reporters adapater. Supported ' +
+    throw new Error('Failed to register js-reporters adapter. Supported ' +
       'frameworks are: QUnit, Mocha, Jasmine')
@@ -25,0 +25,0 @@ }

lib/reporters/TapReporter.js

@@ -46,12 +46,12 @@ import chalk from 'chalk'
     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'
-      console.log(`  actual: ${actualStr}`)
+      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}`)
@@ -61,3 +61,3 @@ }
     if (error.stack) {
-      console.log(`  stack: "${error.stack.replace(/"/g, '\\"').replace(/\n/g, '\\n')}"`)
+      console.log(`  stack: ${error.stack}`)
     }
@@ -64,0 +64,0 @@

package.json

 {
   "name": "js-reporters",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "common reporter interface for javascript testing frameworks",
@@ -13,3 +13,3 @@ "main": "dist/js-reporters.js",
     "build": "rollup -c",
-    "prepublish": "npm run build",
+    "prepare": "npm run build",
     "pretest": "npm run build",
@@ -16,0 +16,0 @@ "test": "standard && npm run test-unit && npm run test-integration",

RELEASE.md

 # Release Process
 
-1. Update `CHANGELOG.md` using `git changelog` from the [`git-extras`](https://github.com/tj/git-extras) package
-2. Commit changelog updates with message: `Changelog: Update for x.x.x release`
-3. Update `package.json` version and tag it using `npm version x.x.x -m "Release: vx.x.x"`
-4. Push the two new commits and tag to GitHub
-5. Run `npm publish`
-6. Publish a new [release on GitHub](https://github.com/js-reporters/js-reporters/releases) with the changelog update
+1. Update `CHANGELOG.md` using `git changelog` from the [`git-extras`](https://github.com/tj/git-extras) package. Edit the changelog file as needed, and stage the changes.
+2. Update `package.json` version, and stage the changes.
+3. Commit with message `Release X.Y.Z`, and create a signed tag `git tag -s "vX.Y.Z" -m "Release X.Y.Z"`
+4. Push the commit and tag to GitHub.
+5. Run `npm publish`.
+6. Publish a new [release on GitHub](https://github.com/js-reporters/js-reporters/releases) with a copy of the changelog.
 
 That's all!

test/unit/tap-reporter.js

@@ -71,5 +71,5 @@ /* eslint-env mocha */
       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('  ...')
@@ -79,2 +79,3 @@ })
     emitter.emit('testEnd', data.failingTest)
+
     for (var i = 0; i < expected.length; i++) {
@@ -90,3 +91,3 @@ expect(spy).to.have.been.calledWith(expected[i])
 
-    expect(spy).to.have.been.calledWith('  actual: undefined')
+    expect(spy).to.have.been.calledWith('  actual  : undefined')
   }))
@@ -99,3 +100,3 @@
 
-    expect(spy).to.have.been.calledWith('  actual: "0"')
+    expect(spy).to.have.been.calledWith('  actual  : 0')
   }))
@@ -116,3 +117,3 @@
 
-    expect(spy).to.have.been.calledWith('  expected: "0"')
+    expect(spy).to.have.been.calledWith('  expected: 0')
   }))
@@ -119,0 +120,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

386853

increased by0.03%

Worsened metrics

Number of medium supply chain risk alerts

2

increased by100%

No dependency changes