lita-envy 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1771ca4a7139bb295de5a3f34643749b612f5f69
-  data.tar.gz: 7d16778ffb99417dbdcea91d498bb95a2d9f94b7
+  metadata.gz: 7b6d149d6bff191a044dc53a59962548260a27d9
+  data.tar.gz: fcafab3b1480b4eace38dff345b45abb54d27cbd
 SHA512:
-  metadata.gz: 5be321f0ef3ad3df62d7c173164c0c8974c449a8a2eba2581fabccc78345b62138c192e23dfb26db9dded90dbe25ac8e68f2ffcf329173443c202d7fc7de1cc2
-  data.tar.gz: 089233c504f022a389b9d6702fca600327c04dc5d6e452b831db5856ff65511e408a7f64729434611b1d9cc490e75b45f6a752fd2b1f9e611af800fc5ae30ee5
+  metadata.gz: af80d2fc2484482c0ba3046454d9e8fbaf9c14470458af905c241f0909c5403386549c119f0a3c1f320448828cdce04370a47bc5106eccca665cdc4d75d7c0dd
+  data.tar.gz: bfb11685eb3467ac104099e8e8a05b93014d4858ecd60e64e5cdf7655536520ac11b76cf38bfd058b5098b6c4e4ffae2d32eb0de05d81bd5329d737ee2acf749

data/README.md

@@ -16,7 +16,10 @@ gem "lita-envy"
 ## Usage
 
 ``` bash
-@bot started using env ENV123
-@bot stopped using env ENV123
+@bot claim ENV123
+@bot release ENV123
+@bot forget ENV123
+@bot envs
+@bot wrestle ENV123 from Jon Doe
 ```
 

data/lib/lita/handlers/envy.rb

@@ -2,35 +2,35 @@ module Lita
   module Handlers
     class Envy < Handler
 
-      route /\Astarted using env ([A-Za-z0-9_]+)\Z/,  :start_using_environment, help:  { "started using env [ENV ID]"  => "Mark environment as in use by you"}, command: true
-      route /\Astopped using env ([A-Za-z0-9_]+)\Z/,  :stop_using_environment, help:  { "stopped using env [ENV ID]"  => "Mark environment as available"}, command: true
-      route /\Aenvironments\Z/,                       :list_environments, help:  { "environments"  => "List environments"}, command: true
-      route /\Aremove env ([A-Za-z0-9_]+)\Z/,         :remove_environment, help:  { "remove env [ENV ID]"  => "Remove environment"}, command: true
-      route /\Awrestle env ([A-Za-z0-9_]+) from (.*)\Z/, :wrestle_environment_from_user, help:  { "wrestle env [ENV ID] from [USER]"  => "Mark environment as in use by you, even though it is currently in use by another user"}, command: true
+      route /\Aclaim ([A-Za-z0-9_]+)\Z/,             :claim_environment, help:  { "claim [ENV ID]"  => "Mark environment as in use by you"}, command: true
+      route /\Arelease ([A-Za-z0-9_]+)\Z/,           :release_environment, help:  { "release [ENV ID]"  => "Mark environment as available"}, command: true
+      route /\Aenvs\Z/,                              :list_environments, help:  { "envs"  => "List environments"}, command: true
+      route /\Aforget ([A-Za-z0-9_]+)\Z/,            :forget_environment, help:  { "forget [ENV ID]"  => "Forget environment"}, command: true
+      route /\Awrestle ([A-Za-z0-9_]+) from (.*)\Z/, :claim_used_environment, help:  { "wrestle [ENV ID] from [USER]"  => "Mark environment as in use by you, even though it is currently in use by another user"}, command: true
 
-      def start_using_environment(response)
+      def claim_environment(response)
         env_id = response.matches.first.first
         current_user = redis.hget(['environments', env_id].join(':'), 'user')
         if current_user.nil? || current_user.empty?
           redis.hset(['environments', env_id].join(':'), 'user', response.user.name)
           response.reply('ok')
         elsif current_user == response.user.name
