Merge pull request #11 from achappuis/master

[fix] fix issues when no test or no assert names are provided

Author: gabrielcsapo

CHANGELOG.md

@@ -1,5 +1,7 @@
 # Unreleased
 
+- fixes issue when no assert name is provided
+- fixes issue when no test name is provided with TAP comment
 - Change behavior when no output file is given to tap-html to restore what is described in README.md
 
 # 1.0.1 (04/20/2019)

index.js

@@ -14,19 +14,23 @@ module.exports = function tapHTML (callback) {
 
   const startTime = Date.now();
 
+  function pushTest(name) {
+    data.push({
+      type: 'test',
+      name: name,
+      start: Date.now(),
+      assertions: []
+    });
+
+    // get the current index of the plan
+    // so that we can use this to push the current assertions to it
+    currentPlan += 1;
+    currentAssertion = -1;
+  }
+
   tap.on('comment', (res) => {
     if (!plan) {
-      data.push({
-        type: 'test',
-        name: res,
-        start: Date.now(),
-        assertions: []
-      });
-
-      // get the current index of the plan
-      // so that we can use this to push the current assertions to it
-      currentPlan += 1;
-      currentAssertion = -1;
+      pushTest(res);
     }
   });
 
@@ -43,6 +47,16 @@ module.exports = function tapHTML (callback) {
   });
 
   tap.on('assert', (res) => {
+    // If no plan is registered yet, create a default plan.
+    if(currentPlan == -1) {
+      pushTest('default');
+    }
+
+    // TAP does not require a name. If no name is registered, set a default name.
+    if (!res.name) {
+      res.name = 'test #' + res.id;
+    }
+
     data[currentPlan].assertions.push({
       type: 'assert',
       number: res.id,