Skip to content

Commit

Permalink
remove unused interface tests (#4247)
Browse files Browse the repository at this point in the history
* Remove bdd.spec.js and tdd.spec.js

These tests are deprecated. Only a few tests in the last of the files
that checks clear cases were running. Since there are integration
tests that checks the `only` behavior in integration/only.spec.js,
it's okay to remove these tests.

* Remove qunit.spec.js

qunit.spec.js was not exeuced in `npm run test`. Also there are
integration tests for the qunit in integration/only.spec.js.

* Remove unused bdd.spec.js and tdd.spec.js
  • Loading branch information
majecty committed Sep 11, 2020
1 parent b3eb2a6 commit 5bdc208
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 347 deletions.
18 changes: 0 additions & 18 deletions package-scripts.js
Original file line number Diff line number Diff line change
Expand Up @@ -158,8 +158,6 @@ module.exports = {
only: {
default: {
script: `nps ${[
'test.node.only.bdd',
'test.node.only.tdd',
'test.node.only.bddRequire',
'test.node.only.globalBdd',
'test.node.only.globalTdd',
Expand All @@ -168,22 +166,6 @@ module.exports = {
description: 'Run Node.js "only" functionality tests',
hiddenFromHelp: true
},
bdd: {
script: test(
'only-bdd',
'--ui bdd test/only/bdd.spec --no-parallel'
),
description: 'Run Node.js "only" w/ BDD interface tests',
hiddenFromHelp: true
},
tdd: {
script: test(
'only-tdd',
'--ui tdd test/only/tdd.spec --no-parallel'
),
description: 'Run Node.js "only" w/ TDD interface tests',
hiddenFromHelp: true
},
bddRequire: {
script: test(
'only-bdd-require',
Expand Down
127 changes: 0 additions & 127 deletions test/only/bdd.spec.js

This file was deleted.

75 changes: 0 additions & 75 deletions test/only/qunit.spec.js

This file was deleted.

127 changes: 0 additions & 127 deletions test/only/tdd.spec.js

This file was deleted.

0 comments on commit 5bdc208

Please sign in to comment.