lita-queue 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 33fd47f0ec810e138b6c4ad01136929edb6c54ac
-  data.tar.gz: bbff179f4261205af069b78e88ddbc7fd946ab87
+  metadata.gz: fb902787e39a653ac07718c6d84dc1cf33f9b066
+  data.tar.gz: 523a6d6ec43a1d645f24e872089ac61a40b11349
 SHA512:
-  metadata.gz: a4dd70f59ae79592e5d0a03b2d1c8b8c514493f16952ea7ceb5ee38a2399d4550208ba492758ef3b2ca21d24973883da63747b8b52e67814a66d12a13f216180
-  data.tar.gz: 6a21b6e05bceb50fb4bf3f6fc675a6337dc7f10a5814e59f73aac45ea7574ead51b76268bd60d42d8150573b71d8b5ebb5b0a9c54402668afc8ab4389704fa5b
+  metadata.gz: 2ff331250ad6460c32ab3808c1eda94408dfb9b73a99743f39191bf11e2566dd9d40983bf51820ad67ade07472bbc780814f454b0c893b79a5de448528613bf6
+  data.tar.gz: 3563b154f31d16ef18329f86d97585358e046b2f92865b118e6babfe81566208fb41858cdf46431b8ecd577fa6986b817d9d57f28b0908b3f6e9a9f8e67c066b

data/.rspec

@@ -0,0 +1 @@
+--color --profile 5

data/CHANGELOG.md

@@ -4,6 +4,14 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 We follow [Keep a Changelog](http://keepachangelog.com/) format.
 
+## 0.2.0 - 2015-06-21
+### Added
+- Respond to the following additional command:
+  * lita rotate!
+
+### Changed
+- More granular specs and better RSpec defaults (.rspec)
+
 ## 0.1.0 - 2015-06-14
 ### Added
 - Initial plugin import with support for the following commands:

data/README.md

@@ -23,7 +23,7 @@ The following commands are available:
 * lita unqueue me
 * lita queue next?
 * lita queue next!
+* lita queue rotate!
 
 The following commands will be available in a further version:
-* lita queue rotate!
 * lita queue = [<new_queue,comma_separated>]

data/lib/lita/handlers/queue.rb

@@ -7,7 +7,7 @@ module Lita
       route(/^unqueue me$/, :unqueue_me, command: :true)
       route(/^queue next\?$/, :queue_list_next, command: :true)
       route(/^queue next!$/, :queue_change_to_next, command: :true)
-      #route(/^queue rotate!$/, :queue_rotate, command: :true)
+      route(/^queue rotate!$/, :queue_rotate, command: :true)
       #route(/^queue = \[([^\]]*)\]\s*$$/, :queue_recreate, command: :true)
 
       # API
@@ -83,13 +83,27 @@ module Lita
         unless queue.empty?
           removed = queue.shift
           store_queue(room, queue)
-          response.reply "#{removed} have been removed from queue"
+          response.reply "#{removed} have been removed from queue."
           response.reply "#{queue.first} is the next. Go ahead!" unless queue.empty?
         end
 
         response.reply display_queue(queue, room)
       end
 
+      def queue_rotate(response)
+        room = room_for(response)
+        queue = fetch_queue(room)
+
+        unless queue.empty?
+          new_queue = queue.rotate
+          store_queue(room, new_queue)
+          response.reply "#{queue.first} has been moved to the end of the queue."
+          response.reply "#{new_queue.first} is the next. Go ahead!"
+        end
+
+        response.reply display_queue(queue, room)
+      end
+
       private
 
       def room_for(response)
@@ -98,7 +112,7 @@ module Lita
 
       def display_queue(queue, room)
         if queue.empty?
-          "Queue is empty"
+          "Queue is empty!"
         else
           "Queue for #{room}: #{queue}"
         end

data/lita-queue.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-queue"
-  spec.version       = "0.1.0"
+  spec.version       = "0.2.0"
   spec.authors       = ["Gabriel Mazetto"]
   spec.email         = ["brodock@gmail.com"]
   spec.description   = "Lita plugin to manage channel specific queue"

data/spec/lita/handlers/queue_spec.rb

@@ -4,6 +4,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
 
   user1 = Lita::User.create(100, name: 'User 1')
   user2 = Lita::User.create(101, name: 'User 2')
+  user3 = Lita::User.create(102, name: 'User 3')
 
   # Routes
   it { is_expected.to route_command("queue").to(:queue_list) }
@@ -11,7 +12,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
   it { is_expected.to route_command("unqueue me").to(:unqueue_me) }
   it { is_expected.to route_command("queue next?").to(:queue_list_next) }
   it { is_expected.to route_command("queue next!").to(:queue_change_to_next) }
-  #it { is_expected.to route_command("queue rotate!").to(:queue_rotate) }
+  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--' }
@@ -20,6 +21,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
   describe "#queue_list" do
     context "when queue is empty" do
       before { subject.store_queue(channel, []) }
+
       it "replies with an empty queue message" do
         send_command("queue")
         expect(replies.last).to include("Queue is empty")
@@ -39,6 +41,7 @@ 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]) }
+
       it "replies with an error message" do
         send_command("queue me")
         expect(replies.last).to include("already on queue")
