build_eval 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c9ed39e2bae3a1ef5d1ffd40242183287fae193a
-  data.tar.gz: bff2e6552ad2a28e5d1dbc0cea6b463cd6a82a3a
+  metadata.gz: 2c32dba8242b9791724f68774da3882fbff6d6f3
+  data.tar.gz: 9b0ac4ca5b0ecdad1bf4fa02cf7052fee8c3630a
 SHA512:
-  metadata.gz: b6006e6709284a157a71b6b48f09b0706f9112c89706e57de79faf3a40d3b46d8e998366c422420349f5d48e1e2beb6e0f6ff3cdddb59993efb5abe34bc862a6
-  data.tar.gz: 6c2978e0ce2095206a9086b018beef07e409f18dbf3c8e9afb5c7b8210942728b496ac04d8af499f3e9d54deb5df6435dfedef34f18d252661602afc1f1113b1
+  metadata.gz: af37b3850de9b567e633194aaef3f146460fcfd2d7e673d138c5a681099042dfee9431c36d61fbb2073321e75d547c428a226f1caeb44154ab698034b1acb3b6
+  data.tar.gz: ea266ac251292a56461437f5ece2ada4fb9c7643c47cf792fd39354d6bb4087fa71a27951a9702f17c266ec410dc72eb8ef3fb7037983acb7e23f914ef7e7064

data/lib/build_eval/error.rb

@@ -1,11 +1,7 @@
 module BuildEval
-
   class Error < ::StandardError
-
     def initialize(message)
       super(message)
     end
-
   end
-
 end

data/lib/build_eval/http.rb

@@ -1,7 +1,5 @@
 module BuildEval
-
   class Http
-
     def initialize(config)
       @username        = config[:username]
       @password        = config[:password]
@@ -20,11 +18,9 @@ module BuildEval
     private
 
     def ssl_options(uri)
-      ssl_options = { use_ssl: uri.scheme == "https" }
+      ssl_options = { use_ssl: uri.scheme == 'https' }
       ssl_options[:ssl_verify_mode] = @ssl_verify_mode if @ssl_verify_mode
       ssl_options
     end
-
   end
-
 end

data/lib/build_eval/monitor/base.rb

@@ -1,13 +1,9 @@
 module BuildEval
   module Monitor
-
     class Base
-
-      def +(monitor)
-        BuildEval::Monitor::Composite.new(self, monitor)
+      def +(other)
+        BuildEval::Monitor::Composite.new(self, other)
       end
-
     end
-
   end
 end

data/lib/build_eval/monitor/composite.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Monitor
-
     class Composite < BuildEval::Monitor::Base
-
       def initialize(*monitors)
         @monitors = monitors
       end
@@ -10,8 +8,6 @@ module BuildEval
       def evaluate
         BuildEval::Result::CompositeResult.new(@monitors.map(&:evaluate))
       end
-
     end
-
   end
 end

data/lib/build_eval/monitor/server.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Monitor
-
     class Server < BuildEval::Monitor::Base
-
       def initialize(args)
         @server      = args[:server]
         @build_names = args[:build_names]
@@ -12,7 +10,6 @@ module BuildEval
         build_results = @build_names.map { |build_name| @server.build_result(build_name) }
         BuildEval::Result::ServerResult.new(@server, build_results)
       end
-
     end
   end
 end

data/lib/build_eval/result/build_result.rb

@@ -1,18 +1,14 @@
 module BuildEval
   module Result
-
     class BuildResult
-
       class << self
-
         def create(args)
-          self.new(build_name: args[:build_name], status: BuildEval::Result::Status.find(args[:status_name]))
+          new(build_name: args[:build_name], status: BuildEval::Result::Status.find(args[:status_name]))
         end
 
         def indeterminate(build_name)
-          self.new(build_name: build_name, status: BuildEval::Result::Status::INDETERMINATE)
+          new(build_name: build_name, status: BuildEval::Result::Status::INDETERMINATE)
         end
-
       end
 
       attr_reader :build_name
@@ -34,8 +30,6 @@ module BuildEval
       def to_s
         "#{@build_name}: #{@status}"
       end
-
     end
-
   end
 end

data/lib/build_eval/result/composite_result.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Result
-
     class CompositeResult
-
       def initialize(results)
         @results = results
       end
@@ -18,8 +16,6 @@ module BuildEval
       def to_s
         @results.map(&:to_s).join("\n")
       end
-
     end
-
   end
 end

data/lib/build_eval/result/server_result.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Result
-
     class ServerResult
-
       def initialize(server, build_results)
         @server        = server
         @build_results = build_results
@@ -19,8 +17,6 @@ module BuildEval
       def to_s
         "#{@server}: #{@build_results.map(&:to_s).join(", ")}"
       end
