lita-queue 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fb902787e39a653ac07718c6d84dc1cf33f9b066
-  data.tar.gz: 523a6d6ec43a1d645f24e872089ac61a40b11349
+  metadata.gz: 1c68bea41fdca6b89d3d91c689a4a2f270204a01
+  data.tar.gz: a46a13c18c6624f7190d9151341b0d95054b00b6
 SHA512:
-  metadata.gz: 2ff331250ad6460c32ab3808c1eda94408dfb9b73a99743f39191bf11e2566dd9d40983bf51820ad67ade07472bbc780814f454b0c893b79a5de448528613bf6
-  data.tar.gz: 3563b154f31d16ef18329f86d97585358e046b2f92865b118e6babfe81566208fb41858cdf46431b8ecd577fa6986b817d9d57f28b0908b3f6e9a9f8e67c066b
+  metadata.gz: 82cdd8f102c7d4fa791484b7a0cc8efca7686d3d8498cf1be886ba003d96d6db41ea13c2b45b985fe4039cb062b8b86342856442f746a27baa342164eece9b3f
+  data.tar.gz: 3e84ef189225c3f501f1f95287b38e88f24b921395a5aa2f67cd2fabdae4def1047af8387fb35bbcde3bbfb983de6ba9509c32f7a52a9fc312fdc16e403fd0cd

data/CHANGELOG.md

@@ -4,6 +4,13 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 We follow [Keep a Changelog](http://keepachangelog.com/) format.
 
+## 0.3.0 - 2015-07-19
+### Modified
+- Correctly using Lita::Room object
+  * create queues based on room ID
+  * display queue using room name metadata
+  * modified specs to catch this change
+
 ## 0.2.0 - 2015-06-21
 ### Added
 - Respond to the following additional command:

data/lib/lita/handlers/queue.rb

@@ -13,17 +13,19 @@ module Lita
       # API
 
       def fetch_queue(room)
-        serialized = redis.get(room)
+        raise ArgumentError, 'must be a Lita::Room object' unless room.is_a? Lita::Room
+
+        serialized = redis.get(room.id)
 
         if serialized
-          JSON.parse(serialized)
+          MultiJson.load(serialized)
         else
           []
         end
       end
 
       def store_queue(room, queue)
-        redis.set room, queue.to_json
+        redis.set room.id, MultiJson.dump(queue)
       end
 
       # Commands
@@ -107,14 +109,16 @@ module Lita
       private
 
       def room_for(response)
-        response.message.source.room || '--global--'
+        response.message.source.room_object
       end
 
       def display_queue(queue, room)
+        log.debug "displaying info for queue: #{queue.inspect} at #{room.inspect}"
+
         if queue.empty?
           "Queue is empty!"
         else
-          "Queue for #{room}: #{queue}"
+          "Queue for #{room.name}: #{queue}"
         end
       end
     end

data/lita-queue.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-queue"
-  spec.version       = "0.2.0"
+  spec.version       = "0.3.0"
   spec.authors       = ["Gabriel Mazetto"]
   spec.email         = ["brodock@gmail.com"]
   spec.description   = "Lita plugin to manage channel specific queue"
@@ -14,7 +14,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_runtime_dependency "lita", ">= 4.3"
+  spec.add_runtime_dependency "lita", ">= 4.4.3"
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "pry-byebug"

data/spec/lita/handlers/queue_spec.rb

@@ -15,24 +15,24 @@ describe Lita::Handlers::Queue, lita_handler: true do
   it { is_expected.to route_command("queue rotate!").to(:queue_rotate) }
   #it { is_expected.to route_command("queue = [something,here]").to(:queue_recreate) }
 
-  let(:channel) { source.room || '--global--' }
+  let(:room) { Lita::Room.new(1, {name: 'my-channel'}) }
 
   # Commands
   describe "#queue_list" do
     context "when queue is empty" do
-      before { subject.store_queue(channel, []) }
+      before { subject.store_queue(room, []) }
 
       it "replies with an empty queue message" do
-        send_command("queue")
+        send_command("queue", from: room)
         expect(replies.last).to include("Queue is empty")
       end
     end
 
     context "when queue has elements" do
-      before { subject.store_queue(channel, [user1.mention_name, user2.mention_name]) }
+      before { subject.store_queue(room, [user1.mention_name, user2.mention_name]) }
 
       it "replies with a list of queue users" do
-        send_command("queue")
+        send_command("queue", from: room)
         expect(replies.last).to include(user1.mention_name, user2.mention_name)
       end
     end
@@ -40,39 +40,39 @@ describe Lita::Handlers::Queue, lita_handler: true do
 
   describe "#queue_me" do
     context "when I'm already queued" do
-      before { subject.store_queue(channel, [user.mention_name]) }
+      before { subject.store_queue(room, [user.mention_name]) }
 
       it "replies with an error message" do
-        send_command("queue me")
+        send_command("queue me", from: room)
         expect(replies.last).to include("already on queue")
       end
     end
 
     context "when I'm not on queue" do
       it "replies with a confirmation message" do
-        send_command("queue me")
+        send_command("queue me", from: room)
         expect(replies.last).to include("#{user.name} have been added to queue")
-        expect(subject.fetch_queue(channel)).to include(user.mention_name)
+        expect(subject.fetch_queue(room)).to include(user.mention_name)
       end
     end
   end
 
   describe "#unqueue_me" do
     context "when I'm already queued" do
-      before { subject.store_queue(channel, [user.mention_name]) }
+      before { subject.store_queue(room, [user.mention_name]) }
 
       it "replies with a confirmation and remove from queue" do
-        send_command("unqueue me")
+        send_command("unqueue me", from: room)
         expect(replies.last).to include("#{user.name} have been removed from queue")
-        expect(subject.fetch_queue(channel)).not_to include(user.mention_name)
+        expect(subject.fetch_queue(room)).not_to include(user.mention_name)
       end
     end
 
     context "when I'm not on queue" do
-      before { subject.store_queue(channel, []) }
+      before { subject.store_queue(room, []) }
 
       it "replies with an error message" do
-        send_command("unqueue me")
+        send_command("unqueue me", from: room)
         expect(replies.last).to include("not on queue!")
       end
     end
@@ -81,26 +81,26 @@ describe Lita::Handlers::Queue, lita_handler: true do
   describe "#queue_list_next" do
     context "when queue is empty" do
       it "replies with an error message" do
-        send_command("queue next?")
+        send_command("queue next?", from: room)
         expect(replies.last).to include("Queue is empty")
       end
     end
 
     context "when queue has only one element" do
-      before { subject.store_queue(channel, [user1.mention_name]) }
+      before { subject.store_queue(room, [user1.mention_name]) }
 
       it "replies listing current user on queue and warning that's the last one" do
-        send_command("queue next?")
+        send_command("queue next?", from: room)
         expect(replies.last).to include(user1.mention_name)
         expect(replies.last).to include("is the last one on queue")
       end
     end
 
     context "when queue has more than one elements" do
-      before { subject.store_queue(channel, [user1.mention_name, user2.mention_name]) }
+      before { subject.store_queue(room, [user1.mention_name, user2.mention_name]) }
 
       it "replies listing the next one on the queue" do
-        send_command("queue next?")
+        send_command("queue next?", from: room)
         expect(replies.last).to include(user2.mention_name)
       end
     end
@@ -109,29 +109,29 @@ describe Lita::Handlers::Queue, lita_handler: true do
   describe "#queue_change_to_next" do
     context "when queue is empty" do
       it "replies with an error message" do
-        send_command("queue next!")
+        send_command("queue next!", from: room)
         expect(replies.last).to include("Queue is empty")
       end
     end
 
     context "when queue has enough elements" do
       let(:queue) { [user1.mention_name, user2.mention_name] }
-      before { subject.store_queue(channel, queue) }
+      before { subject.store_queue(room, queue) }
 
       it "remove the first element from the queue" do
-        send_command("queue next!")
-        expect(subject.fetch_queue(channel)).to eq([user2.mention_name])
+        send_command("queue next!", from: room)
+        expect(subject.fetch_queue(room)).to eq([user2.mention_name])
       end
 
       it "replies informing who was been removed and who is next" do
-        send_command("queue next!")
+        send_command("queue next!", from: room)
         expect(replies.first).to include("#{user1.mention_name} have been removed from queue")
         expect(replies[1]).to include("#{user2.mention_name} is the next")
 
       end
 
       it "informs the new queue after execution" do
-        send_command("queue next!")
+        send_command("queue next!", from: room)
         expect(replies.last).to include(user2.mention_name)
         expect(replies.last).not_to include(user1.mention_name)
       end
@@ -140,7 +140,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
         let(:queue) { [user2.mention_name] }
 
         it "replies with a notification message when removing the last element from queue" do
-          send_command("queue next!")
+          send_command("queue next!", from: room)
           expect(replies.first).to include("#{user2.mention_name} have been removed from queue")
           expect(replies.last).to include("Queue is empty")
         end
@@ -150,15 +150,15 @@ describe Lita::Handlers::Queue, lita_handler: true do
 
   describe "#queue_rotate" do
     context "when queue has enough elements" do
-      before { subject.store_queue(channel, [user1.mention_name, user2.mention_name, user3.mention_name]) }
+      before { subject.store_queue(room, [user1.mention_name, user2.mention_name, user3.mention_name]) }
 
       it "removes the first element and add to the end" do
-        send_command("queue rotate!")
-        expect(subject.fetch_queue(channel)).to eq([user2.mention_name, user3.mention_name, user1.mention_name])
+        send_command("queue rotate!", from: room)
+        expect(subject.fetch_queue(room)).to eq([user2.mention_name, user3.mention_name, user1.mention_name])
       end
 
       it "replies mentioning the next user on queue and notifing the rotated one" do
-        send_command("queue rotate!")
+        send_command("queue rotate!", from: room)
         expect(replies.first).to include("#{user1.mention_name} has been moved to the end of the queue")
         expect(replies[1]).to include("#{user2.mention_name} is the next")
       end

data/spec/spec_helper.rb

@@ -1,3 +1,4 @@
+require "pry"
 require "simplecov"
 require "coveralls"
 SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
@@ -12,3 +13,31 @@ require "lita/rspec"
 # A compatibility mode is provided for older plugins upgrading from Lita 3. Since this plugin
 # was generated with Lita 4, the compatibility mode should be left disabled.
 Lita.version_3_compatibility_mode = false
+
+# Backport / Monkey Patch Lita Spec Handler Helper until a new version > 4.4.3 is released.
+# More info here: https://github.com/jimmycuadra/lita/pull/129
+module Lita
+  module RSpec
+    module Handler
+      # Sends a message to the robot.
+      # @param body [String] The message to send.
+      # @param as [Lita::User] The user sending the message.
+      # @param as [Lita::Room] The room where the message is received from.
+      # @return [void]
+      def send_message(body, as: user, from: nil)
+        message = Message.new(robot, body, Source.new(user: as, room: from))
+
+        robot.receive(message)
+      end
+
+      # Sends a "command" message to the robot.
+      # @param body [String] The message to send.
+      # @param as [Lita::User] The user sending the message.
+      # @param as [Lita::Room] The room where the message is received from.
+      # @return [void]
+      def send_command(body, as: user, from: nil)
+        send_message("#{robot.mention_name}: #{body}", as: as, from: from)
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-queue
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Gabriel Mazetto
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-21 00:00:00.000000000 Z
+date: 2015-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.3'
+        version: 4.4.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.3'
+        version: 4.4.3
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -165,7 +165,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.6
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Lita plugin to manage channel specific queue