lita-pullrequests 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 177ca635bc45eb15a13d62c68e2e251743ca1edf
-  data.tar.gz: 10140d8e50828f11627c1a1a909b8c533e4fdada
+  metadata.gz: b2eeaab6bd4a825ee5ce6018fb4c2de5b7e1d514
+  data.tar.gz: da7813b33b98caa2b6b51dd03f79de499d92f2df
 SHA512:
-  metadata.gz: 2982816a7cfe109c0e1213c4b282d8190af2f9ed7b6cb8b978787b75d629fac1ae1e7e6085aa186a660530a13039f7b81d8b6e2eea893bf82aef4a4951d04af3
-  data.tar.gz: 39c7b3e47f7599df01b7f697abc265b26b06f2568649d441c284e47cab188fcf252132c823728c895d54afca47264df52c5eb5dfccad38ed8c485f3f724da538
+  metadata.gz: 19bc54da7c1049c1f770ad2ae62a2b6cff3e6ff67ef88ca3a8e5441ca76c0fcc707d63e2973a3a7890a7d541c7765873b4196039621eedfd4e6bcadcfe5a25bc
+  data.tar.gz: b146587dfa273e7b887b0aa98b456eb890f7835f5e33e69084bb6819a611afcef17f4b93263ab38230f1e601d653f799aef0a83ee6efa711ccd37dbbe5c07c0c

data/lib/lita/handlers/pullrequests.rb

@@ -1,3 +1,5 @@
+require "rufus-scheduler"
+
 module Lita
   module Handlers
     class Pullrequests < Handler
@@ -6,6 +8,11 @@ module Lita
       config :review_label, type: String, required: false
       config :merge_label,  type: String, required: false
 
+      on :loaded, :remember_reminder
+
+      SCHEDULER = Rufus::Scheduler.new
+      REDIS_KEY = "pullrequests-cron"
+
       route(/^(pull request( me)?)|(give me something to review)$/, :get_random_pr, command: true, help: {
         "give me something to review" => "Shows you a random pull request that needs reviewing.",
         "pull request (me)" => "Shows you a random pull request that needs reviewing."
@@ -15,20 +22,23 @@ 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."
+      })
 
-      # Helper method
-      def truncate(str, truncate_at, options = {})
-        return str unless str.length > truncate_at
+      def remember_reminder(payload)
+        reminder = redis.hgetall(REDIS_KEY)["reminder-info"]
+        if reminder
+          info = MultiJson.load(reminder)
+          job  = SCHEDULER.cron info["cron_expression"] do |job|
+            target = Source.new(user: info["u_id"], room: info["room"])
+            robot.send_messages(target, formatted_pull_request_summary)
+          end
 
-        omission = options[:omission] || '...'
-        length_with_room_for_omission = truncate_at - omission.length
-        stop = if options[:separator]
-           str.rindex(options[:separator], length_with_room_for_omission) || length_with_room_for_omission
+          log.info "Created cron job: #{info['cron_expression']}."
         else
-          length_with_room_for_omission
+          log.info "no reminder found."
         end
-
-        "#{str[0, stop]}#{omission}"
       end
 
       def fetch_pull_requests
@@ -41,15 +51,8 @@ module Lita
       def get_random_pr(chat)
         pr = pulls_that_need_reviews.sample
         if pr
-          title, user, body = pr["title"], pr["user"]["login"], truncate(pr["body"], 200)
-          url = pr["pull_request"]["html_url"]
-
-          chat.reply %Q(
-  #{title} - #{user}
-  #{url}
-  -------------------
-  #{body}
-)
+          title, user, url = pr["title"], pr["user"]["login"], pr["pull_request"]["html_url"]
+          chat.reply "_#{title}_ - #{user} \n    #{url}"
         else
           chat.reply "No pull requests need reviews right now!"
         end
@@ -73,7 +76,7 @@ module Lita
         end
       end
 
-      def list_all_pull_requests(chat)
+      def formatted_pull_request_summary
         response = ":heavy_exclamation_mark: *Pull Requests that need review*:\n"
 
         response += pulls_that_need_reviews.map do |pr|
@@ -89,8 +92,27 @@ module Lita
           url = pr["pull_request"]["html_url"]
           "- _#{title}_ - #{user} \n    #{url}"
         end.join("\n\n")
+      end
+
+      def list_all_pull_requests(chat)
+        chat.reply(formatted_pull_request_summary)
+      end
+
+      def set_reminder(chat)
+        input = chat.matches[0][0].split(" ")
+        cron_expression = input[0..4].join(" ")
+        job = SCHEDULER.cron cron_expression do |job|
+          list_all_pull_requests(chat)
+        end
+
+        redis.hset(REDIS_KEY, "reminder-info", {
+          :cron_expression => cron_expression,
+          :j_id => job,
+          :u_id => chat.message.source.user.id,
+          :room => chat.message.source.room
+        }.to_json)
 
-        chat.reply(response)
+        chat.reply("I will post a pull request summary according to this cron: #{cron_expression}")
       end
     end
 

data/lita-pullrequests.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-pullrequests"
-  spec.version       = "0.0.3"
+  spec.version       = "0.0.4"
   spec.authors       = ["Taylor Lapeyre"]
   spec.email         = ["taylorlapeyre@gmail.com"]
   spec.description   = %q{A Lita handler to help you keep track of your pull requests.}
@@ -15,6 +15,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_runtime_dependency "lita", ">= 4.1"
+  spec.add_runtime_dependency "rufus-scheduler"
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"

data/spec/lita/handlers/pullrequests_spec.rb

@@ -13,6 +13,7 @@ describe Lita::Handlers::Pullrequests, lita_handler: true do
   it { is_expected.to route_command("give me something to review").to(:get_random_pr) }
   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_not route_command("all pull requests").to(:get_random_pr) }
 
@@ -28,4 +29,9 @@ describe Lita::Handlers::Pullrequests, lita_handler: true do
     expect(replies.last).to match /Example of a pull request ready for merge/
     expect(replies.last).to match /Example of a pull request ready for review/
   end
+
+  it "can schedule a reminder" do
+    send_command("set pull requests reminder for 0 20 * * 1-5")
+    expect(replies.last).to match /0 20 \* \* 1\-5/
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-pullrequests
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Taylor Lapeyre
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.1'
+- !ruby/object:Gem::Dependency
+  name: rufus-scheduler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement