lita-envy 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7b6d149d6bff191a044dc53a59962548260a27d9
-  data.tar.gz: fcafab3b1480b4eace38dff345b45abb54d27cbd
+  metadata.gz: 25e7123d1dcedb17ef798f0a8de34d821f1222c8
+  data.tar.gz: 634e009f649a0f27359f0bb14a2ed470c9883d30
 SHA512:
-  metadata.gz: af80d2fc2484482c0ba3046454d9e8fbaf9c14470458af905c241f0909c5403386549c119f0a3c1f320448828cdce04370a47bc5106eccca665cdc4d75d7c0dd
-  data.tar.gz: bfb11685eb3467ac104099e8e8a05b93014d4858ecd60e64e5cdf7655536520ac11b76cf38bfd058b5098b6c4e4ffae2d32eb0de05d81bd5329d737ee2acf749
+  metadata.gz: 1ff8c2852f565d0bd49d7d86c0fb4932e1de81e7c1a26020915529fecb495a2e9a92e5b808a91b2bdcf61944fa9874b306870e0885d5d0703f13b6fe85a0b803
+  data.tar.gz: a60fd300627ebb64594baa252deceed6f649f05ba0e6fe18bf70fe251fb8b63bae85f7d019882dfbfb705df06780bf971a789cb06b3dbe225d6ae4c006df1502

data/README.md

@@ -13,6 +13,12 @@ Add lita-envy to your Lita instance's Gemfile:
 gem "lita-envy"
 ```
 
+## Configuration
+
+``` ruby
+config.handlers.envy.namespace = 'my_project'
+```
+
 ## Usage
 
 ``` bash

data/lib/lita/handlers/envy.rb

@@ -8,11 +8,17 @@ module Lita
       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
 
+      config :namespace, :required => true do
+        validate do |value|
+          "can only contain lowercase letters, numbers and underscores" unless value.match(/\A[a-z0-9_]+\Z/)
+        end
+      end
+
       def claim_environment(response)
         env_id = response.matches.first.first
-        current_user = redis.hget(['environments', env_id].join(':'), 'user')
+        current_user = redis.hget(key(env_id), 'user')
         if current_user.nil? || current_user.empty?
-          redis.hset(['environments', env_id].join(':'), 'user', response.user.name)
+          redis.hset(key(env_id), 'user', response.user.name)
           response.reply('ok')
         elsif current_user == response.user.name
           response.reply("Hmm, you are already using #{env_id}")
@@ -23,9 +29,9 @@ module Lita
 
       def release_environment(response)
         env_id = response.matches.first.first
-        current_user = redis.hget(['environments', env_id].join(':'), 'user')
+        current_user = redis.hget(key(env_id), 'user')
         if current_user == response.user.name
-          redis.hset(['environments', env_id].join(':'), 'user', nil)
+          redis.hset(key(env_id), 'user', nil)
           response.reply('ok')
         elsif current_user.nil? || current_user.empty?
           response.reply("Hmm, you are not currently using #{env_id}")
@@ -49,13 +55,13 @@ module Lita
 
       def forget_environment(response)
         env_id = response.matches.first.first
-        current_user = redis.hget(['environments', env_id].join(':'), 'user')
+        current_user = redis.hget(key(env_id), 'user')
         if current_user == response.user.name
           response.reply("Hmm, you are currently using #{env_id}")
         elsif current_user.nil?
           response.reply("Hmm, I do not know about #{env_id}")
         elsif current_user.empty?
-          redis.del(['environments', env_id].join(':'))
+          redis.del(key(env_id))
           response.reply('ok')
         else
           response.reply("Hmm, #{env_id} is currently in use by #{current_user}")
@@ -64,9 +70,9 @@ module Lita
 
       def claim_used_environment(response)
         env_id, specified_user = response.matches.first
-        current_user = redis.hget(['environments', env_id].join(':'), 'user')
+        current_user = redis.hget(key(env_id), 'user')
         if specified_user == current_user
-          redis.hset(['environments', env_id].join(':'), 'user', response.user.name)
+          redis.hset(key(env_id), 'user', response.user.name)
           response.reply('ok')
         elsif current_user.nil? or current_user.empty?
           response.reply("Hmm, #{env_id} is not currently in use")
@@ -77,6 +83,12 @@ module Lita
         end
       end
 
+      private
+
+      def key(env_id)
+        ['environments', config.namespace, env_id].join(':')
+      end
+
       Lita.register_handler(self)
     end
   end

data/lita-envy.gemspec

@@ -1,9 +1,9 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-envy"
-  spec.version       = "0.1.3"
+  spec.version       = "0.1.4"
   spec.authors       = ["Ingo Weiss"]
-  spec.email         = ["ingo.weiss@lab49.com"]
-  spec.description   = "Mark environments as in use. Mark environments as available. List environments"
+  spec.email         = ["ingo@ingoweiss.com"]
+  spec.description   = "Helps with management of environment ussage. Adds commands for claiming and releasing environments"
   spec.summary       = "Record and retrieve information about environment usage"
   spec.homepage      = "https://github.com/ingoweiss/lita-envy"
   spec.license       = "MIT"

data/spec/lita/handlers/envy_spec.rb

@@ -2,6 +2,10 @@ require "spec_helper"
 
 describe Lita::Handlers::Envy, lita_handler: true do
 
+  before(:each) do
+    allow(subject.config).to receive(:namespace).and_return('my_project')
+  end
+
   describe 'Routing' do
 
     it { is_expected.to route_command("claim ENV123").to(:claim_environment)  }
@@ -17,13 +21,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is available" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV123', 'user', '')
+        subject.redis.hset('environments:my_project:ENV123', 'user', '')
       end
 
       it "should mark environment as in use by user" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('claim ENV123', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with confirmation" do
@@ -36,13 +40,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is unknown to bot" do
 
       before(:each) do
-        subject.redis.del('environments:ENV123')
+        subject.redis.del('environments:my_project:ENV123')
       end
 
       it "should mark environment as in use by user" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('claim ENV123', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with confirmation" do
@@ -55,17 +59,17 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is in use by another user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV123', 'user', 'Alicia')
+        subject.redis.hset('environments:my_project:ENV123', 'user', 'Alicia')
       end
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('claim ENV123', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Alicia")
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to eq("Alicia")
       end
 
       it "should reply with notification" do
-        subject.redis.hset('environments:ENV123', 'user', 'Alicia')
+        subject.redis.hset('environments:my_project:ENV123', 'user', 'Alicia')
         carl = Lita::User.create(123, name: "Carl")
         send_command('claim ENV123', :as => carl)
         expect(replies.first).to eq("Hmm, ENV123 is currently in use by Alicia")
@@ -76,17 +80,17 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is already in use by user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV123', 'user', 'Carl')
+        subject.redis.hset('environments:my_project:ENV123', 'user', 'Carl')
       end
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('claim ENV123', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with notification" do
-        subject.redis.hset('environments:ENV123', 'user', 'Carl')
+        subject.redis.hset('environments:my_project:ENV123', 'user', 'Carl')
         carl = Lita::User.create(123, name: "Carl")
         send_command('claim ENV123', :as => carl)
         expect(replies.first).to eq("Hmm, you are already using ENV123")
@@ -101,13 +105,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is in use by user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV234', 'user', 'Alicia')
+        subject.redis.hset('environments:my_project:ENV234', 'user', 'Alicia')
       end
 
       it "should mark environment as available" do
         alicia = Lita::User.create(123, name: "Alicia")
         send_command('release ENV234', :as => alicia)
-        expect(subject.redis.hget('environments:ENV234', 'user')).to be_empty
+        expect(subject.redis.hget('environments:my_project:ENV234', 'user')).to be_empty
       end
 
       it "should reply with confirmation" do
@@ -121,13 +125,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is in use by another user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV234', 'user', 'Carl')
+        subject.redis.hset('environments:my_project:ENV234', 'user', 'Carl')
       end
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
         send_command('release ENV234', :as => alicia)
-        expect(subject.redis.hget('environments:ENV234', 'user')).to eq('Carl')
+        expect(subject.redis.hget('environments:my_project:ENV234', 'user')).to eq('Carl')
       end
 
       it "should reply with notification" do
@@ -141,13 +145,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is not in use" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV234', 'user', '')
+        subject.redis.hset('environments:my_project:ENV234', 'user', '')
       end
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
         send_command('release ENV234', :as => alicia)
-        expect(subject.redis.hget('environments:ENV234', 'user')).to eq('')
+        expect(subject.redis.hget('environments:my_project:ENV234', 'user')).to eq('')
       end
 
       it "should reply with notification" do
@@ -161,13 +165,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is unknown to bot" do
 
       before(:each) do
-        subject.redis.del('environments:ENV234')
+        subject.redis.del('environments:my_project:ENV234')
       end
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
         send_command('release ENV234', :as => alicia)
-        expect(subject.redis.hget('environments:ENV234', 'user')).to be_nil
+        expect(subject.redis.hget('environments:my_project:ENV234', 'user')).to be_nil
       end
 
       it "should reply with notification" do
@@ -183,9 +187,9 @@ describe Lita::Handlers::Envy, lita_handler: true do
   describe 'User listing environments' do
 
     it "should list environments" do
-      subject.redis.hset('environments:ENV123', 'user', 'Alicia')
-      subject.redis.hset('environments:ENV234', 'user', 'Carl')
-      subject.redis.hset('environments:ENV345', 'user', '')
+      subject.redis.hset('environments:my_project:ENV123', 'user', 'Alicia')
+      subject.redis.hset('environments:my_project:ENV234', 'user', 'Carl')
+      subject.redis.hset('environments:my_project:ENV345', 'user', '')
       send_command('envs')
       expect(replies.first.split("\n")).to eq([
         "ENV123 (Alicia)",
@@ -201,12 +205,12 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is available" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV345', 'user', '')
+        subject.redis.hset('environments:my_project:ENV345', 'user', '')
       end
 
       it "should forgetironments" do
         send_command('forget ENV345')
-        expect(subject.redis.keys).to_not include('environments:ENV345')
+        expect(subject.redis.keys).to_not include('environments:my_project:ENV345')
       end
 
       it "should confirm" do
@@ -219,7 +223,7 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is unknown to bot" do
 
       before(:each) do
-        subject.redis.del('environments:ENV345')
+        subject.redis.del('environments:my_project:ENV345')
       end
 
       it "should reply with notification" do
@@ -232,13 +236,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is in use by another user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV345', 'user', 'Carl')
+        subject.redis.hset('environments:my_project:ENV345', 'user', 'Carl')
       end
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
         send_command('forget ENV345', :as => alicia)
-        expect(subject.redis.hget('environments:ENV345', 'user')).to eq('Carl')
+        expect(subject.redis.hget('environments:my_project:ENV345', 'user')).to eq('Carl')
       end
 
       it "should reply with notification" do
@@ -251,13 +255,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is in use by user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV345', 'user', 'Alicia')
+        subject.redis.hset('environments:my_project:ENV345', 'user', 'Alicia')
       end
 
       it "should leave the environment untouched" do
         alicia = Lita::User.create(123, name: "Alicia")
         send_command('forget ENV345', :as => alicia)
-        expect(subject.redis.hget('environments:ENV345', 'user')).to eq('Alicia')
+        expect(subject.redis.hget('environments:my_project:ENV345', 'user')).to eq('Alicia')
       end
 
       it "should reply with notification" do
@@ -275,13 +279,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is currently in use by specified user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV123', 'user', 'Alicia')
+        subject.redis.hset('environments:my_project:ENV123', 'user', 'Alicia')
       end
 
       it "should mark environment as in use" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('wrestle ENV123 from Alicia', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Carl")
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to eq("Carl")
       end
 
       it "should reply with confirmation" do
@@ -294,13 +298,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is currently in use by a user other than the specified one" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV123', 'user', 'Alicia')
+        subject.redis.hset('environments:my_project:ENV123', 'user', 'Alicia')
       end
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('wrestle ENV123 from Ben', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to eq("Alicia")
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to eq("Alicia")
       end
 
       it "should reply with notification" do
@@ -313,13 +317,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is not currently in use" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV123', 'user', nil)
+        subject.redis.hset('environments:my_project:ENV123', 'user', nil)
       end
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('wrestle ENV123 from Ben', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to be_empty
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to be_empty
       end
 
       it "should reply with notification" do
@@ -332,13 +336,13 @@ describe Lita::Handlers::Envy, lita_handler: true do
     context "when environment is already marked as in use by requesting user" do
 
       before(:each) do
-        subject.redis.hset('environments:ENV123', 'user', 'Carl')
+        subject.redis.hset('environments:my_project:ENV123', 'user', 'Carl')
       end
 
       it "should leave the environment untouched" do
         carl = Lita::User.create(123, name: "Carl")
         send_command('wrestle ENV123 from Ben', :as => carl)
-        expect(subject.redis.hget('environments:ENV123', 'user')).to eq('Carl')
+        expect(subject.redis.hget('environments:my_project:ENV123', 'user')).to eq('Carl')
       end
 
       it "should reply with notification" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-envy
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Ingo Weiss
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-02 00:00:00.000000000 Z
+date: 2016-08-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -136,9 +136,10 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
-description: Mark environments as in use. Mark environments as available. List environments
+description: Helps with management of environment ussage. Adds commands for claiming
+  and releasing environments
 email:
-- ingo.weiss@lab49.com
+- ingo@ingoweiss.com
 executables: []
 extensions: []
 extra_rdoc_files: []