Skip to content

test: remove redundant cli tests

Rodrigo Muino Tomonari requested to merge github/fork/bengl/remove-cli-test into master

test/parallel/test-cli-eval.js covers it, and many more things.

Note: I know test redundancy can generally be a good thing, but when I came across test/parallel/test-eval.js, it appeared to me that this was the only test for cli eval. Removing this redundant (and incomplete) test reduces the chances of such confusion in the future. In addition, I'm not sure that redundant tests that simply do exactly the same thing as another test are necessarily useful. (EDIT: On further inspection, I also found that test/parallel/test-eval-require.js is similarly redundant, so this is now removed as well.)

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports