wrapp 0.2.1 → 0.2.2

data/README.md

@@ -53,6 +53,7 @@ the given App. (the filename automatically includes the name and version).
 
 Thats it.
 
+(NOTE: On authorization errors try prefixing the command with `sudo`!)
 
 ## Contributing
 

data/features/step_definitions/wrapp_steps.rb

@@ -70,3 +70,8 @@ Then(/^the App should be wrapped including the parent directory$/) do
   attached_app_path = File.join(volumes_dir, @app.prefix)
   check_directory_presence([attached_app_path], true)
 end
+
+Then(/^I should see usage instructions$/) do
+  expected = "Usage: wrapp [options] APP_PATH"
+  assert_partial_output(expected, all_output)
+end

data/features/wrap_app.feature

@@ -13,3 +13,7 @@ Feature: Wrap App
     Given an App in a directory
     When I wrap the App including the parent directory
     Then the App should be wrapped including the parent directory
+
+  Scenario: Display usage
+    When I run `wrapp`
+    Then I should see usage instructions

data/lib/wrapp/cli.rb

@@ -18,7 +18,13 @@ module Wrapp
 
     def run(argv)
       app_path = parse_options(argv).first
-      wrapp(app_path, config)
+      if app_path
+        wrapp(app_path, config)
+      else
+        warn 'ERROR: App path is missing!'
+        puts opt_parser
+        exit 2 
+      end
     end
 
     def wrapp(*opts)

data/lib/wrapp/version.rb

@@ -1,3 +1,3 @@
 module Wrapp
-  VERSION = '0.2.1'
+  VERSION = '0.2.2'
 end

data/spec/wrapp/cli_spec.rb

@@ -5,12 +5,42 @@ module Wrapp
     let(:cli) { CLI.new }
     let(:app_path) { '/Applications/Chunky Bacon.app' }
 
+    it 'include usage instructions in the banner' do
+      expect(CLI.banner).to match(/^usage: \w+ \[options\] \w+$/i)
+    end
+
     describe '.run' do
       it 'runs an instance with ARGV' do
         cli.should_receive(:run).with(ARGV)
         CLI.stub(:new).and_return(cli)
         CLI.run
       end
+
+      context 'without arguments' do
+        let(:argv) { [] }
+
+        before do
+          cli.stub(:warn)
+          cli.stub(:puts)
+          cli.stub(:exit)
+        end
+
+        it 'exits non-zero' do
+          cli.should_receive(:exit).with(2)
+          cli.run(argv)
+        end
+
+        it 'displays usage on stdout' do
+          cli.should_receive(:opt_parser).and_return('usage')
+          cli.should_receive(:puts).with('usage')
+          cli.run(argv)
+        end
+
+        it 'displays what is missing on stderr' do
+          cli.should_receive(:warn).with('ERROR: App path is missing!')
+          cli.run(argv)
+        end
+      end
     end
 
     describe '#run' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wrapp
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
   prerelease: 
 platform: ruby
 authors:
@@ -183,7 +183,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -527064214639857781
+      hash: 490571108544621046
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -192,13 +192,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -527064214639857781
+      hash: 490571108544621046
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23
 signing_key: 
 specification_version: 3
-summary: wrapp-0.2.1
+summary: wrapp-0.2.2
 test_files:
 - features/step_definitions/wrapp_steps.rb
 - features/support/app.rb