lita-pullrequests 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b2eeaab6bd4a825ee5ce6018fb4c2de5b7e1d514
-  data.tar.gz: da7813b33b98caa2b6b51dd03f79de499d92f2df
+  metadata.gz: 67a9de81353aaec844771cb2829ec714ba015481
+  data.tar.gz: 5814f6cc0ba4a548413daae6b5b4821de48fd1ed
 SHA512:
-  metadata.gz: 19bc54da7c1049c1f770ad2ae62a2b6cff3e6ff67ef88ca3a8e5441ca76c0fcc707d63e2973a3a7890a7d541c7765873b4196039621eedfd4e6bcadcfe5a25bc
-  data.tar.gz: b146587dfa273e7b887b0aa98b456eb890f7835f5e33e69084bb6819a611afcef17f4b93263ab38230f1e601d653f799aef0a83ee6efa711ccd37dbbe5c07c0c
+  metadata.gz: 856690826d022b56ec598672446823095c833c2c9b0de161b8b1c4cf92e07bd84b9764a69fba8c4b8b90e71473927edb61b00ec1f32d306a7d6947530e81cd27
+  data.tar.gz: 86e57efa9b4303bc6f6e3e200b715ce739bdad36d3d96ad58c114ba555e3d19023199b8be07432dd533ae093acf3e3babbfc631aec0ff82df42a9ca22d02ae7f

data/lib/lita/handlers/pullrequests.rb

@@ -22,8 +22,16 @@ module Lita
         "(summarize|all) pull requests" => "Lists all pull requests that need action."
       })
 
-      route(/^set pull requests reminder for (.*)$/, :set_reminder, command: true, help: {
-        "set pull requests reminder for CRON EXPRESSION" => "Sets a cron task that will trigger a pr summary."
+      route(/^set pull request(s)? reminder for (.*)$/, :set_reminder, command: true, help: {
+        "set pull request(s)? reminder for CRON EXPRESSION" => "Sets a cron task that will trigger a pr summary."
+      })
+
+      route(/^stop reminding me about pull requests$/, :remove_reminder, command: true, help: {
+        "stop reminding me about pull requests" => "Shows you info about your next pull request reminder."
+      })
+
+      route(/^show pull request(s)? reminder$/, :show_reminder, command: true, help: {
+        "show pull request(s) reminder" => "Shows you info about your next pull request reminder."
       })
 
       def remember_reminder(payload)
@@ -99,7 +107,7 @@ module Lita
       end
 
       def set_reminder(chat)
-        input = chat.matches[0][0].split(" ")
+        input = chat.matches[0][1].split(" ")
         cron_expression = input[0..4].join(" ")
         job = SCHEDULER.cron cron_expression do |job|
           list_all_pull_requests(chat)
@@ -114,6 +122,28 @@ module Lita
 
         chat.reply("I will post a pull request summary according to this cron: #{cron_expression}")
       end
+
+      def show_reminder(chat)
+        reminder = redis.hgetall(REDIS_KEY)["reminder-info"]
+        if reminder
+          info = MultiJson.load(reminder)
+          chat.reply "I will remind you in channel #{info["room"]} at #{info["cron_expression"]}"
+        else
+          chat.reply "Your reminder is not set."
+        end
+      end
+
+      def remove_reminder(chat)
+        reminder = redis.hgetall(REDIS_KEY)["reminder-info"]
+        if reminder
+          info = MultiJson.load(reminder)
+          SCHEDULER.unschedule(info["j_id"])
+          redis.hdel(REDIS_KEY, "reminder-info")
+          chat.reply "okay, I turned off your reminder."
+        else
+          chat.reply "Your reminder is not set."
+        end
+      end
     end
 
     Lita.register_handler(Pullrequests)

data/lita-pullrequests.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-pullrequests"
-  spec.version       = "0.0.4"
+  spec.version       = "0.0.5"
   spec.authors       = ["Taylor Lapeyre"]
   spec.email         = ["taylorlapeyre@gmail.com"]
   spec.description   = %q{A Lita handler to help you keep track of your pull requests.}

data/spec/lita/handlers/pullrequests_spec.rb

@@ -14,6 +14,10 @@ describe Lita::Handlers::Pullrequests, lita_handler: true do
   it { is_expected.to route_command("all pull requests").to(:list_all_pull_requests) }
   it { is_expected.to route_command("summarize pull requests").to(:list_all_pull_requests) }
   it { is_expected.to route_command("set pull requests reminder for 0 20 * * 1-5").to(:set_reminder) }
+  it { is_expected.to route_command("set pull request reminder for 0 20 * * 1-5").to(:set_reminder) }
+  it { is_expected.to route_command("stop reminding me about pull requests").to(:remove_reminder) }
+  it { is_expected.to route_command("show pull requests reminder").to(:show_reminder) }
+  it { is_expected.to route_command("show pull request reminder").to(:show_reminder) }
 
   it { is_expected.to_not route_command("all pull requests").to(:get_random_pr) }
 
@@ -34,4 +38,18 @@ describe Lita::Handlers::Pullrequests, lita_handler: true do
     send_command("set pull requests reminder for 0 20 * * 1-5")
     expect(replies.last).to match /0 20 \* \* 1\-5/
   end
+
+  it "can stop reminding you" do
+    send_command("set pull requests reminder for 0 20 * * 1-5")
+    expect(replies.last).to match /0 20 \* \* 1\-5/
+    send_command("stop reminding me about pull requests")
+    expect(replies.last).to eq "okay, I turned off your reminder."
+  end
+
+  it "can tell you when it will remind you next" do
+    send_command("set pull requests reminder for 0 20 * * 1-5")
+    expect(replies.last).to match /0 20 \* \* 1\-5/
+    send_command("show pull requests reminder")
+    expect(replies.last).to match /I will remind you/
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-pullrequests
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Taylor Lapeyre