lita-debug-queue 0.1.9 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 197609338433856720c939f87f4c3d66884e7142
-  data.tar.gz: 39b599c089cec86a32aaa3c089ede61464f9c5c8
+  metadata.gz: 13062639eb8ef466d93d44bc1421d2d8a6643cd5
+  data.tar.gz: 3349600d80156d2c2df971fc0c61b8bd32ed18a1
 SHA512:
-  metadata.gz: 0b1ecc9e9ba6db3e24ff6aa564f8db187f5cd4ce3be4b03cd42f31f3d60824330081d4cf9dd8415ac6a5417aefabfba6476a946acb953d5211652ac086893c5d
-  data.tar.gz: c3c7f599fe5097d6361cab84141a569e0e00ae1038a2c2488da97b04a57b15713700d2ae07f9c9520357bfb7d065aca950fd12c6f70ae29669865951068eeba7
+  metadata.gz: e6f8d5c8fe8c84d6d18e3e15e135358e23238bf350e6e3f78471e30b1046500c18d8a97dc80e3b32262c7f0c6d8f5702bd3721b6bfbb24abd67243199ad7030b
+  data.tar.gz: 2c15f5ced824dc95a2d0e246422ba3118ce7f848b3170a980d3ace919afd27d1b8ea098a62a1d60221d0ec9a25b5459fc68532f5c003e554d9e04ba8c79bfb3a

data/NEWS.md

@@ -1,5 +1,15 @@
 ## News
 
+### 0.2.0 (2015-10-18)
+
+* Restrict users adding themselves to the queue
+  to hours defined by weekeday in the configuration.
+
+  A `config.schedule` hash must be present in your
+  config file, mapping `strftime("%a")` keys to hour ranges.
+
+* Relax routing requirements on instructor commands.
+
 ### 0.1.9 (2015-10-08)
 
 * Relax some of the routing requirements and fix the tests.

data/README.md

@@ -15,16 +15,21 @@ gem "lita-debug-queue"
 
 ## Configuration
 
-lita-debug-queue expects two things to be present for correct operation:
+lita-debug-queue expects three things to be present for correct operation:
 
 1. An `:instructors` authorization group containing admin users.
-  If you want TAs to also be able to modify (but not clear) the
-  queue you should add them to an `:assistants` group.
-2. A `classrooms` config option containing a Hash that maps instructor mention names to classroom channels.
+   If you want TAs to also be able to modify (but not clear) the
+   queue you should add them to an `:assistants` group.
 
-## Usage
+2. A `classrooms` config option containing a Hash that maps instructor mention nam   es to classroom channels.
+
+3. A `schedule` config option containing a Hash that maps from days of the week
+   (as from `DateTime.now.strftime("%a")`), to ranges of hours in the
+   [server's time zone][heroku-tz].
 
-All usage is based on commands so statements must be directed `@ironbot: foo` or sent as a DM.
+[heroku-tz]: http://blog.pardner.com/2012/08/setting-the-default-time-zone-for-a-heroku-app/
+
+## Usage
 
 ### General Commands
 
@@ -34,6 +39,7 @@ All usage is based on commands so statements must be directed `@ironbot: foo` or
 * `debug count` - Count the number of people waiting for help.
 
 ### Instructor Commands
+
 * `debug next` - Notify the next student to be helped.
 * `debug drop NAME` - Remove the student with NAME from the queue.
 * `debug clear` - Empty the queue.

data/lib/lita/handlers/debug_queue.rb

@@ -4,6 +4,7 @@ module Lita
       extend Lita::Handler::ChatRouter
 
       config :classrooms # A mapping from instructor names to classroom channels.
+      config :schedule   # A mapping from weekdays to available instructor hours.
 
       route(/debug me/, :add,
         help: { 'debug me' => 'Put your name in the queue for debugging help.' })
@@ -13,20 +14,22 @@ module Lita
         help: { 'debug queue' => 'Show the current queue for your class.' })
       route(/debug count/i, :count,
         help: { 'debug count' => 'Count the number of people waiting for help.' })
-      route(/^debug next$/, :next, command: true,
+      route(/debug next/, :next,
         restrict_to: [:instructors, :assistants],
         help: { 'debug next' => 'Notify the next student to be helped.' })
-      route(/^debug drop\s+(.+)$/, :drop, command: true,
+      route(/debug drop\s+(.+)/, :drop,
         restrict_to: [:instructors, :assistants],
         help: { 'debug drop NAME' => 'Remove the student with NAME from the queue.' })
-      route(/^debug clear$/, :clear, command: true,
+      route(/^debug clear$/, :clear,
         restrict_to: [:instructors],
         help: { 'debug clear' => 'Empty the queue.' })
 
       def add(response)
         return unless check_room!(response)
         student = response.user.mention_name
-        if @room.include?(student)
+        if closed?(DateTime.now)
+          response.reply("#{student}: Sorry, the queue is closed for the day.")
+        elsif @room.include?(student)
           response.reply("#{student}: Easy there killer. You're already on the list.")
         else
           @room.add(student)
@@ -95,6 +98,11 @@ module Lita
       def target_for(name)
         Lita::Source.new(user: Lita::User.find_by_mention_name(name))
       end
+
+      def closed?(now)
+        hours = config.schedule[now.strftime("%a")]
+        hours && !hours.include?(now.hour)
+      end
     end
 
     Lita.register_handler(DebugQueue)

data/lita-debug-queue.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-debug-queue"
-  spec.version       = "0.1.9"
+  spec.version       = "0.2.0"
   spec.authors       = ["Brit Butler"]
   spec.email         = ["brit@kingcons.io"]
   spec.description   = "Queue tracking of users who need debugging help with per-channel management"

data/spec/lita/handlers/help_queue_spec.rb

@@ -10,6 +10,7 @@ describe Lita::Handlers::DebugQueue, lita_handler: true do
     allow(Lita::Handlers::RoomFinder).to receive(:get_room_name) do |room|
       room.name
     end
+    allow_any_instance_of(Lita::Handlers::DebugQueue).to receive(:closed?) { false }
     registry.config.handlers.debug_queue.classrooms = {
       'brit' => 'rails'
     }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-debug-queue
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.2.0
 platform: ruby
 authors:
 - Brit Butler
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-09 00:00:00.000000000 Z
+date: 2015-10-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita