lita-digitalocean 1.0.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ceb867fa1a82a608494d9af9340cc2a9f2ccff2f
-  data.tar.gz: 79c78fd54b3ba4fb044e71c2d3687909c594af7f
+  metadata.gz: 897834e48fba0df1715fc70d3b6ae15a5fea915e
+  data.tar.gz: 9c478b4e84b69afd2d306fae75d079ddefea192c
 SHA512:
-  metadata.gz: 28c176473b7afe92034be4a40e66855264a7d6f2281b8760a1e8dd99eca7566c1997aa3daf58caf3b97a821ff74a802c8843d80166825c8e890fe6c47a9867fb
-  data.tar.gz: 6d9cda0c6800bee0cec9c2a015dbfd71eb7ec9b8c6d9d7c1573d2550f81a51c9bc9a0c99ef12c6fd542c082650c30fad628a41a0090436070e126a0ed48eafd0
+  metadata.gz: aac5a9f7feccc0bccce84bc9449686b394386b357d826f46cb6dde08622b75a67068a4c39a4d600d4f280972f3851efc03e21f40a57e0e6d62929f9e5c4ab376
+  data.tar.gz: e2240a4b0be9966fb88f0abe68e408553e48c35d8963b2dcb47756280bfd5bf5b3c869f6303235579e55b4976641725ce3a5f6da57034ead7b4c26b9a433ee5a

data/lib/lita/handlers/digitalocean.rb

@@ -1,10 +1,8 @@
 module Lita
   module Handlers
     class Digitalocean < Handler
-      def self.default_config(config)
-        config.client_id = nil
-        config.api_key = nil
-      end
+      config :client_id, type: String, required: true
+      config :api_key, type: String, required: true
     end
 
     Lita.register_handler(Digitalocean)

data/lib/lita/handlers/digitalocean/base.rb

@@ -6,10 +6,7 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class Base < Handler
-        # Override key for locale namespace.
-        def self.name
-          "Digitalocean"
-        end
+        namespace "digitalocean"
 
         private
 

data/lib/lita/handlers/digitalocean/domain.rb

@@ -2,6 +2,8 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class Domain < Base
+        namespace "digitalocean"
+
         do_route /^do\s+domains?\s+create\s+[^\s]+\s+[^\s]+$/i, :create, {
           t("help.domains.create_key") => t("help.domains.create_value")
         }

data/lib/lita/handlers/digitalocean/domain_record.rb

@@ -2,6 +2,8 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class DomainRecord < Base
+        namespace "digitalocean"
+
         do_route /^do\s+domain\s+records?\s+create\s(?:[^\s]+\s+){2}[^\s]/i, :create, {
           t("help.domain_records.create_key") => t("help.domain_records.create_value")
         }, {

data/lib/lita/handlers/digitalocean/droplet.rb

@@ -2,6 +2,8 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class Droplet < Base
+        namespace "digitalocean"
+
         do_route /^do\s+droplets?\s+create(?:\s+[^\s]+){4}/i, :create, {
           t("help.droplets.create_key") => t("help.droplets.create_value")
         }, {

data/lib/lita/handlers/digitalocean/image.rb

@@ -2,6 +2,8 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class Image < Base
+        namespace "digitalocean"
+
         do_route /^do\s+images?\s+delete\s+([^\s]+)$/i, :delete, {
           t("help.images.delete_key") => t("help.images.delete_value")
         }

data/lib/lita/handlers/digitalocean/region.rb

@@ -2,6 +2,8 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class Region < Base
+        namespace "digitalocean"
+
         do_route /^do\s+regions?\s+list$/i, :list, {
           t("help.regions.list_key") => t("help.regions.list_value")
         }

data/lib/lita/handlers/digitalocean/size.rb

@@ -2,6 +2,8 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class Size < Base
+        namespace "digitalocean"
+
         do_route /^do\s+sizes\s+list$/, :list, {
           t("help.sizes.list_key") => t("help.sizes.list_value")
         }

data/lib/lita/handlers/digitalocean/ssh_key.rb

@@ -2,6 +2,8 @@ module Lita
   module Handlers
     class Digitalocean < Handler
       class SSHKey < Base
+        namespace "digitalocean"
+
         do_route /^do\s+ssh\s+keys?\s+add\s+.+$/i, :add, {
           t("help.ssh_keys.add_key") => t("help.ssh_keys.add_value")
         }

data/lita-digitalocean.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-digitalocean"
-  spec.version       = "1.0.0"
+  spec.version       = "2.0.0"
   spec.authors       = ["Jimmy Cuadra"]
   spec.email         = ["jimmy@jimmycuadra.com"]
   spec.description   = %q{A Lita handler for managing DigitalOcean services.}
@@ -14,13 +14,13 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_runtime_dependency "lita", ">= 3.2"
+  spec.add_runtime_dependency "lita", ">= 4.0"
   spec.add_runtime_dependency "lita-keyword-arguments"
   spec.add_runtime_dependency "digital_ocean", ">= 1.5.0"
 
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
-  spec.add_development_dependency "rspec", ">= 3.0.0.beta2"
+  spec.add_development_dependency "rspec", ">= 3.0.0"
   spec.add_development_dependency "simplecov"
   spec.add_development_dependency "coveralls"
 end

data/spec/lita/handlers/digitalocean/base_spec.rb

@@ -1,10 +1,9 @@
 require "spec_helper"
 
 handler_class = Class.new(Lita::Handlers::Digitalocean::Base) do
-  route /do droplets list/, :list, command: true
+  namespace "digitalocean"
 
-  def self.default_config(config)
-  end
+  route /do droplets list/, :list, command: true
 
   def list(response)
     do_call(response) do |client|
@@ -15,19 +14,8 @@ end
 
 describe handler_class, lita_handler: true do
   describe "#do_call" do
-    it "responds with an error if the DigitalOcean API key is not set" do
-      send_command("do droplets list")
-      expect(replies.last).to include("client_id and api_key must be set")
-    end
-
-    it "responds with an error if the DigitalOcean client ID is not set" do
-      Lita.config.handlers.digitalocean.api_key = "secret"
-      send_command("do droplets list")
-      expect(replies.last).to include("client_id and api_key must be set")
-    end
-
     it "responds with an error if the DigitalOcean API responds with an error" do
-      Lita.config.handlers.digitalocean.tap do |config|
+      registry.config.handlers.digitalocean.tap do |config|
         config.api_key = "secret"
         config.client_id = "secret"
       end

data/spec/lita/handlers/digitalocean/domain_record_spec.rb

@@ -2,33 +2,29 @@ require "spec_helper"
 
 describe Lita::Handlers::Digitalocean::DomainRecord, lita_handler: true do
   it do
-    routes_command(
+    is_expected.to route_command(
       "do domain records create example.com txt 'some value' --name example.com"
     ).to(:create)
   end
-  it { routes_command("do domain records delete example.com 123").to(:delete) }
+  it { is_expected.to route_command("do domain records delete example.com 123").to(:delete) }
   it do
-    routes_command(
+    is_expected.to route_command(
       "do domain records edit example.com 123 txt 'some value' name=example.com"
     ).to(:edit)
   end
-  it { routes_command("do domain records list example.com").to(:list) }
-  it { routes_command("do domain records show example.com 123").to(:show) }
+  it { is_expected.to route_command("do domain records list example.com").to(:list) }
+  it { is_expected.to route_command("do domain records show example.com 123").to(:show) }
 
   let(:client) { instance_double("::DigitalOcean::API", domains: client_domains) }
   let(:client_domains) { instance_double("::DigitalOcean::Resource::Domain") }
 
   before do
-    Lita.config.handlers.digitalocean = Lita::Config.new
-    Lita.config.handlers.digitalocean.tap do |config|
+    registry.config.handlers.digitalocean.tap do |config|
       config.client_id = "CLIENT_ID"
       config.api_key = "API_KEY"
     end
 
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :digitalocean_admins
-    ).and_return(true)
+    robot.auth.add_user_to_group!(user, :digitalocean_admins)
 
     allow(::DigitalOcean::API).to receive(:new).and_return(client)
   end

data/spec/lita/handlers/digitalocean/domain_spec.rb

@@ -1,25 +1,21 @@
 require "spec_helper"
 
 describe Lita::Handlers::Digitalocean::Domain, lita_handler: true do
-  it { routes_command("do domains create example.com 10.10.10.10").to(:create) }
-  it { routes_command("do domains delete example.com").to(:delete) }
-  it { routes_command("do domains list").to(:list) }
-  it { routes_command("do domains show example.com").to(:show) }
+  it { is_expected.to route_command("do domains create example.com 10.10.10.10").to(:create) }
+  it { is_expected.to route_command("do domains delete example.com").to(:delete) }
+  it { is_expected.to route_command("do domains list").to(:list) }
+  it { is_expected.to route_command("do domains show example.com").to(:show) }
 
   let(:client) { instance_double("::DigitalOcean::API", domains: client_domains) }
   let(:client_domains) { instance_double("::DigitalOcean::Resource::Domain") }
 
   before do
-    Lita.config.handlers.digitalocean = Lita::Config.new
-    Lita.config.handlers.digitalocean.tap do |config|
+    registry.config.handlers.digitalocean.tap do |config|
       config.client_id = "CLIENT_ID"
       config.api_key = "API_KEY"
     end
 
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :digitalocean_admins
-    ).and_return(true)
+    robot.auth.add_user_to_group!(user, :digitalocean_admins)
 
     allow(::DigitalOcean::API).to receive(:new).and_return(client)
   end

data/spec/lita/handlers/digitalocean/droplet_spec.rb

@@ -2,24 +2,24 @@ require "spec_helper"
 
 describe Lita::Handlers::Digitalocean::Droplet, lita_handler: true do
   it do
-    routes_command(
+    is_expected.to route_command(
       "do droplets create example.com 512mb centos-5-8-x64 nyc1"
     ).to(:create)
   end
-  it { routes_command("do droplets delete 123").to(:delete) }
-  it { routes_command("do droplets delete 123 --scrub").to(:delete) }
-  it { routes_command("do droplets list").to(:list) }
-  it { routes_command("do droplets password reset 123").to(:password_reset) }
-  it { routes_command("do droplets power cycle 123").to(:power_cycle) }
-  it { routes_command("do droplets power off 123").to(:power_off) }
-  it { routes_command("do droplets power on 123").to(:power_on) }
-  it { routes_command("do droplets reboot 123").to(:reboot) }
-  it { routes_command("do droplets rebuild 123 456").to(:rebuild) }
-  it { routes_command("do droplets resize 123 1gb").to(:resize) }
-  it { routes_command("do droplets restore 123 456").to(:restore) }
-  it { routes_command("do droplets show 123").to(:show) }
-  it { routes_command("do droplets shutdown 123").to(:shutdown) }
-  it { routes_command("do droplets snapshot 123 'My Snapshot'").to(:snapshot) }
+  it { is_expected.to route_command("do droplets delete 123").to(:delete) }
+  it { is_expected.to route_command("do droplets delete 123 --scrub").to(:delete) }
+  it { is_expected.to route_command("do droplets list").to(:list) }
+  it { is_expected.to route_command("do droplets password reset 123").to(:password_reset) }
+  it { is_expected.to route_command("do droplets power cycle 123").to(:power_cycle) }
+  it { is_expected.to route_command("do droplets power off 123").to(:power_off) }
+  it { is_expected.to route_command("do droplets power on 123").to(:power_on) }
+  it { is_expected.to route_command("do droplets reboot 123").to(:reboot) }
+  it { is_expected.to route_command("do droplets rebuild 123 456").to(:rebuild) }
+  it { is_expected.to route_command("do droplets resize 123 1gb").to(:resize) }
+  it { is_expected.to route_command("do droplets restore 123 456").to(:restore) }
+  it { is_expected.to route_command("do droplets show 123").to(:show) }
+  it { is_expected.to route_command("do droplets shutdown 123").to(:shutdown) }
+  it { is_expected.to route_command("do droplets snapshot 123 'My Snapshot'").to(:snapshot) }
 
   let(:client) { instance_double("::DigitalOcean::API", droplets: client_droplets) }
   let(:client_droplets) { instance_double("::DigitalOcean::Resource::Droplet") }
@@ -27,16 +27,12 @@ describe Lita::Handlers::Digitalocean::Droplet, lita_handler: true do
   let(:do_ok) { { status: "OK" } }
 
   before do
-    Lita.config.handlers.digitalocean = Lita::Config.new
-    Lita.config.handlers.digitalocean.tap do |config|
+    registry.config.handlers.digitalocean.tap do |config|
       config.client_id = "CLIENT_ID"
       config.api_key = "API_KEY"
     end
 
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :digitalocean_admins
-    ).and_return(true)
+    robot.auth.add_user_to_group!(user, :digitalocean_admins)
 
     allow(::DigitalOcean::API).to receive(:new).and_return(client)
   end

data/spec/lita/handlers/digitalocean/image_spec.rb

@@ -1,25 +1,21 @@
 require "spec_helper"
 
 describe Lita::Handlers::Digitalocean::Image, lita_handler: true do
-  it { routes_command("do images delete 123").to(:delete) }
-  it { routes_command("do images list").to(:list) }
-  it { routes_command("do images list filter").to(:list) }
-  it { routes_command("do images show 123").to(:show) }
+  it { is_expected.to route_command("do images delete 123").to(:delete) }
+  it { is_expected.to route_command("do images list").to(:list) }
+  it { is_expected.to route_command("do images list filter").to(:list) }
+  it { is_expected.to route_command("do images show 123").to(:show) }
 
   let(:client) { instance_double("::DigitalOcean::API", images: client_images) }
   let(:client_images) { instance_double("::DigitalOcean::Resource::Image") }
 
   before do
-    Lita.config.handlers.digitalocean = Lita::Config.new
-    Lita.config.handlers.digitalocean.tap do |config|
+    registry.config.handlers.digitalocean.tap do |config|
       config.client_id = "CLIENT_ID"
       config.api_key = "API_KEY"
     end
 
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :digitalocean_admins
-    ).and_return(true)
+    robot.auth.add_user_to_group!(user, :digitalocean_admins)
 
     allow(::DigitalOcean::API).to receive(:new).and_return(client)
   end

data/spec/lita/handlers/digitalocean/region_spec.rb

@@ -1,22 +1,18 @@
 require "spec_helper"
 
 describe Lita::Handlers::Digitalocean::Region, lita_handler: true do
-  it { routes_command("do regions list").to(:list) }
+  it { is_expected.to route_command("do regions list").to(:list) }
 
   let(:client) { instance_double("::DigitalOcean::API", regions: client_regions) }
   let(:client_regions) { instance_double("::DigitalOcean::Resource::Region") }
 
   before do
-    Lita.config.handlers.digitalocean = Lita::Config.new
-    Lita.config.handlers.digitalocean.tap do |config|
+    registry.config.handlers.digitalocean.tap do |config|
       config.client_id = "CLIENT_ID"
       config.api_key = "API_KEY"
     end
 
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :digitalocean_admins
-    ).and_return(true)
+    robot.auth.add_user_to_group!(user, :digitalocean_admins)
 
     allow(::DigitalOcean::API).to receive(:new).and_return(client)
   end

data/spec/lita/handlers/digitalocean/size_spec.rb

@@ -1,22 +1,18 @@
 require "spec_helper"
 
 describe Lita::Handlers::Digitalocean::Size, lita_handler: true do
-  it { routes_command("do sizes list").to(:list) }
+  it { is_expected.to route_command("do sizes list").to(:list) }
 
   let(:client) { instance_double( "::DigitalOcean::API", sizes: client_sizes) }
   let(:client_sizes) { instance_double("::DigitalOcean::Resource::Size") }
 
   before do
-    Lita.config.handlers.digitalocean = Lita::Config.new
-    Lita.config.handlers.digitalocean.tap do |config|
+    registry.config.handlers.digitalocean.tap do |config|
       config.client_id = "CLIENT_ID"
       config.api_key = "API_KEY"
     end
 
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :digitalocean_admins
-    ).and_return(true)
+    robot.auth.add_user_to_group!(user, :digitalocean_admins)
 
     allow(::DigitalOcean::API).to receive(:new).and_return(client)
   end

data/spec/lita/handlers/digitalocean/ssh_key_spec.rb

@@ -1,30 +1,26 @@
 require "spec_helper"
 
 describe Lita::Handlers::Digitalocean::SSHKey, lita_handler: true do
-  it { routes_command("do ssh keys add 'foo bar' 'ssh-rsa abcdefg'").to(:add) }
-  it { routes_command("do ssh keys delete 123").to(:delete) }
+  it { is_expected.to route_command("do ssh keys add 'foo bar' 'ssh-rsa abcdefg'").to(:add) }
+  it { is_expected.to route_command("do ssh keys delete 123").to(:delete) }
   it do
-    routes_command(
+    is_expected.to route_command(
       "do ssh keys edit 123 --name foo --public-key 'ssh-rsa changed'"
     ).to(:edit)
   end
-  it { routes_command("do ssh keys list").to(:list) }
-  it { routes_command("do ssh keys show 123").to(:show) }
+  it { is_expected.to route_command("do ssh keys list").to(:list) }
+  it { is_expected.to route_command("do ssh keys show 123").to(:show) }
 
   let(:client) { instance_double("::DigitalOcean::API", ssh_keys: client_ssh_keys) }
   let(:client_ssh_keys) { instance_double("::DigitalOcean::Resource::SSHKey") }
 
   before do
-    Lita.config.handlers.digitalocean = Lita::Config.new
-    Lita.config.handlers.digitalocean.tap do |config|
+    registry.config.handlers.digitalocean.tap do |config|
       config.client_id = "CLIENT_ID"
       config.api_key = "API_KEY"
     end
 
-    allow(Lita::Authorization).to receive(:user_in_group?).with(
-      user,
-      :digitalocean_admins
-    ).and_return(true)
+    robot.auth.add_user_to_group!(user, :digitalocean_admins)
 
     allow(::DigitalOcean::API).to receive(:new).and_return(client)
   end

data/spec/spec_helper.rb

@@ -8,3 +8,11 @@ SimpleCov.start { add_filter "/spec/" }
 
 require "lita-digitalocean"
 require "lita/rspec"
+
+Lita.version_3_compatibility_mode = false
+
+RSpec.configure do |config|
+  config.before do
+    registry.register_hook(:trigger_route, Lita::Extensions::KeywordArguments)
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-digitalocean
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 2.0.0
 platform: ruby
 authors:
 - Jimmy Cuadra
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-08 00:00:00.000000000 Z
+date: 2014-10-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: lita-keyword-arguments
   requirement: !ruby/object:Gem::Requirement
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0.beta2
+        version: 3.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.0.beta2
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement