lita-cricket 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c638ba5ba8f85f6a7ec494d3dd863571b068cf5d
-  data.tar.gz: 47f8400449b02dedc21fb73b1423a1b314bd620d
+  metadata.gz: bc6176aa9d1a016b6fe0afeebce549d554db81e7
+  data.tar.gz: a98b714bb1d4c4d197dae5cad64aa9e9082c0195
 SHA512:
-  metadata.gz: 5a3e9bbdeb8b625047cd39704a7c020258895af0a095cccad3eac88c4249fa7e741f7f99d5dd3c5fcb6607833180575290639379f9c3decca6b62ad7e59a4518
-  data.tar.gz: bca90ad637245dd4c17a5c262f47708eb774acc7b661419d7cd0890bd5d4ea0eacb7b556518c5933105a891d1fcd3c550551babcbef44941a5d9d0e4230c0479
+  metadata.gz: 303277d220fbcc0fb3574a87471853048b279cef20760756e28ccbe145e92c83dd31c1ffe0ede776ff73f02451eb3215b79e59cabf5a754abf583d3dd0fcb981
+  data.tar.gz: 44ac23ec8fe8aac2cd01d4317d068b803669564da930cfca9ce98e8c393c78232065c4dcf6de833e53667f8ba3387eadb96f19808eb5f260f8694a36ce287c2a

data/Rakefile

@@ -4,3 +4,12 @@ require "rspec/core/rake_task"
 RSpec::Core::RakeTask.new(:spec)
 
 task default: :spec
+
+# rake console is fun!
+task :console do
+  require 'irb'
+  require 'irb/completion'
+  require 'lita-cricket' # You know what to do.
+  ARGV.clear
+  IRB.start
+end

data/lib/lita/handlers/cricket.rb

@@ -2,7 +2,7 @@ module Lita
   module Handlers
     class Cricket < Handler
       # Version
-      VERSION = '0.0.3'
+      VERSION = '0.0.4'
 
       # Dependencies
       require 'json'
@@ -84,11 +84,13 @@ module Lita
         if my_favourites.empty?
           set_my_subscriptions(response,['Australia'].to_json,'favourite')
           response.reply('I can give you live cricket updates! Type `help cricket` for more information.')
-        elsif
-          matches = get_list_of_live_matches
-          matches.each do |m|
-            unless ([ m['t1'], m['t2']] & my_favourites).empty?
-              subscribe_to(response,m['id'],'match')
+        else
+          status, matches = get_list_of_live_matches
+          if status == :success
+            matches.each do |m|
+              unless ([ m['t1'], m['t2']] & my_favourites).empty?
+                subscribe_to(response,m['id'],'match')
+              end
             end
           end
         end
@@ -123,13 +125,18 @@ module Lita
       end
 
       def list(response)
-        resp = get_list_of_live_matches
-        #TODO: parse this list and keep going!!!
-        response.reply("There are #{resp.count} live matches on!")
-        resp.each do |r|
-          response.reply("#{r['t1']} vs #{r['t2']} (http://www.espncricinfo.com/c/engine/match/#{r['id']}.html)")
+        status, resp = get_list_of_live_matches
+        if status == :success
+          response.reply("There are #{resp.count} live matches on!")
+          resp.each do |m|
+            response.reply("#{m['t1']} vs #{m['t2']} (http://www.espncricinfo.com/c/engine/match/#{m['id']}.html)")
+          end
+        else
+          Lita.logger.error(resp.inspect)
+          response.reply("It looks like the API is down - #{resp.response.code}")
         end
-      rescue
+
+      rescue => e
         response.reply("An error may have occured, or maybe there are no live matches")
       end
 
@@ -159,7 +166,12 @@ module Lita
       end
 
       def get_list_of_live_matches
-        HTTParty.get(@@ENDPOINT).parsed_response
+        resp = HTTParty.get(@@ENDPOINT)
+        if resp.response.code.to_i > 299 || resp.response.code.to_i < 200
+          return :failure, resp
+        else
+          return :success, resp.parsed_response
+        end
       end
 
       def get_match_score(response,id)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-cricket
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Stuart Auld
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-27 00:00:00.000000000 Z
+date: 2015-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita