lita-travis 0.2.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 281766fc85013339455d961b7e50a8d474652096
-  data.tar.gz: 6344a3fdf756025ef3c61b0320b67cf4bfa3ac28
+  metadata.gz: 88c770748392b7b5086b6c89694bce642cbc8128
+  data.tar.gz: 9787a09aa44ad5ebe43ee32d5ef4819db5926e25
 SHA512:
-  metadata.gz: e58c006f89c8bc6fa830e06c89761a63e512e12c7df7767868c6494f67d96244ff729e26f757dcf1a65ba1924921a91972596b5526e9426aa8729983303ae8f9
-  data.tar.gz: bdafa443957cad73565a555b140f9d8e15f86b85990860379b48dbecf75b9254580917be1b7bf1e2fcacc5031d837f26e1e57a6085e33511af9d5f1f7f970108
+  metadata.gz: 1e0d7dbcd66792787ec222046a088f350d0b76ab36be3b37e1d988facddeb2efe9ce0b06a65ad1097f5f1467290291c324f2b5a1d5b649015a984d27e195ac00
+  data.tar.gz: dcefccc5175a2385b09d5833cd3b57cf6898cfbe35c348ccb093a79a37dd3514681558009cbf6131f695308ed77e3a1ea9ec5ef4d8852666b844c096907ef597

data/lib/lita/handlers/travis.rb

@@ -6,11 +6,9 @@ module Lita
   module Handlers
     # Provides Travis CI webhooks for Lita.
     class Travis < Handler
-      def self.default_config(config)
-        config.token = nil
-        config.repos = {}
-        config.default_rooms = nil
-      end
+      config :token, type: String, required: true
+      config :repos, type: Hash, default: {}
+      config :default_rooms, type: [Array, String]
 
       http.post "/travis", :receive
 
@@ -56,8 +54,8 @@ module Lita
       end
 
       def rooms_for_repo(repo)
-        rooms = Lita.config.handlers.travis.repos[repo]
-        default_rooms = Lita.config.handlers.travis.default_rooms
+        rooms = config.repos[repo]
+        default_rooms = config.default_rooms
 
         if rooms
           Array(rooms)
@@ -70,14 +68,7 @@ module Lita
       end
 
       def validate_repo(repo, auth_hash)
-        token = Lita.config.handlers.travis.token
-
-        unless token
-          Lita.logger.warn(t("no_token"))
-          return
-        end
-
-        unless Digest::SHA2.hexdigest("#{repo}#{token}") == auth_hash
+        unless Digest::SHA2.hexdigest("#{repo}#{config.token}") == auth_hash
           Lita.logger.warn(t("auth_failed"), repo: repo)
           return
         end

data/lita-travis.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-travis"
-  spec.version       = "0.2.0"
+  spec.version       = "1.0.0"
   spec.authors       = ["Jimmy Cuadra"]
   spec.email         = ["jimmy@jimmycuadra.com"]
   spec.description   = %q{A Lita handler for receiving notifications from Travis CI.}
@@ -14,11 +14,11 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_runtime_dependency "lita", ">= 2.5"
+  spec.add_runtime_dependency "lita", ">= 4.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/locales/en.yml

@@ -10,6 +10,3 @@ en:
         no_room_configured: >-
           Notification from Travis CI for %{repo} ignored because no rooms were specified in the
           config.repos hash, and no default rooms were specified in config.default_rooms.
-        no_token: >-
-          Notification from Travis CI could not be validated because Lita.config.handlers.token
-          is not set.

data/spec/lita/handlers/travis_spec.rb

@@ -1,17 +1,7 @@
 require "spec_helper"
 
 describe Lita::Handlers::Travis, lita_handler: true do
-  it { routes_http(:post, "/travis").to(:receive) }
-
-  describe ".default_config" do
-    it "sets token to nil" do
-      expect(Lita.config.handlers.travis.token).to be_nil
-    end
-
-    it "sets repos to an empty hash" do
-      expect(Lita.config.handlers.travis.repos).to eq({})
-    end
-  end
+  it { is_expected.to route_http(:post, "/travis").to(:receive) }
 
   describe "#receive" do
     let(:request) do
@@ -54,10 +44,14 @@ describe Lita::Handlers::Travis, lita_handler: true do
       JSON
     end
 
+    before do
+      registry.config.handlers.travis.repos = {}
+    end
+
     context "happy path" do
       before do
-        Lita.config.handlers.travis.token = "abc123"
-        Lita.config.handlers.travis.repos["foo/bar"] = "#baz"
+        registry.config.handlers.travis.token = "abc123"
+        registry.config.handlers.travis.repos["foo/bar"] = "#baz"
         allow(request).to receive(:env).and_return(valid_env)
         allow(params).to receive(:[]).with("payload").and_return(valid_payload)
       end
@@ -71,25 +65,10 @@ describe Lita::Handlers::Travis, lita_handler: true do
       end
     end
 
-    context "with a missing token" do
-      before do
-        Lita.config.handlers.travis.repos["foo/bar"] = "#baz"
-        allow(request).to receive(:env).and_return(valid_env)
-        allow(params).to receive(:[]).with("payload").and_return(valid_payload)
-      end
-
-      it "logs a warning that the token is not set" do
-        expect(Lita.logger).to receive(:warn) do |warning|
-          expect(warning).to include("token is not set")
-        end
-        subject.receive(request, response)
-      end
-    end
-
     context "with an invalid authorization header" do
       before do
-        Lita.config.handlers.travis.token = "abc123"
-        Lita.config.handlers.travis.repos["foo/bar"] = "#baz"
+        registry.config.handlers.travis.token = "abc123"
+        registry.config.handlers.travis.repos["foo/bar"] = "#baz"
         allow(request).to receive(:env).and_return(invalid_env)
         allow(params).to receive(:[]).with("payload").and_return(valid_payload)
       end
@@ -104,8 +83,8 @@ describe Lita::Handlers::Travis, lita_handler: true do
 
     context "with only config.default_rooms set" do
       before do
-        Lita.config.handlers.travis.token = "abc123"
-        Lita.config.handlers.travis.default_rooms = "#default"
+        registry.config.handlers.travis.token = "abc123"
+        registry.config.handlers.travis.default_rooms = "#default"
         allow(request).to receive(:env).and_return(valid_env)
         allow(params).to receive(:[]).with("payload").and_return(valid_payload)
       end
@@ -121,7 +100,8 @@ describe Lita::Handlers::Travis, lita_handler: true do
 
     context "without setting a value for the repo in config.repos and no default" do
       before do
-        Lita.config.handlers.travis.token = "abc123"
+        registry.config.handlers.travis.token = "abc123"
+        allow(registry.config.handlers.travis).to receive(:default_rooms).and_return(nil)
         allow(request).to receive(:env).and_return(valid_env)
         allow(params).to receive(:[]).with("payload").and_return(valid_payload)
       end

data/spec/spec_helper.rb

@@ -8,3 +8,5 @@ SimpleCov.start { add_filter "/spec/" }
 
 require "lita-travis"
 require "lita/rspec"
+
+Lita.version_3_compatibility_mode = false

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-travis
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Jimmy Cuadra
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-27 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: '2.5'
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '2.5'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +58,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
@@ -141,4 +141,3 @@ summary: A Lita handler for receiving notifications from Travis CI.
 test_files:
 - spec/lita/handlers/travis_spec.rb
 - spec/spec_helper.rb
-has_rdoc: