lita-ping-me 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9810b61053b12381679d74f7be819e14d9b98406
-  data.tar.gz: 9e9ff345bd62649e66b5fe1dfe055c62864a38b0
+  metadata.gz: 5634f70a83b8256e0411d384b9af6425d7805656
+  data.tar.gz: e6af0a61a28a859a3ce91b76c5b7d77cfe46d544
 SHA512:
-  metadata.gz: f675f5fbff9e9eb3932a5c9a40818363231e251f9de921fd990711d96c2f0ce8d8a6a4b94d8f46e4e5bcfe7b65b423d29cfdd27dbeaf242c4deb407842934ab4
-  data.tar.gz: 1cfee08513c40e8a1a2b80d0fb0b23de6b84489f064686ef5812f9906d79585d82869ea39df027d0f596931127ab0b2e5ab629d3cdb572859786907796947a91
+  metadata.gz: 55f94679ca8c27028357c2e986e5f9fec27f522011a36af3c31ad8b672fa55cb08017beaa2969a04cefdb997f9724aa3e05b2f4d1cde9175194bf892511edfbb
+  data.tar.gz: 6dbf15089de9fcb6b9079528bd1546dc8c461c2761ac4bfa0948da1d72e0bbd999e21c6be1288db88f18ded7e283c43c1b869a8bc3d847b47667ff8b573b43f1

data/lib/lita/handlers/ping-me.rb

@@ -30,6 +30,7 @@ module Lita
       end
 
       route(/^sleep[\s+]?(\d+)?/, :sleep)
+      route(/^wake?/, :wake)
 
       route(/^any errors\?/, :check_for_errors)
 
@@ -50,9 +51,13 @@ module Lita
       end
 
       def sleep(response)
+        @sleeping = true
         time = parse_time(response.args.first.to_i)
         timer.stop
-        after(time) { start_pinging }
+        after(time) do
+          @sleeping = false
+          start_pinging
+        end
         response.reply("Sure, will sleep for #{seconds_to_minutes(time)} minutes")
       end
 
@@ -63,10 +68,15 @@ module Lita
       end
 
       def notify(request, response)
+        return if @sleeping
         id = request.env["router.params"][:id]
         msg = id.split('_').join(" ")
         send_message("#{msg}")
-        response.body << "#{msg} #{request.user_agent}!"
+        response.body << "#{msg}"
+      end
+
+      def wake(_)
+        @sleeping = false
       end
 
       private
@@ -100,7 +110,7 @@ module Lita
       end
 
       def target(payload = {})
-        @target ||= Source.new(room: config.room)
+        @target ||= Source.new(room: config.room || '')
       end
 
       def send_message(msg)

data/lita-ping-me.gemspec

@@ -4,7 +4,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = "lita-ping-me"
-  spec.version       = '0.0.2'
+  spec.version       = '0.0.3'
   spec.authors       = ["Douglas Roper"]
   spec.email         = ["dougdroper@gmail.com"]
   spec.summary       = %q{Lita plugin for periodically checking statuses of websites}
@@ -25,4 +25,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.7"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec", "~> 3.3"
+  spec.add_development_dependency "rack-test", '~> 0.6'
 end

data/spec/lib/handlers/ping-me_spec.rb

@@ -59,10 +59,26 @@ RSpec.describe Lita::Handlers::PingMe, lita_handler: true do
       let(:current_status) { {'http://test.com' => { code: 200, time: 4.882903 }} }
 
       it 'responds with the correct status' do
-        expect(Lita::Handlers::Connection).to receive(:new).with(["http://test.com"]).and_return(response)
+        expect(Lita::Handlers::Connection).to receive(:new).and_return(response)
         send_message("status <http://test.com|test.com>")
         expect(replies.last).to eq("http://test.com: code 200, time: 4.882903")
       end
     end
+
+    context 'notify' do
+      it 'notifies when something goes wrong' do
+        response = http.get("/notify/something_has_gone_wrong")
+        expect(response.body).to eq("something has gone wrong")
+      end
+
+      it 'doesn\'t notify when it is sleeping' do
+        send_message('sleep')
+        response = http.get("/notify/something_has_gone_wrong")
+        expect(response.body).to eq("")
+        send_message('wake')
+        response = http.get("/notify/something_has_gone_wrong")
+        expect(response.body).to eq('something has gone wrong')
+      end
+    end
   end
 end

data/spec/spec_helper.rb

@@ -1,7 +1,6 @@
-require 'pry'
-
 require 'rspec'
 require "lita/rspec"
+require "rack/test"
 require_relative '../lib/lita-ping-me.rb'
 
 Lita.version_3_compatibility_mode = false

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-ping-me
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Douglas Roper
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-26 00:00:00.000000000 Z
+date: 2015-06-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -80,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.3'
+- !ruby/object:Gem::Dependency
+  name: rack-test
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
 description: "\n    periodically checks status of website and alerts chat room if
   response error\n  "
 email: