Skip to content

test_help changes #211

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.

Sign up for GitHub

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

Merged
BurtBiel merged 8 commits into from
May 5, 2016
Merged

test_help changes #211

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d2e6be5
put changes back to the way they were originally
BurtBiel May 5, 2016
c8106c1
override exit to match 3.5
BurtBiel May 5, 2016
fc22b0c
lint
BurtBiel May 5, 2016
785d4c8
manually raise SystemExit
BurtBiel May 5, 2016
35695f0
another approach
BurtBiel May 5, 2016
7a2a162
yet another approach
BurtBiel May 5, 2016
66a7204
remove dup code
BurtBiel May 5, 2016
e1fc882
add comment
BurtBiel May 5, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 11 additions & 3 deletions src/azure/cli/tests/test_help.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -469,10 +469,18 @@ def test_handler(args):
config.get_command_table = lambda: cmd_table
app = Application(config)

# there is an argparse bug on <2.7.10 where SystemExit is not thrown on missing required param
# work around an argparse behavior where output is not printed and SystemExit
# is not raised on Python 2.7.9
if sys.version_info < (2, 7, 10):
app.execute('n1 -fb a --foobar value'.split())
app.execute('n1 -fb a --foobar2 value --foobar3 extra'.split())
try:
app.execute('n1 -fb a --foobar value'.split())
except SystemExit:
pass

try:
app.execute('n1 -fb a --foobar2 value --foobar3 extra'.split())
except SystemExit:
pass
else:
with self.assertRaises(SystemExit):
app.execute('n1 -fb a --foobar value'.split())
Expand Down