@@ -57,6 +60,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
   describe "#unqueue_me" do
     context "when I'm already queued" do
       before { subject.store_queue(channel, [user.mention_name]) }
+
       it "replies with a confirmation and remove from queue" do
         send_command("unqueue me")
         expect(replies.last).to include("#{user.name} have been removed from queue")
@@ -66,6 +70,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
 
     context "when I'm not on queue" do
       before { subject.store_queue(channel, []) }
+
       it "replies with an error message" do
         send_command("unqueue me")
         expect(replies.last).to include("not on queue!")
@@ -83,6 +88,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
 
     context "when queue has only one element" do
       before { subject.store_queue(channel, [user1.mention_name]) }
+
       it "replies listing current user on queue and warning that's the last one" do
         send_command("queue next?")
         expect(replies.last).to include(user1.mention_name)
@@ -92,6 +98,7 @@ describe Lita::Handlers::Queue, lita_handler: true do
 
     context "when queue has more than one elements" do
       before { subject.store_queue(channel, [user1.mention_name, user2.mention_name]) }
+
       it "replies listing the next one on the queue" do
         send_command("queue next?")
         expect(replies.last).to include(user2.mention_name)
@@ -107,23 +114,53 @@ describe Lita::Handlers::Queue, lita_handler: true do
       end
     end
 
-    context "when queue has elements" do
-      it "remove the first element and replies with current queue" do
-        subject.store_queue(channel, [user1.mention_name, user2.mention_name])
+    context "when queue has enough elements" do
+      let(:queue) { [user1.mention_name, user2.mention_name] }
+      before { subject.store_queue(channel, queue) }
+
+      it "remove the first element from the queue" do
+        send_command("queue next!")
+        expect(subject.fetch_queue(channel)).to eq([user2.mention_name])
+      end
 
+      it "replies informing who was been removed and who is next" do
         send_command("queue next!")
         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!")
         expect(replies.last).to include(user2.mention_name)
         expect(replies.last).not_to include(user1.mention_name)
       end
 
-      it "replies with a notification message when removing the last element from queue" do
-        subject.store_queue(channel, [user2.mention_name])
+      context "when queue has only one element" do
+        let(:queue) { [user2.mention_name] }
 
-        send_command("queue next!")
-        expect(replies.first).to include("#{user2.mention_name} have been removed from queue")
-        expect(replies.last).to include("Queue is empty")
+        it "replies with a notification message when removing the last element from queue" do
+          send_command("queue next!")
+          expect(replies.first).to include("#{user2.mention_name} have been removed from queue")
+          expect(replies.last).to include("Queue is empty")
+        end
+      end
+    end
+  end
+
+  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]) }
+
+      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])
+      end
+
+      it "replies mentioning the next user on queue and notifing the rotated one" do
+        send_command("queue rotate!")
+        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
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-queue
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Gabriel Mazetto
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-15 00:00:00.000000000 Z
+date: 2015-06-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -130,6 +130,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".rspec"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile