[Core] Remove todo and skip explanation options

tape-testing · Jun 24, 2019 · c347a5d · c347a5d
1 parent b5dce9c
commit c347a5d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 64 deletions.
diff --git a/lib/results.js b/lib/results.js
@@ -144,8 +144,8 @@ function encodeResult(res, count) {
     output += (res.ok ? 'ok ' : 'not ok ') + count;
     output += res.name ? ' ' + res.name.toString().replace(/\s+/g, ' ') : '';
 
-    if (res.skip) output += ' # SKIP' + (typeof res.skip === 'string' ? ' ' + res.skip : '');
-    else if (res.todo) output += ' # TODO' + (typeof res.todo === 'string' ? ' ' + res.todo : '');
+    if (res.skip) output += ' # SKIP';
+    else if (res.todo) output += ' # TODO';
 
     output += '\n';
     if (res.ok) return output;

diff --git a/test/objectMode.js b/test/objectMode.js
@@ -17,7 +17,7 @@ tap.test('object results', function (assert) {
         var todos = 0;
         var skips = 0;
         var testIds = [];
-        var endIds = []
+        var endIds = [];
         var asserts = 0;
 
         assert.equal(objects.length, 13);
@@ -52,17 +52,17 @@ tap.test('object results', function (assert) {
 
     tape('parent', function (t1) {
         t1.equal(true, true);
-        t1.test('child1', {skip: 'not installed'}, function (t2) {
+        t1.test('child1', {skip: true}, function (t2) {
             t2.equal(true, true);
             t2.equal(true, false);
             t2.end();
         });
-        t1.test('child2', {todo: 'will get done soon'}, function (t3) {
+        t1.test('child2', {todo: true}, function (t3) {
             t3.equal(true, false);
             t3.equal(true, true);
             t3.end();
         });
-        t1.test('child3', {todo: 'will do'});
+        t1.test('child3', {todo: true});
         t1.equal(true, true);
         t1.equal(true, true);
         t1.end();

diff --git a/test/todo_explanation.js b/test/todo_explanation.js