-          response.reply("You are already using #{env_id}")
+          response.reply("Hmm, you are already using #{env_id}")
         else
-          response.reply("Sorry, #{env_id} is currently in use by #{current_user}")
+          response.reply("Hmm, #{env_id} is currently in use by #{current_user}")
         end
       end
 
-      def stop_using_environment(response)
+      def release_environment(response)
         env_id = response.matches.first.first
         current_user = redis.hget(['environments', env_id].join(':'), 'user')
         if current_user == response.user.name
           redis.hset(['environments', env_id].join(':'), 'user', nil)
           response.reply('ok')
         elsif current_user.nil? || current_user.empty?
-          response.reply("You are not currently using #{env_id}")
+          response.reply("Hmm, you are not currently using #{env_id}")
         else
-          response.reply("You are not currently using #{env_id} (#{current_user} is)")
+          response.reply("Hmm, you are not currently using #{env_id} (#{current_user} is)")
         end
 
       end
@@ -47,33 +47,33 @@ module Lita
         response.reply(lines.join("\n"))
       end
 
-      def remove_environment(response)
+      def forget_environment(response)
         env_id = response.matches.first.first
         current_user = redis.hget(['environments', env_id].join(':'), 'user')
         if current_user == response.user.name
-          response.reply("You are currently using #{env_id}")
+          response.reply("Hmm, you are currently using #{env_id}")
         elsif current_user.nil?
-          response.reply("I do not know about environment #{env_id}")
+          response.reply("Hmm, I do not know about #{env_id}")
         elsif current_user.empty?
           redis.del(['environments', env_id].join(':'))
           response.reply('ok')
         else
-          response.reply("Sorry, #{env_id} is currently in use by #{current_user}")
+          response.reply("Hmm, #{env_id} is currently in use by #{current_user}")
         end
       end
 
-      def wrestle_environment_from_user(response)
+      def claim_used_environment(response)
         env_id, specified_user = response.matches.first
         current_user = redis.hget(['environments', env_id].join(':'), 'user')
         if specified_user == current_user
           redis.hset(['environments', env_id].join(':'), 'user', response.user.name)
           response.reply('ok')
         elsif current_user.nil? or current_user.empty?
-          response.reply("Sorry, #{env_id} is not currently in use")
+          response.reply("Hmm, #{env_id} is not currently in use")
         elsif current_user == response.user.name
-          response.reply("You are already using #{env_id}")
+          response.reply("Hmm, you are already using #{env_id}")
         else
-          response.reply("Sorry, #{env_id} is currently in use by #{current_user}, not #{specified_user}")
+          response.reply("Hmm, #{env_id} is currently in use by #{current_user}, not #{specified_user}")
         end
       end
 

data/lita-envy.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-envy"
-  spec.version       = "0.1.2"
+  spec.version       = "0.1.3"
   spec.authors       = ["Ingo Weiss"]
   spec.email         = ["ingo.weiss@lab49.com"]
   spec.description   = "Mark environments as in use. Mark environments as available. List environments"

data/spec/lita/handlers/envy_spec.rb

@@ -4,11 +4,11 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
   describe 'Routing' do
 
-    it { is_expected.to route_command("started using env ENV123").to(:start_using_environment)  }
-    it { is_expected.to route_command("stopped using env ENV123").to(:stop_using_environment)  }
-    it { is_expected.to route_command("environments").to(:list_environments)  }
-    it { is_expected.to route_command("remove env ENV123").to(:remove_environment)  }
-    it { is_expected.to route_command("wrestle env ENV123 from Alicia").to(:wrestle_environment_from_user)  }
+    it { is_expected.to route_command("claim ENV123").to(:claim_environment)  }
+    it { is_expected.to route_command("release ENV123").to(:release_environment)  }
+    it { is_expected.to route_command("envs").to(:list_environments)  }
+    it { is_expected.to route_command("forget ENV123").to(:forget_environment)  }
+    it { is_expected.to route_command("wrestle ENV123 from Alicia").to(:claim_used_environment)  }
 
   end
 
@@ -22,12 +22,12 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should mark environment as in use by user" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('started using env ENV123', :as => carl)
+        send_command('claim ENV123', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with confirmation" do
-        send_command('started using env ENV123')
+        send_command('claim ENV123')
         expect(replies.first).to eq("ok")
       end
 
@@ -41,12 +41,12 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should mark environment as in use by user" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('started using env ENV123', :as => carl)
+        send_command('claim ENV123', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with confirmation" do
-        send_command('started using env ENV123')
+        send_command('claim ENV123')
         expect(replies.first).to eq("ok")
       end
 
@@ -60,15 +60,15 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('started using env ENV123', :as => carl)
+        send_command('claim ENV123', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Alicia")
       end
 
       it "should reply with notification" do
         subject.redis.hset('environments:ENV123', 'user', 'Alicia')
         carl = Lita::User.create(123, name: "Carl")
-        send_command('started using env ENV123', :as => carl)
-        expect(replies.first).to eq("Sorry, ENV123 is currently in use by Alicia")
+        send_command('claim ENV123', :as => carl)
+        expect(replies.first).to eq("Hmm, ENV123 is currently in use by Alicia")
       end
 
     end
@@ -81,15 +81,15 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('started using env ENV123', :as => carl)
+        send_command('claim ENV123', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with notification" do
         subject.redis.hset('environments:ENV123', 'user', 'Carl')
         carl = Lita::User.create(123, name: "Carl")
-        send_command('started using env ENV123', :as => carl)
-        expect(replies.first).to eq("You are already using ENV123")
+        send_command('claim ENV123', :as => carl)
+        expect(replies.first).to eq("Hmm, you are already using ENV123")
       end
 
     end
@@ -106,13 +106,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should mark environment as available" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
+        send_command('release ENV234', :as => alicia)
         expect(subject.redis.hget('environments:ENV234', 'user')).to be_empty
       end
 
       it "should reply with confirmation" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
+        send_command('release ENV234', :as => alicia)
         expect(replies.first).to eq("ok")
       end
 
@@ -126,14 +126,14 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
+        send_command('release ENV234', :as => alicia)
         expect(subject.redis.hget('environments:ENV234', 'user')).to eq('Carl')
       end
 
       it "should reply with notification" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
-        expect(replies.first).to eq("You are not currently using ENV234 (Carl is)")
+        send_command('release ENV234', :as => alicia)
+        expect(replies.first).to eq("Hmm, you are not currently using ENV234 (Carl is)")
       end
 
     end
@@ -146,14 +146,14 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
+        send_command('release ENV234', :as => alicia)
         expect(subject.redis.hget('environments:ENV234', 'user')).to eq('')
       end
 
       it "should reply with notification" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
-        expect(replies.first).to eq("You are not currently using ENV234")
+        send_command('release ENV234', :as => alicia)
+        expect(replies.first).to eq("Hmm, you are not currently using ENV234")
       end
 
     end
@@ -166,14 +166,14 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
+        send_command('release ENV234', :as => alicia)
         expect(subject.redis.hget('environments:ENV234', 'user')).to be_nil
       end
 
       it "should reply with notification" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('stopped using env ENV234', :as => alicia)
-        expect(replies.first).to eq("You are not currently using ENV234")
+        send_command('release ENV234', :as => alicia)
+        expect(replies.first).to eq("Hmm, you are not currently using ENV234")
       end
 
     end
@@ -186,7 +186,7 @@ describe Lita::Handlers::Envy, lita_handler: true do
       subject.redis.hset('environments:ENV123', 'user', 'Alicia')
       subject.redis.hset('environments:ENV234', 'user', 'Carl')
       subject.redis.hset('environments:ENV345', 'user', '')
-      send_command('environments')
+      send_command('envs')
       expect(replies.first.split("\n")).to eq([
         "ENV123 (Alicia)",
         "ENV234 (Carl)",
@@ -204,13 +204,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
         subject.redis.hset('environments:ENV345', 'user', '')
       end
 
-      it "should remove environments" do
-        send_command('remove env ENV345')
+      it "should forgetironments" do
+        send_command('forget ENV345')
         expect(subject.redis.keys).to_not include('environments:ENV345')
       end
 
       it "should confirm" do
-        send_command('remove env ENV345')
+        send_command('forget ENV345')
         expect(replies.first).to eq("ok")
       end
 
@@ -223,8 +223,8 @@ describe Lita::Handlers::Envy, lita_handler: true do
       end
 
       it "should reply with notification" do
-        send_command('remove env ENV345')
-        expect(replies.first).to eq("I do not know about environment ENV345")
+        send_command('forget ENV345')
+        expect(replies.first).to eq("Hmm, I do not know about ENV345")
       end
 
     end
@@ -237,13 +237,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('remove env ENV345', :as => alicia)
+        send_command('forget ENV345', :as => alicia)
         expect(subject.redis.hget('environments:ENV345', 'user')).to eq('Carl')
       end
 
       it "should reply with notification" do
-        send_command('remove env ENV345')
-        expect(replies.first).to eq("Sorry, ENV345 is currently in use by Carl")
+        send_command('forget ENV345')
+        expect(replies.first).to eq("Hmm, ENV345 is currently in use by Carl")
       end
 
     end
@@ -256,14 +256,14 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('remove env ENV345', :as => alicia)
+        send_command('forget ENV345', :as => alicia)
         expect(subject.redis.hget('environments:ENV345', 'user')).to eq('Alicia')
       end
 
       it "should reply with notification" do
         alicia = Lita::User.create(123, name: "Alicia")
-        send_command('remove env ENV345', :as => alicia)
-        expect(replies.first).to eq("You are currently using ENV345")
+        send_command('forget ENV345', :as => alicia)
+        expect(replies.first).to eq("Hmm, you are currently using ENV345")
       end
 
     end
@@ -280,12 +280,12 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should mark environment as in use" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('wrestle env ENV123 from Alicia', :as => carl)
+        send_command('wrestle ENV123 from Alicia', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with confirmation" do
-        send_command('wrestle env ENV123 from Alicia')
+        send_command('wrestle ENV123 from Alicia')
         expect(replies.first).to eq("ok")
       end
 
@@ -299,13 +299,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('wrestle env ENV123 from Ben', :as => carl)
+        send_command('wrestle ENV123 from Ben', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Alicia")
       end
 
       it "should reply with notification" do
-        send_command('wrestle env ENV123 from Ben')
-        expect(replies.first).to eq("Sorry, ENV123 is currently in use by Alicia, not Ben")
+        send_command('wrestle ENV123 from Ben')
+        expect(replies.first).to eq("Hmm, ENV123 is currently in use by Alicia, not Ben")
       end
 
     end
@@ -318,13 +318,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('wrestle env ENV123 from Ben', :as => carl)
+        send_command('wrestle ENV123 from Ben', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to be_empty
       end
 
       it "should reply with notification" do
-        send_command('wrestle env ENV123 from Ben')
-        expect(replies.first).to eq("Sorry, ENV123 is not currently in use")
+        send_command('wrestle ENV123 from Ben')
+        expect(replies.first).to eq("Hmm, ENV123 is not currently in use")
       end
 
     end
@@ -337,14 +337,14 @@ describe Lita::Handlers::Envy, lita_handler: true do
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('wrestle env ENV123 from Ben', :as => carl)
+        send_command('wrestle ENV123 from Ben', :as => carl)
         expect(subject.redis.hget('environments:ENV123', 'user')).to eq('Carl')
       end
 
       it "should reply with notification" do
         carl = Lita::User.create(123, name: "Carl")
-        send_command('wrestle env ENV123 from Ben', :as => carl)
-        expect(replies.first).to eq("You are already using ENV123")
+        send_command('wrestle ENV123 from Ben', :as => carl)
+        expect(replies.first).to eq("Hmm, you are already using ENV123")
       end
 
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-envy
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Ingo Weiss