Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

split cli parsing #2553

Closed
wants to merge 26 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
e23b447
WIP shifted files around
rijnhard Oct 20, 2016
abdf5eb
few defaults to make the functions more lenient.
rijnhard Oct 20, 2016
8774377
got rid of shitty getOptions and fixex program.opts() on Command.
rijnhard Oct 21, 2016
818492f
decided to force using makeCommand where possible,
rijnhard Oct 21, 2016
53f77c9
new mocha cli (untested)
rijnhard Oct 21, 2016
7e5e27b
storing globals in a seperate var is pointless.
rijnhard Oct 21, 2016
ad489f5
un-dufused the removal of the globals.
rijnhard Oct 21, 2016
9cc9efd
updated cli with changes
rijnhard Oct 21, 2016
1ae8842
fixed parser, it now runs
rijnhard Oct 24, 2016
639b441
fixed resolveFiles
rijnhard Oct 24, 2016
eb96de7
make run throw error if files is empty
rijnhard Oct 24, 2016
8d583d0
change program.parse to be protected.
rijnhard Oct 24, 2016
375734d
more tests
rijnhard Oct 24, 2016
411026b
gave up on tests i have no idea how to write
rijnhard Oct 24, 2016
3858674
added make test-cli command
rijnhard Oct 24, 2016
cc3d7cc
expose mocha cli on node only
rijnhard Oct 24, 2016
08ba0ef
Stopped overriding Command.parse and moved it all to Command.parseOpt…
rijnhard Oct 25, 2016
6a335ef
fixed regression tests
rijnhard Oct 25, 2016
8b8db38
Cheeky commit: updated Commander options --global --require
rijnhard Oct 26, 2016
bbc79ca
removed mkdir in command
rijnhard Nov 10, 2016
7fff92a
Merge branch 'master' of github.com:mochajs/mocha into feature/split_…
rijnhard Nov 10, 2016
5572b50
stripped out hacky Command.parseOptions code
rijnhard Nov 16, 2016
38102ef
No longer expose Cli for reuse
rijnhard Nov 16, 2016
2488485
addActions can be used on any Command instance
rijnhard Nov 16, 2016
9668709
fixed loadOptsFile
rijnhard Dec 12, 2016
833413f
Removed confusing files requirement for cli run
rijnhard Dec 22, 2017
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 6 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ lint:
@printf "==> [Test :: Lint]\n"
$(ESLINT) .

test-node: test-bdd test-tdd test-qunit test-exports test-unit test-integration test-jsapi test-compilers test-glob test-requires test-reporters test-only test-global-only
test-node: test-bdd test-tdd test-qunit test-exports test-unit test-integration test-jsapi test-compilers test-glob test-requires test-reporters test-cli test-only test-global-only

test-browser: clean mocha.js test-browser-unit test-browser-bdd test-browser-qunit test-browser-tdd test-browser-exports

Expand Down Expand Up @@ -116,6 +116,11 @@ test-reporters:
$(MOCHA) --reporter $(REPORTER) \
test/reporters/*.js

test-cli:
@printf "==> [Test :: Cli]\n"
$(MOCHA) --reporter $(REPORTER) \
test/cli/*.js

test-only:
@printf "==> [Test :: Only]\n"
$(MOCHA) --reporter $(REPORTER) \
Expand Down