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

No error message when an unknown option and two args are given #121

Closed
wants to merge 1 commit into from

Conversation

nitoyon
Copy link

@nitoyon nitoyon commented Jan 26, 2013

When an option -u is not defined:

  • node foo.js -u -> unknown option error (expected)
  • node foo.js -u arg1 -> unknown option error (expected)
  • node foo.js -u arg1 arg2 -> NO ERROR (unexpected)

Another example: mocha --reporterrrr spec test (should be mocha --reporter spec test) shows no error.

This bug is caused by Command.parseArgs() assuming arg2 to be a command name.

@SomeKittens
Copy link
Collaborator

Closing, will reopen if this gets updated with latest master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Commander is not working as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants