summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatt Robinson <matt@puppetlabs.com>2011-07-26 13:03:59 -0700
committerMatt Robinson <matt@puppetlabs.com>2011-07-26 13:03:59 -0700
commit4857ac928ccf1bd56d513eae201b57b12dd14a38 (patch)
tree3ab71a6160b48e270ff3e483c1a4ebab7fa1e29c
parentdd97d7b054df5dd1d36f32387ddf30afeefad769 (diff)
parent778127d0c3cf01701d227ec88a8e5e6550638c35 (diff)
downloadpuppet-4857ac928ccf1bd56d513eae201b57b12dd14a38.tar.gz
puppet-4857ac928ccf1bd56d513eae201b57b12dd14a38.tar.xz
puppet-4857ac928ccf1bd56d513eae201b57b12dd14a38.zip
Merge branch 'ticket/2.7.x/maint-show_help_for_cert_without_subcommand' into 2.7.x
* ticket/2.7.x/maint-show_help_for_cert_without_subcommand: maint: Fix cert app to print help and exit if no subcommand
-rw-r--r--lib/puppet/application/cert.rb3
-rwxr-xr-xspec/unit/application/cert_spec.rb10
2 files changed, 12 insertions, 1 deletions
diff --git a/lib/puppet/application/cert.rb b/lib/puppet/application/cert.rb
index 162672b6a..330fba8bd 100644
--- a/lib/puppet/application/cert.rb
+++ b/lib/puppet/application/cert.rb
@@ -218,7 +218,8 @@ Copyright (c) 2011 Puppet Labs, LLC Licensed under the Apache 2.0 License
if sub = self.command_line.args.shift then
self.subcommand = sub
else
- help
+ puts help
+ exit
end
end
result
diff --git a/spec/unit/application/cert_spec.rb b/spec/unit/application/cert_spec.rb
index 7510f0783..300234c2b 100755
--- a/spec/unit/application/cert_spec.rb
+++ b/spec/unit/application/cert_spec.rb
@@ -208,5 +208,15 @@ describe Puppet::Application::Cert, :'fails_on_ruby_1.9.2' => true do
args.should == ["fun.example.com"]
end
end
+
+ it "should print help and exit if there is no subcommand" do
+ args = []
+ @cert_app.command_line.stubs(:args).returns(args)
+ @cert_app.stubs(:help).returns("I called for help!")
+ @cert_app.expects(:puts).with("I called for help!")
+
+ expect { @cert_app.parse_options }.to exit_with 0
+ @cert_app.subcommand.should be_nil
+ end
end
end