-
     end
-
   end
 end

data/lib/build_eval/result/status.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Result
-
     class Status
-
       private
 
       def initialize(args)
@@ -13,26 +11,22 @@ module BuildEval
 
       public
 
-      SUCCESS       = self.new(severity: 0, symbol: :success!, description: "succeeded")
-      UNKNOWN       = self.new(severity: 1, symbol: :warning!, description: "unknown")
-      INDETERMINATE = self.new(severity: 2, symbol: :warning!, description: "indeterminate")
-      FAILURE       = self.new(severity: 3, symbol: :failure!, description: "failed")
-      ERROR         = self.new(severity: 4, symbol: :failure!, description: "errored")
+      SUCCESS       = new(severity: 0, symbol: :success!, description: 'succeeded')
+      UNKNOWN       = new(severity: 1, symbol: :warning!, description: 'unknown')
+      INDETERMINATE = new(severity: 2, symbol: :warning!, description: 'indeterminate')
+      FAILURE       = new(severity: 3, symbol: :failure!, description: 'failed')
+      ERROR         = new(severity: 4, symbol: :failure!, description: 'errored')
 
       class << self
-
         def find(name)
-          begin
-            self.const_get(name.upcase)
-          rescue NameError
-            raise "Build status '#{name}' is invalid"
-          end
+          const_get(name.upcase)
+        rescue NameError
+          raise "Build status '#{name}' is invalid"
         end
 
         def effective_status(statuses)
-          statuses.sort_by { |status| status.severity }.last
+          statuses.sort_by(&:severity).last
         end
-
       end
 
       attr_reader :severity
@@ -48,8 +42,6 @@ module BuildEval
       def to_s
         @description
       end
-
     end
-
   end
 end

data/lib/build_eval/server/cruise_control_response.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Server
-
     class CruiseControlResponse
-
       def initialize(raw_response)
         @raw_response = raw_response
       end
@@ -11,12 +9,10 @@ module BuildEval
         build_element = Nokogiri::XML(@raw_response.body).xpath(project_selector).first
         raise BuildEval::Server::InvalidSelectorError.new(@raw_response, project_selector) unless build_element
         BuildEval::Result::BuildResult.create(
-          build_name:  build_element.attribute("name").value.match(/[^\/]+$/)[0],
-          status_name: build_element.attribute("lastBuildStatus").value
+          build_name:  build_element.attribute('name').value.match(%r{[^\/]+$})[0],
+          status_name: build_element.attribute('lastBuildStatus').value
         )
       end
-
     end
-
   end
 end

data/lib/build_eval/server/decorator.rb

@@ -1,25 +1,19 @@
 module BuildEval
   module Server
-
     class Decorator
-
       def initialize(delegate)
         @delegate = delegate
       end
 
       def build_result(name)
-        begin
-          @delegate.build_result(name)
-        rescue Exception
-          BuildEval::Result::BuildResult.indeterminate(name)
-        end
+        @delegate.build_result(name)
+      rescue StandardError
+        BuildEval::Result::BuildResult.indeterminate(name)
       end
 
       def monitor(*build_names)
         BuildEval::Monitor::Server.new(server: @delegate, build_names: build_names.flatten)
       end
-
     end
-
   end
 end

data/lib/build_eval/server/invalid_selector_error.rb

@@ -1,13 +1,9 @@
 module BuildEval
   module Server
-
     class InvalidSelectorError < BuildEval::Error
-
       def initialize(response, selector)
         super("Build response did not match selector:\nResponse: #{response.message}\nSelector: #{selector}")
       end
-
     end
-
   end
 end

data/lib/build_eval/server/jenkins.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Server
-
     class Jenkins
-
       def initialize(args)
         @http     = BuildEval::Http.new(args)
         @base_uri = args[:uri]
@@ -16,8 +14,6 @@ module BuildEval
       def to_s
         "Jenkins server #{@base_uri}"
       end
-
     end
-
   end
 end

data/lib/build_eval/server/team_city.rb

@@ -1,8 +1,6 @@
 module BuildEval
   module Server
-
     class TeamCity
-
       def initialize(args)
         @http     = BuildEval::Http.new(args)
         @base_uri = args[:uri]
@@ -10,16 +8,14 @@ module BuildEval
 
       def build_result(name)
         response = @http.get("#{@base_uri}/httpAuth/app/rest/buildTypes/id:#{name}/builds")
-        build_element = Nokogiri::XML(response.body).xpath("//build").first
+        build_element = Nokogiri::XML(response.body).xpath('//build').first
         raise "Unexpected build response: #{response.message}" unless build_element
