pushover 1.0.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0e2a717fd5728c512f6c58a9fef3f1e40a8b3103
-  data.tar.gz: 020060276a59e65b83fc5a363dcfe7013f5e534e
+  metadata.gz: 5a70f1e785179b7356c2c887632f283de549006f
+  data.tar.gz: d54b26659eeefb08394b6c266af442832182f5d7
 SHA512:
-  metadata.gz: fd7f18dfbd1b3584b5ef27c45a6f011c02f9465cdcda9258ee343bffe465c79b57c3bccefa0a405a8989cbbd53410193111c9ae79bf5a06aaad01e7554d0e70d
-  data.tar.gz: 6941590b5140b4c38bd173e21d090b7c3f026ed8a8dd023668cf83aaf30af9e37bb9121fdbdc2e357c785800bb8b5f4f88353964419d50be4c4f04911a24c7dd
+  metadata.gz: 465930a2259084a11269263cd8eb5d4048a9c0551c9214e282bbc6277684f271686812acbd80dc26558325cd6ef9ccc5badddf97c8ca51f6100322fa8296daf6
+  data.tar.gz: d3e911be81b0e044f3dde95a06c114d3f8ad8005a64e2de665aa2aeb829bcbb7edbb4de342d73d4b5d5d0defac475b0153787b2ea139e8c7abe3c4be5cecb90e

data/bin/pushover

@@ -13,8 +13,8 @@ Options.on("-c", "--config_file [FILE]", "Set the target config file.") {|o| Opt
 Options.on("--url [URL]", "Supplementary URL") { |o| Options[:url] = o }
 Options.on("--url_title [TITLE]", "Supplementary URL title.") { |o| Options[:url_title] = o }
 Options.on("--time [TIME]", "Set the messages time.") {|o| Options[:timestamp] = o}
-Options.on("--save-app NAME", "Saves the application to the config file under NAME.") { |o| Options[:save_app] = [Options[:appkey], o]}
-Options.on("--save-user NAME", "Saves the user to the config file under NAME.") { |o| Options[:save_user] = [Options[:user], o]}
+Options.on("--save-app NAME", "Saves the application to the config file under NAME.") { |o| Options[:save_app] = o}
+Options.on("--save-user NAME", "Saves the user to the config file under NAME.") { |o| Options[:save_user] = o}
 Options.on("--sound [SOUND]", "Specify the sound to use.  Can be a partial string as long as it is unambiguous enough.") { |o| Options[:sound] = o}
 Options.on("--sound_list", "Display the current list of available sounds.  Requires an app token.") { |o| Options[:sound_list] = true}
 Options.on("--emergency_retry [TIME]", "The time in seconds between retries.") { |o| Options[:retry] = o}
@@ -29,17 +29,20 @@ bail = false
 if Options[:config_file]
 	Bini::Config.options[:file] = Options[:config_file]
 	puts "Selected config file: #{Options[:config_file]}"
+	# TODO: Make a healper for changing config so it clears/loads automatically.
+	Bini::Config.clear
+	Bini::Config.load
 end
 
 if Options[:save_app]
-	App.add Options[:save_app][1], Options[:save_app][0]
-	puts "Saved application #{Options[:save_app][0]} to #{Options[:save_app][1]}."
+	App.add Options[:save_app], Options[:token]
+	puts "Saved application #{Options[:token]} to #{Options[:save_app]}."
 	bail = true
 end
 
 if Options[:save_user]
-	User.add Options[:save_user][1], Options[:save_user][0]
-	puts "Saved user #{Options[:save_user][0]} to #{Options[:save_user][1]}."
+	User.add Options[:save_user], Options[:user]
+	puts "Saved user #{Options[:user]} to #{Options[:save_user]}."
 	bail = true
 end
 

data/lib/pushover/version.rb

@@ -1,4 +1,4 @@
 module Pushover
 	# The current version of Pushover.
-  VERSION = "1.0.2"
+  VERSION = "1.0.3"
 end

data/spec/bin/pushover_spec.rb

@@ -50,16 +50,18 @@ if ENV["TEST_CLI"] =~ /^t/
 
     describe "saving" do
       it "saves app:key pairs" do
-        p = CLIProcess.new "#{CMD} --config_file #{FAKE_CRED_FILE} --save-app default --app default"
+        p = CLIProcess.new "#{CMD} --config_file #{FAKE_CRED_FILE} --save-app saveapp --app application"
         p.run!
         p.stdout.should include 'Saved'
-        open(FAKE_CRED_FILE).read.should include 'default'
+        output = YAML.load open(FAKE_CRED_FILE).read
+        output[:applications]["saveapp"].should eq 'application'
       end
       it "saves user:token pairs" do
-        p = CLIProcess.new "#{CMD} --config_file #{FAKE_CRED_FILE} --save-user default --user default"
+        p = CLIProcess.new "#{CMD} --config_file #{FAKE_CRED_FILE} --save-user saveuser --user user"
         p.run!
         p.stdout.should include 'Saved'
-        open(FAKE_CRED_FILE).read.should include 'default'
+        output = YAML.load open(FAKE_CRED_FILE).read
+        output[:users]["saveuser"].should eq 'user'
       end
     end
     describe "sounds" do

data/whatsnew.md

@@ -1,3 +1,6 @@
+## 1.0.3
+  * Fixed the save option to properly store the appkey and user key (@cptobvious for this).  As a result I added tests and cleaned up some internal code.
+
 ## 1.0.2
   * hotfixes
   * #13 added license to the gemspec

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pushover
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Ernie Brodeur
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-26 00:00:00.000000000 Z
+date: 2013-09-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: yajl-ruby