lita-time 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d14a585cc3df387b9605bd2a99339ba52e79bae4
-  data.tar.gz: 1ad22d8244bdb6c359fd04e6036988f6e08bfca4
+  metadata.gz: 27602df60f8785484a1e6fb79b7ec4bf0e5617e7
+  data.tar.gz: 3e23d6d15a05b2ddc6f922694b3397599b117ff8
 SHA512:
-  metadata.gz: 7d4f9ef11149796f451409b249b4fbe31fd66beb5c4cf3d9c9d18c6d4b22efcfff1f0a84dd77afbd5fb45c0e6d8038c69d78a8dc21559311d799de09f083d6f1
-  data.tar.gz: ab8c35813b016cec8ab59716ff17e3d78c8d41778e2b051d7014ce4d73cabc0938430122e55b56082b3e3f3882790f2eaa795e4e4c87b6a275489332e4d2ae4b
+  metadata.gz: d3dee1818a67de71adf0d6b5a2e264e89733d7abdaa905cc8bf069f271e2df6b91c35763b27642237f4723a2095570db4abcfa9d023c467b3903ebb5ed1a1ff6
+  data.tar.gz: 4795735dfbdd9de0e181640c332540277e168fca4af281a99ab09eaaccd12fbc612f454c556825eeda90aaf83b62a48687d626621aab9d3a5184488f019b84f4

data/lib/lita/handlers/time.rb

@@ -23,13 +23,14 @@ module Lita
           format: 'json'
         )
 
-        data = MultiJson.load(http_response.body)["data"]
-
         begin
+          data = MultiJson.load(http_response.body)["data"]
           type  = data['request'][0]['type']
           query = data['request'][0]['query']
           time  = data['time_zone'][0]['localtime'].split(/ /)[1]
           response.reply t("response.success", :type => type, :query => query, :time => time)
+        rescue MultiJson::ParseError
+          response.reply t("response.json_failure", :location => location)
         rescue
           response.reply t("response.failure", :location => location)
         end

data/lita-time.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-time"
-  spec.version       = "0.1.0"
+  spec.version       = "0.1.1"
   spec.authors       = ["Harlan Barnes"]
   spec.email         = ["hbarnes@pobox.com"]
   spec.description   = %q{A Lita handler for getting the curent time for a given location}

data/locales/en.yml

@@ -8,3 +8,4 @@ en:
         response:
           success: "The current time for the %{type} of %{query} is %{time}"
           failure: "Sorry, couldn't find %{location}"
+          json_failure: "Error parsing API response for %{location}"

data/spec/lita/handlers/time_spec.rb

@@ -64,5 +64,14 @@ JSON
         "Sorry, couldn't find not a real place"
       )
     end
+
+    it "replies with a helpful message when the API returns non-JSON" do
+      allow(response).to receive(:body).and_return("")
+
+      send_command("time Mableton, GA")
+      expect(replies.last).to eq(
+        "Error parsing API response for Mableton, GA"
+      )
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-time
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Harlan Barnes
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-25 00:00:00.000000000 Z
+date: 2016-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -134,11 +134,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.1
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: A Lita handler for getting the curent time for a given location
 test_files:
 - spec/lita/handlers/time_spec.rb
 - spec/spec_helper.rb
-has_rdoc: