lita-locker 0.5.3 → 0.7.0

data/lita-locker.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = 'lita-locker'
-  spec.version       = '0.5.3'
+  spec.version       = '0.7.0'
   spec.authors       = ['Eric Sigler']
   spec.email         = ['me@esigler.com']
   spec.description   = '"lock" and "unlock" arbitrary subjects'
@@ -14,12 +14,12 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(/^(test|spec|features)\//)
   spec.require_paths = ['lib']
 
-  spec.add_runtime_dependency 'lita', '>= 3.2'
+  spec.add_runtime_dependency 'lita', '>= 4.0.1'
 
   spec.add_development_dependency 'bundler', '~> 1.3'
+  spec.add_development_dependency 'coveralls'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'rspec', '>= 3.0.0'
-  spec.add_development_dependency 'simplecov'
-  spec.add_development_dependency 'coveralls'
   spec.add_development_dependency 'rubocop'
+  spec.add_development_dependency 'simplecov'
 end

data/locales/en.yml

@@ -2,6 +2,10 @@ en:
   lita:
     handlers:
       locker:
+        steal:
+          stolen: "%{label} stolen from %{old_owner} %{mention}"
+          already_unlocked: "%{label} was already unlocked"
+          self: Why are you stealing the lock from yourself?
         help:
           lock:
             syntax: lock <subject>
@@ -15,6 +19,9 @@ en:
           status:
             syntax: locker status <label or resource>
             desc: Show the current state of <label or resource>
+          list:
+            syntax: locker list <username>
+            desc: Show what locks a user currently holds
           resource:
             list:
               syntax: locker resource list
@@ -80,4 +87,4 @@ en:
           resources: "Label %{name} has: %{resources}"
           does_not_have_resource: "Label %{label} does not have Resource %{resource}"
           no_resources: "%{name} has no resources, so it cannot be locked"
-          dependency: 'Label unable to be locked, blocked on a dependency'
+          dependency: 'Label unable to be locked, blocked on:'

data/spec/lita/handlers/locker_events_spec.rb

@@ -0,0 +1,6 @@
+require 'spec_helper'
+
+describe Lita::Handlers::LockerEvents, lita_handler: true do
+  it { is_expected.to route_event(:lock_attempt).to(:lock_attempt) }
+  it { is_expected.to route_event(:unlock_attempt).to(:unlock_attempt) }
+end

data/spec/lita/handlers/locker_http_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe Lita::Handlers::LockerHttp, lita_handler: true do
+  it do
+    is_expected.to route_http(:get, '/locker/label/foobar')
+      .to(:http_label_show)
+
+    is_expected.to route_http(:get, '/locker/resource/foobar')
+      .to(:http_resource_show)
+  end
+end

data/spec/lita/handlers/locker_labels_spec.rb

@@ -0,0 +1,147 @@
+require 'spec_helper'
+
+describe Lita::Handlers::LockerLabels, lita_handler: true do
+  before do
+    robot.auth.add_user_to_group!(user, :locker_admins)
+  end
+
+  label_examples = ['foobar', 'foo bar', 'foo-bar', 'foo_bar']
+
+  label_examples.each do |l|
+    it do
+      is_expected.to route_command("locker label create #{l}").to(:create)
+      is_expected.to route_command("locker label delete #{l}").to(:delete)
+      is_expected.to route_command("locker label show #{l}").to(:show)
+      is_expected.to route_command("locker label add resource to #{l}").to(:add)
+      is_expected.to route_command("locker label remove resource from #{l}").to(:remove)
+    end
+  end
+
+  it { is_expected.to route_command('locker label list').to(:list) }
+
+  describe '#label_list' do
+    it 'shows a list of labels if there are any' do
+      send_command('locker label create foobar')
+      send_command('locker label create bazbat')
+      send_command('locker label list')
+      expect(replies.include?('Label: foobar, state: unlocked')).to eq(true)
+      expect(replies.include?('Label: bazbat, state: unlocked')).to eq(true)
+    end
+  end
+
+  describe '#label_create' do
+    it 'creates a label with <name>' do
+      send_command('locker label create foobar')
+      expect(replies.last).to eq('Label foobar created')
+    end
+
+    it 'shows a warning when the <name> already exists as a label' do
+      send_command('locker label create foobar')
+      send_command('locker label create foobar')
+      expect(replies.last).to eq('foobar already exists')
+    end
+
+    it 'shows a warning when the <name> already exists as a resource' do
+      send_command('locker resource create foobar')
+      send_command('locker label create foobar')
+      expect(replies.last).to eq('foobar already exists')
+    end
+  end
+
+  describe '#label_delete' do
+    it 'deletes a label with <name>' do
+      send_command('locker label create foobar')
+      send_command('locker label delete foobar')
+      expect(replies.last).to eq('Label foobar deleted')
+    end
+
+    it 'shows a warning when <name> does not exist' do
+      send_command('locker label delete foobar')
+      expect(replies.last).to eq('Label foobar does not exist.  To create it: "!locker label create foobar"')
+    end
+  end
+
+  describe '#label_show' do
+    it 'shows a list of resources for a label if there are any' do
+      send_command('locker resource create whatever')
+      send_command('locker label create foobar')
+      send_command('locker label add whatever to foobar')
+      send_command('locker label show foobar')
+      expect(replies.last).to eq('Label foobar has: whatever')
+    end
+
+    it 'shows a warning if there are no resources for the label' do
+      send_command('locker label create foobar')
+      send_command('locker label show foobar')
+      expect(replies.last).to eq('Label foobar has no resources')
+    end
+
+    it 'shows an error if the label does not exist' do
+      send_command('locker label show foobar')
+      expect(replies.last).to eq('Label foobar does not exist.  To create it: "!locker label create foobar"')
+    end
+  end
+
+  describe '#label_add' do
+    it 'adds a resource to a label if both exist' do
+      send_command('locker resource create foo')
+      send_command('locker label create bar')
+      send_command('locker label add foo to bar')
+      expect(replies.last).to eq('Resource foo has been added to bar')
+      send_command('locker label show bar')
+      expect(replies.last).to eq('Label bar has: foo')
+    end
+
+    it 'adds multiple resources to a label if all exist' do
+      send_command('locker resource create foo')
+      send_command('locker resource create baz')
+      send_command('locker label create bar')
+      send_command('locker label add foo to bar')
+      send_command('locker label add baz to bar')
+      send_command('locker label show bar')
+      expect(replies.last).to match(/Label bar has:/)
+      expect(replies.last).to match(/foo/)
+      expect(replies.last).to match(/baz/)
+    end
+
+    it 'shows an error if the label does not exist' do
+      send_command('locker label add foo to bar')
+      expect(replies.last).to eq('Label bar does not exist.  To create it: "!locker label create bar"')
+    end
+
+    it 'shows an error if the resource does not exist' do
+      send_command('locker label create bar')
+      send_command('locker label add foo to bar')
+      expect(replies.last).to eq('Resource foo does not exist')
+    end
+  end
+
+  describe '#label_remove' do
+    it 'removes a resource from a label if both exist and are related' do
+      send_command('locker resource create foo')
+      send_command('locker label create bar')
+      send_command('locker label add foo to bar')
+      send_command('locker label remove foo from bar')
+      send_command('locker label show bar')
+      expect(replies.last).to eq('Label bar has no resources')
+    end
+
+    it 'shows an error if they both exist but are not related' do
+      send_command('locker resource create foo')
+      send_command('locker label create bar')
+      send_command('locker label remove foo from bar')
+      expect(replies.last).to eq('Label bar does not have Resource foo')
+    end
+
+    it 'shows an error if the label does not exist' do
+      send_command('locker label add foo to bar')
+      expect(replies.last).to eq('Label bar does not exist.  To create it: "!locker label create bar"')
+    end
+
+    it 'shows an error if the resource does not exist' do
+      send_command('locker label create bar')
+      send_command('locker label add foo to bar')
+      expect(replies.last).to eq('Resource foo does not exist')
+    end
+  end
+end

data/spec/lita/handlers/locker_misc_spec.rb

@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+describe Lita::Handlers::LockerMisc, lita_handler: true do
+  before do
+    robot.auth.add_user_to_group!(user, :locker_admins)
+  end
+
+  label_examples = ['foobar', 'foo bar', 'foo-bar', 'foo_bar']
+  resource_examples = ['foobar', 'foo.bar', 'foo-bar', 'foo_bar']
+
+  label_examples.each do |l|
+    it { is_expected.to route_command("locker status #{l}").to(:status) }
+  end
+
+  resource_examples.each do |r|
+    it { is_expected.to route_command("locker status #{r}").to(:status) }
+  end
+
+  it { is_expected.to route_command('locker list @alice').to(:list) }
+  it { is_expected.to route_command('locker list Alice').to(:list) }
+
+  describe '#status' do
+    it 'shows the status of a label' do
+      send_command('locker resource create bar')
+      send_command('locker label create foo')
+      send_command('locker label add bar to foo')
+      send_command('locker status foo')
+      expect(replies.last).to eq('Label: foo, state: unlocked')
+      send_command('lock foo')
+      send_command('locker status foo')
+      expect(replies.last).to eq('Label: foo, state: locked, owner: Test User')
+    end
+
+    it 'shows the status of a resource' do
+      send_command('locker resource create bar')
+      send_command('locker label create foo')
+      send_command('locker label add bar to foo')
+      send_command('locker status bar')
+      expect(replies.last).to eq('Resource: bar, state: unlocked')
+      send_command('lock foo')
+      send_command('locker status bar')
+      expect(replies.last).to eq('Resource: bar, state: locked')
+    end
+
+    it 'shows an error if nothing exists with that name' do
+      send_command('locker status foo')
+      expect(replies.last).to eq('Sorry, that does not exist')
+    end
+  end
+
+  describe '#user_locks' do
+    it 'shows if a user has taken any locks' do
+      send_command('locker resource create foobar')
+      send_command('locker label create bazbat')
+      send_command('locker label add foobar to bazbat')
+      alice = Lita::User.create('9001@hipchat', name: 'Alice Alpha', mention_name: 'alice')
+      send_command('lock bazbat', as: alice)
+      send_command('locker list Alice Alpha')
+      expect(replies.last).to eq("Label: bazbat\n")
+    end
+
+    it 'shows if a mention name has taken any locks' do
+      send_command('locker resource create foobar')
+      send_command('locker label create bazbat')
+      send_command('locker label add foobar to bazbat')
+      alice = Lita::User.create('9001@hipchat', name: 'Alice Alpha', mention_name: 'alice')
+      send_command('lock bazbat', as: alice)
+      send_command('locker list @alice')
+      expect(replies.last).to eq("Label: bazbat\n")
+    end
+
+    it 'shows an empty set if the user has not taken any locks' do
+      send_command('locker resource create foobar')
+      send_command('locker label create bazbat')
+      send_command('locker label add foobar to bazbat')
+      alice = Lita::User.create('9001@hipchat', name: 'Alice', mention_name: 'alice')
+      send_command('locker list Alice', as: alice)
+      expect(replies.last).to eq('That user has no active locks')
+      send_command('lock bazbat', as: alice)
+      send_command('unlock bazbat', as: alice)
+      send_command('locker list Alice', as: alice)
+      expect(replies.last).to eq('That user has no active locks')
+    end
+
+    it 'shows a warning when the user does not exist' do
+      send_command('locker list foobar')
+      expect(replies.last).to eq('Unknown user')
+    end
+  end
+end

data/spec/lita/handlers/locker_resources_spec.rb

@@ -0,0 +1,74 @@
+require 'spec_helper'
+
+describe Lita::Handlers::LockerResources, lita_handler: true do
+  before do
+    robot.auth.add_user_to_group!(user, :locker_admins)
+  end
+
+  resource_examples = ['foobar', 'foo.bar', 'foo-bar', 'foo_bar']
+
+  resource_examples.each do |r|
+    it do
+      is_expected.to route_command("locker resource create #{r}").to(:create)
+      is_expected.to route_command("locker resource delete #{r}").to(:delete)
+      is_expected.to route_command("locker resource show #{r}").to(:show)
+    end
+  end
+
+  it { is_expected.to route_command('locker resource list').to(:list) }
+
+  describe '#resource_list' do
+    it 'shows a list of resources if there are any' do
+      send_command('locker resource create foobar')
+      send_command('locker resource create bazbat')
+      send_command('locker resource list')
+      expect(replies.last).to match(/Resource: foobar, state: unlocked/)
+      expect(replies.last).to match(/Resource: bazbat, state: unlocked/)
+    end
+  end
+
+  describe '#resource_create' do
+    it 'creates a resource with <name>' do
+      send_command('locker resource create foobar')
+      expect(replies.last).to eq('Resource foobar created')
+    end
+
+    it 'shows a warning when the <name> already exists as a resource' do
+      send_command('locker resource create foobar')
+      send_command('locker resource create foobar')
+      expect(replies.last).to eq('foobar already exists')
+    end
+
+    it 'shows a warning when the <name> already exists as a label' do
+      send_command('locker label create foobar')
+      send_command('locker resource create foobar')
+      expect(replies.last).to eq('foobar already exists')
+    end
+  end
+
+  describe '#resource_delete' do
+    it 'deletes a resource with <name>' do
+      send_command('locker resource create foobar')
+      send_command('locker resource delete foobar')
+      expect(replies.last).to eq('Resource foobar deleted')
+    end
+
+    it 'shows a warning when <name> does not exist' do
+      send_command('locker resource delete foobar')
+      expect(replies.last).to eq('Resource foobar does not exist')
+    end
+  end
+
+  describe '#resource_show' do
+    it 'shows the state of a <name> if it exists' do
+      send_command('locker resource create foobar')
+      send_command('locker resource show foobar')
+      expect(replies.last).to eq('Resource: foobar, state: unlocked')
+    end
+
+    it 'shows a warning when <name> does not exist' do
+      send_command('locker resource show foobar')
+      expect(replies.last).to eq('Resource foobar does not exist')
+    end
+  end
+end

data/spec/lita/handlers/locker_spec.rb

@@ -1,60 +1,33 @@
 require 'spec_helper'
 
 describe Lita::Handlers::Locker, lita_handler: true do
-  label_examples = ['foobar', 'foo bar', 'foo-bar', 'foo_bar']
-  resource_examples = ['foobar', 'foo.bar', 'foo-bar', 'foo_bar']
-
-  label_examples.each do |l|
-    it { routes("(lock) #{l}").to(:lock) }
-    it { routes("(unlock) #{l}").to(:unlock) }
-    it { routes("(release) #{l}").to(:unlock) }
-
-    it { routes("(lock) #{l} #this is a comment").to(:lock) }
-    it { routes("(unlock) #{l} #this is a comment").to(:unlock) }
-    it { routes("(release) #{l} #this is a comment").to(:unlock) }
-
-    it { routes_command("lock #{l}").to(:lock) }
-    it { routes_command("lock #{l} #this is a comment").to(:lock) }
-    it { routes_command("unlock #{l}").to(:unlock) }
-    it { routes_command("unlock #{l} #this is a comment").to(:unlock) }
-    it { routes_command("steal #{l}").to(:steal) }
-    it { routes_command("steal #{l} #this is a comment").to(:steal) }
-  end
-
-  label_examples.each do |l|
-    it { routes_command("locker status #{l}").to(:status) }
-  end
-
-  resource_examples.each do |r|
-    it { routes_command("locker status #{r}").to(:status) }
-  end
-
-  it { routes_command('locker resource list').to(:resource_list) }
-
-  resource_examples.each do |r|
-    it { routes_command("locker resource create #{r}").to(:resource_create) }
-    it { routes_command("locker resource delete #{r}").to(:resource_delete) }
-    it { routes_command("locker resource show #{r}").to(:resource_show) }
+  before do
+    robot.auth.add_user_to_group!(user, :locker_admins)
   end
 
-  it { routes_command('locker label list').to(:label_list) }
+  label_examples = ['foobar', 'foo bar', 'foo-bar', 'foo_bar']
 
   label_examples.each do |l|
-    it { routes_command("locker label create #{l}").to(:label_create) }
-    it { routes_command("locker label delete #{l}").to(:label_delete) }
-    it { routes_command("locker label show #{l}").to(:label_show) }
-    it { routes_command("locker label add resource to #{l}").to(:label_add) }
-    it { routes_command("locker label remove resource from #{l}").to(:label_remove) }
-  end
+    it do
+      is_expected.to route("(lock) #{l}").to(:lock)
+      is_expected.to route("(unlock) #{l}").to(:unlock)
+      is_expected.to route("(release) #{l}").to(:unlock)
 
-  it { routes_http(:get, '/locker/label/foobar').to(:http_label_show) }
-  it { routes_http(:get, '/locker/resource/foobar').to(:http_resource_show) }
+      is_expected.to route("(Lock) #{l}").to(:lock)
+      is_expected.to route("(Unlock) #{l}").to(:unlock)
+      is_expected.to route("(Release) #{l}").to(:unlock)
 
-  before do
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :locker_admins
-    ).and_return(true)
+      is_expected.to route("(lock) #{l} #this is a comment").to(:lock)
+      is_expected.to route("(unlock) #{l} #this is a comment").to(:unlock)
+      is_expected.to route("(release) #{l} #this is a comment").to(:unlock)
+
+      is_expected.to route_command("lock #{l}").to(:lock)
+      is_expected.to route_command("lock #{l} #this is a comment").to(:lock)
+      is_expected.to route_command("unlock #{l}").to(:unlock)
+      is_expected.to route_command("unlock #{l} #this is a comment").to(:unlock)
+      is_expected.to route_command("steal #{l}").to(:steal)
+      is_expected.to route_command("steal #{l} #this is a comment").to(:steal)
+    end
   end
 
   let(:alice) do
@@ -92,7 +65,7 @@ describe Lita::Handlers::Locker, lita_handler: true do
       send_command('lock l1', as: alice)
       send_command('lock l2', as: alice)
       expect(replies.last).to eq('(failed) Label unable to be locked, ' \
-                                 'blocked on a dependency')
+                                 "blocked on:\nr1 - Alice")
     end
 
     it 'shows a warning when a label is taken by someone else' do
@@ -166,226 +139,30 @@ describe Lita::Handlers::Locker, lita_handler: true do
       send_command('locker label add foobar to bazbat')
       send_command('lock bazbat', as: alice)
       send_command('steal bazbat # with a comment', as: bob)
-      expect(replies.last).to eq('(successful) bazbat unlocked')
-    end
-
-    it 'shows an error when a <subject> does not exist' do
-      send_command('steal foobar')
-      expect(replies.last).to eq('(failed) Sorry, that does not exist')
-    end
-  end
-
-  describe '#status' do
-    it 'shows the status of a label' do
-      send_command('locker resource create bar')
-      send_command('locker label create foo')
-      send_command('locker label add bar to foo')
-      send_command('locker status foo')
-      expect(replies.last).to eq('Label: foo, state: unlocked')
-      send_command('lock foo')
-      send_command('locker status foo')
-      expect(replies.last).to eq('Label: foo, state: locked, owner: Test User')
-    end
-
-    it 'shows the status of a resource' do
-      send_command('locker resource create bar')
-      send_command('locker label create foo')
-      send_command('locker label add bar to foo')
-      send_command('locker status bar')
-      expect(replies.last).to eq('Resource: bar, state: unlocked')
-      send_command('lock foo')
-      send_command('locker status bar')
-      expect(replies.last).to eq('Resource: bar, state: locked')
-    end
-
-    it 'shows an error if nothing exists with that name' do
-      send_command('locker status foo')
-      expect(replies.last).to eq('Sorry, that does not exist')
-    end
-  end
-
-  describe '#label_list' do
-    it 'shows a list of labels if there are any' do
-      send_command('locker label create foobar')
-      send_command('locker label create bazbat')
-      send_command('locker label list')
-      expect(replies.include?('Label: foobar, state: unlocked')).to eq(true)
-      expect(replies.include?('Label: bazbat, state: unlocked')).to eq(true)
-    end
-  end
-
-  describe '#label_create' do
-    it 'creates a label with <name>' do
-      send_command('locker label create foobar')
-      expect(replies.last).to eq('Label foobar created')
-    end
-
-    it 'shows a warning when the <name> already exists as a label' do
-      send_command('locker label create foobar')
-      send_command('locker label create foobar')
-      expect(replies.last).to eq('foobar already exists')
-    end
-
-    it 'shows a warning when the <name> already exists as a resource' do
-      send_command('locker resource create foobar')
-      send_command('locker label create foobar')
-      expect(replies.last).to eq('foobar already exists')
-    end
-  end
-
-  describe '#label_delete' do
-    it 'deletes a label with <name>' do
-      send_command('locker label create foobar')
-      send_command('locker label delete foobar')
-      expect(replies.last).to eq('Label foobar deleted')
-    end
-
-    it 'shows a warning when <name> does not exist' do
-      send_command('locker label delete foobar')
-      expect(replies.last).to eq('Label foobar does not exist.  To create ' \
-                                 'it: "!locker label create foobar"')
-    end
-  end
-
-  describe '#label_show' do
-    it 'shows a list of resources for a label if there are any' do
-      send_command('locker resource create whatever')
-      send_command('locker label create foobar')
-      send_command('locker label add whatever to foobar')
-      send_command('locker label show foobar')
-      expect(replies.last).to eq('Label foobar has: whatever')
-    end
-
-    it 'shows a warning if there are no resources for the label' do
-      send_command('locker label create foobar')
-      send_command('locker label show foobar')
-      expect(replies.last).to eq('Label foobar has no resources')
-    end
-
-    it 'shows an error if the label does not exist' do
-      send_command('locker label show foobar')
-      expect(replies.last).to eq('Label foobar does not exist.  To create ' \
-                                 'it: "!locker label create foobar"')
-    end
-  end
-
-  describe '#label_add' do
-    it 'adds a resource to a label if both exist' do
-      send_command('locker resource create foo')
-      send_command('locker label create bar')
-      send_command('locker label add foo to bar')
-      expect(replies.last).to eq('Resource foo has been added to bar')
-      send_command('locker label show bar')
-      expect(replies.last).to eq('Label bar has: foo')
-    end
-
-    it 'adds multiple resources to a label if all exist' do
-      send_command('locker resource create foo')
-      send_command('locker resource create baz')
-      send_command('locker label create bar')
-      send_command('locker label add foo to bar')
-      send_command('locker label add baz to bar')
-      send_command('locker label show bar')
-      expect(replies.last).to match(/Label bar has:/)
-      expect(replies.last).to match(/foo/)
-      expect(replies.last).to match(/baz/)
-    end
-
-    it 'shows an error if the label does not exist' do
-      send_command('locker label add foo to bar')
-      expect(replies.last).to eq('Label bar does not exist.  To create ' \
-                                 'it: "!locker label create bar"')
-    end
-
-    it 'shows an error if the resource does not exist' do
-      send_command('locker label create bar')
-      send_command('locker label add foo to bar')
-      expect(replies.last).to eq('Resource foo does not exist')
-    end
-  end
-
-  describe '#label_remove' do
-    it 'removes a resource from a label if both exist and are related' do
-      send_command('locker resource create foo')
-      send_command('locker label create bar')
-      send_command('locker label add foo to bar')
-      send_command('locker label remove foo from bar')
-      send_command('locker label show bar')
-      expect(replies.last).to eq('Label bar has no resources')
-    end
-
-    it 'shows an error if they both exist but are not related' do
-      send_command('locker resource create foo')
-      send_command('locker label create bar')
-      send_command('locker label remove foo from bar')
-      expect(replies.last).to eq('Label bar does not have Resource foo')
-    end
-
-    it 'shows an error if the label does not exist' do
-      send_command('locker label add foo to bar')
-      expect(replies.last).to eq('Label bar does not exist.  To create ' \
-                                 'it: "!locker label create bar"')
-    end
-
-    it 'shows an error if the resource does not exist' do
-      send_command('locker label create bar')
-      send_command('locker label add foo to bar')
-      expect(replies.last).to eq('Resource foo does not exist')
-    end
-  end
-
-  describe '#resource_list' do
-    it 'shows a list of resources if there are any' do
-      send_command('locker resource create foobar')
-      send_command('locker resource create bazbat')
-      send_command('locker resource list')
-      expect(replies.last).to match(/Resource: foobar, state: unlocked/)
-      expect(replies.last).to match(/Resource: bazbat, state: unlocked/)
-    end
-  end
-
-  describe '#resource_create' do
-    it 'creates a resource with <name>' do
-      send_command('locker resource create foobar')
-      expect(replies.last).to eq('Resource foobar created')
-    end
-
-    it 'shows a warning when the <name> already exists as a resource' do
-      send_command('locker resource create foobar')
-      send_command('locker resource create foobar')
-      expect(replies.last).to eq('foobar already exists')
-    end
-
-    it 'shows a warning when the <name> already exists as a label' do
-      send_command('locker label create foobar')
-      send_command('locker resource create foobar')
-      expect(replies.last).to eq('foobar already exists')
+      expect(replies.last).to eq('(successful) bazbat stolen from ' \
+                                 'Alice (@alice)')
     end
-  end
 
-  describe '#resource_delete' do
-    it 'deletes a resource with <name>' do
+    it 'shows a warning when the label is already unlocked' do
       send_command('locker resource create foobar')
-      send_command('locker resource delete foobar')
-      expect(replies.last).to eq('Resource foobar deleted')
-    end
-
-    it 'shows a warning when <name> does not exist' do
-      send_command('locker resource delete foobar')
-      expect(replies.last).to eq('Resource foobar does not exist')
+      send_command('locker label create bazbat')
+      send_command('locker label add foobar to bazbat')
+      send_command('steal bazbat # with a comment', as: alice)
+      expect(replies.last).to eq('bazbat was already unlocked')
     end
-  end
 
-  describe '#resource_show' do
-    it 'shows the state of a <name> if it exists' do
+    it 'shows a warning when the label is being stolen by the owner' do
       send_command('locker resource create foobar')
-      send_command('locker resource show foobar')
-      expect(replies.last).to eq('Resource: foobar, state: unlocked')
+      send_command('locker label create bazbat')
+      send_command('locker label add foobar to bazbat')
+      send_command('lock bazbat', as: alice)
+      send_command('steal bazbat # with a comment', as: alice)
+      expect(replies.last).to eq('Why are you stealing the lock from yourself?')
     end
 
-    it 'shows a warning when <name> does not exist' do
-      send_command('locker resource show foobar')
-      expect(replies.last).to eq('Resource foobar does not exist')
+    it 'shows an error when a <subject> does not exist' do
+      send_command('steal foobar')
+      expect(replies.last).to eq('(failed) Sorry, that does not exist')
     end
   end
 end