-        BuildEval::Result::BuildResult.create(build_name: name, status_name: build_element.attribute("status").value)
+        BuildEval::Result::BuildResult.create(build_name: name, status_name: build_element.attribute('status').value)
       end
 
       def to_s
         "TeamCity server #{@base_uri}"
       end
-
     end
-
   end
 end

data/lib/build_eval/server/travis_com.rb

@@ -0,0 +1,23 @@
+module BuildEval
+  module Server
+    class TravisCom
+      def initialize(args)
+        @username = args[:username]
+        Travis::Pro.github_auth(args[:github_token])
+      end
+
+      def build_result(name)
+        repo_string = "#{@username}/#{name}"
+        has_failed = Travis::Pro::Repository.find(repo_string).last_build.failed?
+        BuildEval::Result::BuildResult.create(
+          build_name:  repo_string,
+          status_name: has_failed ? 'Failure' : 'Success'
+        )
+      end
+
+      def to_s
+        "Travis CI Com #{@username}"
+      end
+    end
+  end
+end

data/lib/build_eval/server/travis_org.rb

@@ -0,0 +1,22 @@
+module BuildEval
+  module Server
+    class TravisOrg
+      def initialize(args)
+        @username = args[:username]
+      end
+
+      def build_result(name)
+        repo_string = "#{@username}/#{name}"
+        has_failed = Travis::Repository.find(repo_string).last_build.failed?
+        BuildEval::Result::BuildResult.create(
+          build_name:  repo_string,
+          status_name: has_failed ? 'Failure' : 'Success'
+        )
+      end
+
+      def to_s
+        "Travis CI Org #{@username}"
+      end
+    end
+  end
+end

data/lib/build_eval/version.rb

@@ -1,3 +1,3 @@
 module BuildEval
-  VERSION = "0.0.4".freeze
+  VERSION = '0.0.5'.freeze
 end

data/lib/build_eval.rb

@@ -2,6 +2,8 @@ require 'nokogiri'
 require 'uri'
 require 'net/http'
 require 'net/https'
+require 'travis'
+require 'travis/tools/github'
 require_relative 'build_eval/error'
 require_relative 'build_eval/http'
 require_relative 'build_eval/result/status'
@@ -12,16 +14,15 @@ require_relative 'build_eval/server/invalid_selector_error'
 require_relative 'build_eval/server/decorator'
 require_relative 'build_eval/server/cruise_control_response'
 require_relative 'build_eval/server/team_city'
-require_relative 'build_eval/server/travis'
+require_relative 'build_eval/server/travis_org'
+require_relative 'build_eval/server/travis_com'
 require_relative 'build_eval/server/jenkins'
 require_relative 'build_eval/monitor/base'
 require_relative 'build_eval/monitor/server'
 require_relative 'build_eval/monitor/composite'
 
 module BuildEval
-
   class << self
-
     def server(args)
       type_args = args.clone
       server_type = type_args.delete(:type)
@@ -31,14 +32,9 @@ module BuildEval
     private
 
     def server_class_for(type)
-      begin
-        BuildEval::Server.const_get(type.to_s)
-      rescue NameError
-        raise "Server type '#{type}' is invalid"
-      end
-
+      BuildEval::Server.const_get(type.to_s)
+    rescue NameError
+      raise "Server type '#{type}' is invalid"
     end
-
   end
-
 end

data/spec/lib/build_eval/error_spec.rb

@@ -1,21 +1,17 @@
 describe BuildEval::Error do
-
-  let(:message) { "some message" }
+  let(:message) { 'some message' }
 
   let(:error) { described_class.new(message) }
 
-  it "is a standard error" do
+  it 'is a standard error' do
     expect(error).to be_a(::StandardError)
   end
 
-  describe "#messsage" do
-
+  describe '#messsage' do
     subject { error.message }
 
-    it "returns the provided message" do
+    it 'returns the provided message' do
       expect(subject).to eql(message)
     end
-
   end
-
 end

data/spec/lib/build_eval/http_shared_context.rb

@@ -1,7 +1,5 @@
-shared_context "stubbed http interactions" do
-
+shared_context 'stubbed http interactions' do
   let(:http) { instance_double(BuildEval::Http) }
 
   before(:example) { allow(BuildEval::Http).to receive(:new).and_return(http) }
-
-end
+end

data/spec/lib/build_eval/http_spec.rb

@@ -1,102 +1,82 @@
 describe BuildEval::Http do
-
   let(:config) { {} }
 
   let(:http)   { described_class.new(config) }
 
-  shared_examples_for "a http method returning a response" do
-
-    let(:response_status) { [ "200", "OK" ] }
-    let(:response_body)   { "Some Response Body" }
+  shared_examples_for 'a http method returning a response' do
+    let(:response_status) { %w(200 OK) }
+    let(:response_body)   { 'Some Response Body' }
 
-    it "returns a response containing the response body" do
+    it 'returns a response containing the response body' do
       expect(subject.body).to eql(response_body)
     end
 
-    it "returns a response containing the response status" do
-      expect(subject.code).to eql("200")
+    it 'returns a response containing the response status' do
+      expect(subject.code).to eql('200')
     end
 
-    it "returns a response containing the response message" do
-      expect(subject.message).to eql("OK")
+    it 'returns a response containing the response message' do
+      expect(subject.message).to eql('OK')
     end
-
   end
 
-  describe "#get" do
-
-    let(:scheme)     { "http" }
-    let(:host)       { "a.host" }
-    let(:path)       { "some/path" }
+  describe '#get' do
+    let(:scheme)     { 'http' }
+    let(:host)       { 'a.host' }
+    let(:path)       { 'some/path' }
     let(:uri_string) { "#{scheme}://#{host}/#{path}" }
 
     subject { http.get(uri_string) }
 
-    context "when the uri is valid" do
-
+    context 'when the uri is valid' do
       let(:expected_request_uri) { uri_string }
 
       before(:example) do
         FakeWeb.register_uri(:get, expected_request_uri, status: response_status, body: response_body)
       end
 
-      context "and the uri contains a http scheme" do
-
-        let(:scheme) { "http" }
-
-        it_behaves_like "a http method returning a response"
+      context 'and the uri contains a http scheme' do
+        let(:scheme) { 'http' }
 
+        it_behaves_like 'a http method returning a response'
       end
 
-      context "and the uri contains a https scheme" do
-
-        let(:scheme) { "https" }
-
-        it_behaves_like "a http method returning a response"
+      context 'and the uri contains a https scheme' do
+        let(:scheme) { 'https' }
 
+        it_behaves_like 'a http method returning a response'
       end
 
-      context "and an ssl verification mode configuration option was established" do
-
+      context 'and an ssl verification mode configuration option was established' do
         let(:ssl_verification_mode) { OpenSSL::SSL::VERIFY_NONE }
         let(:config)                { { ssl_verify_mode: ssl_verification_mode } }
 
-        it_behaves_like "a http method returning a response"
-
+        it_behaves_like 'a http method returning a response'
       end
 
-      context "and partial authentication configuration options were established" do
-
-        let(:config)   { { username: "some_username" } }
-
-        it_behaves_like "a http method returning a response"
+      context 'and partial authentication configuration options were established' do
+        let(:config) { { username: 'some_username' } }
 
+        it_behaves_like 'a http method returning a response'
       end
 
-      context "and basic authentication configuration options were established" do
-
-        let(:username) { "some_username" }
-        let(:password) { "some_password" }
+      context 'and basic authentication configuration options were established' do
+        let(:username) { 'some_username' }
+        let(:password) { 'some_password' }
         let(:config)   { { username: username, password: password } }
 
         let(:expected_request_uri) { "#{scheme}://#{username}:#{password}@#{host}/#{path}" }
 
-        it_behaves_like "a http method returning a response"
-
+        it_behaves_like 'a http method returning a response'
       end
-
     end
 
-    context "when the uri is invalid" do
-
+    context 'when the uri is invalid' do
       before(:example) { FakeWeb.clean_registry }
 
-      it "raises an error" do
+      it 'raises an error' do
         expect { subject }.to raise_error(SocketError)
       end
-
     end
-
   end
-
 end

data/spec/lib/build_eval/monitor/base_spec.rb

@@ -1,29 +1,25 @@
 describe BuildEval::Monitor::Base do
-
   class BuildEval::Monitor::TestableBase < BuildEval::Monitor::Base
   end
 
   let(:base_monitor) { BuildEval::Monitor::TestableBase.new }
 
-  describe "#+" do
-
+  describe '#+' do
     let(:provided_monitor) { instance_double(BuildEval::Monitor::Base) }
 
     subject { base_monitor + provided_monitor }
 
-    it "creates a composite monitor combining the monitor with the provided monitor" do
+    it 'creates a composite monitor combining the monitor with the provided monitor' do
       expect(BuildEval::Monitor::Composite).to receive(:new).with(base_monitor, provided_monitor)
 
       subject
     end
 
-    it "returns the composite monitor" do
+    it 'returns the composite monitor' do
       composite_monitor = instance_double(BuildEval::Monitor::Composite)
       allow(BuildEval::Monitor::Composite).to receive(:new).and_return(composite_monitor)
 
       expect(subject).to eql(composite_monitor)
     end
-
   end
-
 end