test: remove redundant cli tests
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), orvcbuild test
(Windows) passes -
commit message follows commit guidelines