Merge pull request #40 from dhershman1/development

Development v5.0.1

Author: dhershman1

README.md

@@ -1,6 +1,7 @@
 [![CircleCI](https://circleci.com/gh/dhershman1/tap-junit.svg?style=svg)](https://circleci.com/gh/dhershman1/tap-junit)
 [![npm](https://img.shields.io/npm/v/tap-junit.svg?style=flat-square)](https://www.npmjs.com/package/tap-junit)
 [![Downloads](https://img.shields.io/npm/dm/tap-junit.svg?style=flat-square)](https://www.npmjs.com/package/tap-junit)
+[![Language grade: JavaScript](https://img.shields.io/lgtm/grade/javascript/g/dhershman1/tap-junit.svg?style=flat-square&logo=lgtm&logoWidth=18)](https://lgtm.com/projects/g/dhershman1/tap-junit/context:javascript)
 
 # tap-junit
 

changelog.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## v5.0.1
+
+### Added
+
+- Language Grade
+
+### Fixed
+
+- Error/Failure messaging returning `undefined` becuase it was preferring todo instead of message, which might not exist. [#39](https://github.com/dhershman1/tap-junit/issues/39)
+
+
 ## v5.0.0
 
 Please report any issues or serliazation that does not follow the below spec for output! Thanks!

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "tap-junit",
-  "version": "5.0.0",
+  "version": "5.0.1",
   "description": "Silly small, silly easy junit output formatter for tap.",
   "main": "src/index.js",
   "bin": {
@@ -21,6 +21,7 @@
     "test:tapx": "cross-env bin/tap-junit -i tests/tapX.tap -o output/test -n tapx",
     "test:bat": "cross-env bin/tap-junit -i tests/bats-1.0.0.txt -o output/test -n bat -p",
     "test:basic": "cross-env bin/tap-junit -i tests/basic.tap -o output/test -n basic -p",
+    "test:basic2": "cross-env bin/tap-junit -i tests/basic2.tap -o output/test -n basic2 -p",
     "all": "npm-run-all clear test test:* ci:*",
     "ci": "npm-run-all clear test ci:*",
     "test:fail": "tape tests/fail.js | cross-env bin/tap-junit --output output/test -n failure -p",

package.json

@@ -43,12 +43,11 @@ function buildFailureParams (fail) {
   }
 
   if (fail.diag) {
-    failObj['@message'] = fail.diag.message || fail.todo
-      ? `${fail.todo}`
-      : `
-    ---
-    ${formatDiag(fail.diag)}
-    ...`
+    if (fail.todo) {
+      failObj['@message'] = fail.todo
+    } else {
+      failObj['@message'] = fail.diag.message || formatDiag(fail.diag)
+    }
     failObj['@type'] = fail.diag.severity || 'fail'
   }
 

src/serialize.js

@@ -0,0 +1,12 @@
+TAP version 13
+1..4
+not ok 1 - Manual/about.md
+  ---
+  message: /tmp/lint/Manual/about.md 9 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context  "# About this manual"]
+  ...
+ok 2 - Manual/welcome.md
+ok 3 - Media/README.md
+not ok 4 - README.md
+  ---
+  message: /tmp/lint/README.md 4 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context  "# Getting Started"]\n/tmp/lint/README.md 11 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context  "# Build and Test"]\n/tmp/lint/README.md 17 MD025/single-title/single-h1 Multiple top-level headings in the same document [Context  "# Contribute"]
+  ...