Skip to content

Commit

Permalink
remove redundant tests from test/skip.js - still testing the document…
Browse files Browse the repository at this point in the history
…ed API adequately
  • Loading branch information
nelsonic committed Sep 2, 2016
1 parent 8826099 commit aa021eb
Showing 1 changed file with 2 additions and 19 deletions.
21 changes: 2 additions & 19 deletions test/skip.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,6 @@ tap.test('test SKIP comment', function (assert) {
});
});


test('do not skip this', { skip: false }, function(t) {
t.pass('this should run');
ran ++;
t.end();
});

test('skip this', { skip: true }, function(t) {
t.fail('this should not even run');
ran++;
Expand All @@ -43,27 +36,17 @@ test('skip this', { skip: true }, function(t) {

test.skip('skip this too', function(t) {
t.fail('this should not even run');
ran++;
ran++;
t.end();
});

test('skip subtest', function(t) {
ran ++;
t.test('do not skip this', { skip: false }, function(t) {
ran ++;
t.pass('this should run');
t.end();
});
ran++;
t.test('skip this', { skip: true }, function(t) {
t.fail('this should not even run');
t.end();
});
t.end();
});

test('right number of tests ran', function(t) {
t.equal(ran, 3, 'ran the right number of tests');
t.end();
});

// vim: set softtabstop=4 shiftwidth=4:

0 comments on commit aa021eb

Please sign in to comment.