hive-messages 1.0.1

data/spec/lib/hive/messages_spec.rb

@@ -0,0 +1,30 @@
+require "spec_helper"
+
+describe Hive::Messages do
+
+  describe "class methods" do
+
+    describe ".configure" do
+
+      let(:base_path) { "http://localhost:3000" }
+
+      before(:each) do
+        Hive::Messages.configure do |config|
+          config.base_path = base_path
+        end
+      end
+
+      it "creates a Configuration instance" do
+        expect(Hive::Messages.configuration).to be_instance_of(Hive::Messages::Configuration)
+      end
+
+      it "set the Hive::Paths.base" do
+        expect(Hive::Paths.base).to eq(base_path)
+      end
+
+      it "set the base_path on the Configuration" do
+        expect(Hive::Messages.configuration.base_path).to eq(base_path)
+      end
+    end
+  end
+end

data/spec/lib/hive/paths/artifacts_spec.rb

@@ -0,0 +1,22 @@
+require "spec_helper"
+
+describe Hive::Paths::Artifacts do
+
+  describe "class methods" do
+
+    let(:base_path) { "http://hive.bbc" }
+
+    before(:each) do
+      Hive::Paths.base=base_path
+    end
+
+    describe ".create_url" do
+
+      let(:job_id) { 99 }
+
+      it "generates a valid create artifact url" do
+        expect(Hive::Paths::Artifacts.create_url(job_id)).to eq "#{base_path}/api/jobs/#{job_id.to_s}/artifacts"
+      end
+    end
+  end
+end

data/spec/lib/hive/paths/jobs_spec.rb

@@ -0,0 +1,49 @@
+require "spec_helper"
+
+describe Hive::Paths::Jobs do
+
+  describe "class methods" do
+
+    let(:base_path) { "http://hive.bbc" }
+    let(:job_id) { 99 }
+
+    before(:each) do
+      Hive::Paths.base = base_path
+    end
+
+    describe ".start_url" do
+
+      it "generates a valid job start url" do
+        expect(Hive::Paths::Jobs.start_url(job_id)).to eq "#{base_path}/api/jobs/#{job_id.to_s}/start"
+      end
+    end
+
+    describe ".update_results_url" do
+
+      it "generates a valid update counts url" do
+        expect(Hive::Paths::Jobs.update_results_url(job_id)).to eq "#{base_path}/api/jobs/#{job_id.to_s}/update_results"
+      end
+    end
+
+    describe ".report_artifacts_url" do
+
+      it "generates a valid report artifacts url" do
+        expect(Hive::Paths::Jobs.report_artifacts_url(job_id)).to eq "#{base_path}/api/jobs/#{job_id.to_s}/report_artifacts"
+      end
+    end
+
+    describe ".end_url" do
+
+      it "generates a valid job end url" do
+        expect(Hive::Paths::Jobs.end_url(job_id)).to eq "#{base_path}/api/jobs/#{job_id.to_s}/end"
+      end
+    end
+
+    describe ".error_url" do
+
+      it "generates a valid job error url" do
+        expect(Hive::Paths::Jobs.error_url(job_id)).to eq "#{base_path}/api/jobs/#{job_id.to_s}/error"
+      end
+    end
+  end
+end

data/spec/lib/hive/paths/queues_spec.rb

@@ -0,0 +1,35 @@
+require "spec_helper"
+
+describe Hive::Paths::Queues do
+
+  describe "class methods" do
+
+    let(:base_path) { "http://hive.bbc" }
+
+    before(:each) do
+      Hive::Paths.base=base_path
+    end
+
+    describe ".job_reservation_url" do
+
+      let(:queues) { ["queue_one", "queue_two"] }
+
+      let(:job_reservation_url) { Hive::Paths::Queues.job_reservation_url(queues) }
+
+      context "single queue name provided" do
+
+        let(:queues) { "queue_one" }
+
+        it "adds the single queue to the reservation path" do
+          expect(job_reservation_url).to eq "#{base_path}/api/queues/queue_one/jobs/reserve"
+        end
+      end
+
+      context "multiple queue names provided" do
+        it "adds the provided queues to the reservation path" do
+          expect(job_reservation_url).to eq "#{base_path}/api/queues/queue_one,queue_two/jobs/reserve"
+        end
+      end
+    end
+  end
+end

data/spec/lib/hive/paths_spec.rb

@@ -0,0 +1,10 @@
+require "spec_helper"
+
+describe Hive::Paths do
+
+  it { should be }
+  it "has an attribute called 'base'" do
+    Hive::Paths.base = :base
+    expect(Hive::Paths.base).to eq :base
+  end
+end

data/spec/lib/hive/representers/artifact_representer_spec.rb

@@ -0,0 +1,49 @@
+require "spec_helper"
+
+describe Hive::Representers::ArtifactRepresenter do
+
+  class TestArtifact
+    include Virtus.model
+
+    attribute :artifact_id
+    attribute :job_id
+    attribute :asset_file_name
+    attribute :asset_content_type
+    attribute :asset_file_size
+  end
+
+  let(:upstream_artifact) do
+    artifact = TestArtifact.new(artifact_attributes)
+    artifact.extend(Hive::Representers::ArtifactRepresenter)
+    artifact
+  end
+
+  let(:artifact_attributes) do
+    {
+        artifact_id:        321,
+        job_id:             123,
+        asset_file_name:    "screenshot1.png",
+        asset_content_type: "image/png",
+        asset_file_size:    2300
+    }
+  end
+
+  describe "serialization " do
+
+    let(:downstream_artifact) do
+      artifact = TestArtifact.new
+      artifact.extend(Hive::Representers::ArtifactRepresenter)
+      artifact.from_json(upstream_artifact.to_json)
+      artifact
+    end
+
+
+    subject { downstream_artifact }
+
+    its(:artifact_id)              { should eq artifact_attributes[:artifact_id] }
+    its(:job_id)              { should eq artifact_attributes[:job_id] }
+    its(:asset_file_name)     { should eq artifact_attributes[:asset_file_name] }
+    its(:asset_content_type)  { should eq artifact_attributes[:asset_content_type] }
+    its(:asset_file_size)     { should eq artifact_attributes[:asset_file_size] }
+  end
+end

data/spec/lib/hive/representers/job_representer_spec.rb

@@ -0,0 +1,96 @@
+require "spec_helper"
+
+describe Hive::Representers::JobRepresenter do
+
+  class TestJob
+    include Virtus.model
+
+    attribute :command
+    attribute :job_id
+    attribute :repository
+    attribute :execution_directory
+    attribute :target
+    attribute :execution_variables
+    attribute :reservation_details
+    attribute :device_id
+    attribute :running_count
+    attribute :passed_count
+    attribute :failed_count
+    attribute :state
+    attribute :errored_count
+    attribute :result
+    attribute :exit_value
+    attribute :message
+    attribute :result_details
+
+    attribute :extra
+  end
+
+  let(:upstream_job) do
+    job = TestJob.new(job_attributes)
+    job.extend(Hive::Representers::JobRepresenter)
+    job
+  end
+
+  let(:target) { { "build" => "http://hive/download/99" } }
+  let(:execution_variables) { { "tests" => [] } }
+  let(:reservation_details) { { "hive" => "99", "pid" => "1024" } }
+
+
+  let(:job_attributes) do
+    {
+        command:             "cmd",
+        job_id:              99,
+        repository:          "svn://",
+        execution_directory: ".",
+        target:              target,
+        execution_variables: execution_variables,
+        reservation_details: reservation_details,
+        device_id:           23,
+        running_count:       4,
+        passed_count:        3,
+        failed_count:        2,
+        errored_count:       1,
+        state:               "running",
+        extra:               "thing",
+        result:              "test result",
+        exit_value:          0,
+        message:             "Hello, mum",
+        result_details:      "Blah!"
+    }
+  end
+
+  describe "serialization " do
+
+    let(:downstream_job) do
+      job = TestJob.new
+      job.extend(Hive::Representers::JobRepresenter)
+      job.from_json(upstream_job.to_json)
+      job
+    end
+
+
+    subject { downstream_job }
+
+    its(:command)             { should eq job_attributes[:command] }
+    its(:job_id)              { should eq job_attributes[:job_id] }
+    its(:repository)          { should eq job_attributes[:repository] }
+    its(:execution_directory) { should eq job_attributes[:execution_directory] }
+    its(:target)              { should eq job_attributes[:target] }
+    its(:execution_variables) { should eq execution_variables }
+    its(:reservation_details) { should eq reservation_details }
+    its(:device_id)           { should eq job_attributes[:device_id] }
+    its(:running_count)       { should eq job_attributes[:running_count] }
+    its(:passed_count)        { should eq job_attributes[:passed_count] }
+    its(:failed_count)        { should eq job_attributes[:failed_count] }
+    its(:errored_count)       { should eq job_attributes[:errored_count] }
+    its(:state)               { should eq job_attributes[:state] }
+    its(:result)              { should eq job_attributes[:result] }
+    its(:exit_value)          { should eq job_attributes[:exit_value] }
+    its(:message)             { should eq job_attributes[:message] }
+    its(:result_details)      { should eq job_attributes[:result_details] }
+
+    # Attribute that should be ignored
+    its(:extra)               { should be_nil }
+  end
+end

data/spec/lib/roar/transport/net_http/request_patch_spec.rb

@@ -0,0 +1,84 @@
+require "spec_helper"
+
+# Monkey patch == so that our expectations below work correctly
+class OpenSSL::X509::Certificate
+  def ==(other)
+    self.to_s == other.to_s
+  end
+end
+
+class OpenSSL::PKey::RSA
+  def ==(other)
+    self.to_s == other.to_s
+  end
+end
+
+describe Roar::Transport::NetHTTP::Request do
+
+  describe "class methods" do
+
+    describe ".new" do
+
+      let(:options) { { uri: "http://www.bbc.co.uk" } }
+      let(:net_http_double) { double(Net::HTTP, "use_ssl=".to_sym => true, "cert=".to_sym => true, "key=".to_sym => true, "verify_mode=".to_sym => true) }
+
+      before(:each) do
+        Hive::Messages.configure do |config|
+          configuration_options.each do |option_key, option_value|
+            config.send("#{option_key}=", option_value)
+          end
+        end
+        Net::HTTP.stub(new: net_http_double)
+        Roar::Transport::NetHTTP::Request.new(options)
+      end
+
+      context "when Hive::Messages is configured to use a client pem_file" do
+
+        let(:pem_file) { File.expand_path("spec/fixtures/sample.pem", Hive::Messages.root) }
+
+        let(:pem) { File.read(pem_file) }
+        let(:cert) { OpenSSL::X509::Certificate.new(pem) }
+        let(:key) { OpenSSL::PKey::RSA.new(pem) }
+
+        let(:configuration_options) { { pem_file: pem_file, ssl_verify_mode: OpenSSL::SSL::VERIFY_NONE } }
+
+        it "turned ssl on" do
+          expect(net_http_double).to have_received(:use_ssl=).with(true)
+        end
+
+        it "provided a client cert" do
+          expect(net_http_double).to have_received(:cert=).with(cert)
+        end
+
+        it "provided a client key" do
+          expect(net_http_double).to have_received(:key=).with(key)
+        end
+
+        it "set the ssl verify mode" do
+          expect(net_http_double).to have_received(:verify_mode=).with(configuration_options[:ssl_verify_mode])
+        end
+      end
+
+      context "when Hive::Messages is NOT configured to use a client pem_file" do
+
+        let(:configuration_options) { {} }
+
+        it "did not turn ssl on" do
+          expect(net_http_double).to_not have_received(:use_ssl=)
+        end
+
+        it "did not provide a cert" do
+          expect(net_http_double).to_not have_received(:cert=)
+        end
+
+        it "did not provide a key" do
+          expect(net_http_double).to_not have_received(:key=)
+        end
+
+        it "did not set an ssl verify mode" do
+          expect(net_http_double).to_not have_received(:verify_mode=)
+        end
+      end
+    end
+  end
+end

data/spec/lib/virtus/attribute/execution_variables_spec.rb

@@ -0,0 +1,60 @@
+require "spec_helper"
+
+describe Virtus::Attribute::ExecutionVariables do
+
+  let(:model_klass) do
+    klass = Class.new do
+      include Virtus.model
+
+      attribute :execution_variables, Virtus::Attribute::ExecutionVariables
+    end
+  end
+
+  let(:model) { model_klass.new(execution_variables: values) }
+  let(:execution_variables) { model.execution_variables }
+
+  context "values are empty" do
+
+    let(:values) { {} }
+
+    it "returned nil for the execution_variables" do
+      expect(execution_variables).to be_nil
+    end
+  end
+
+  context "values just contain job_ib and version" do
+
+    let(:job_id) { 99 }
+    let(:version) { "one" }
+    let(:values) { { job_id: job_id, version: version } }
+
+    subject { execution_variables }
+
+    it { should be_a(Hive::Messages::ExecutionVariablesBase) }
+    its(:job_id) { should eq job_id }
+    its(:version) { should eq version }
+  end
+
+  context "values contain extra attributes" do
+
+    let(:job_id)  { 99 }
+    let(:version) { "one" }
+    let(:foo)     { "foo_value" }
+    let(:values)  { { job_id: job_id, version: version, foo: foo } }
+    subject { execution_variables }
+
+    it { should be_a(Hive::Messages::ExecutionVariablesBase) }
+    its(:job_id) { should eq job_id }
+    its(:version) { should eq version }
+
+    it "created a new 'foo' attribute on the anonymous class that was not set on the base ExecutionVariables class" do
+      attributes = execution_variables.class.attribute_set.collect(&:name)
+      expect(attributes).to include(:foo)
+    end
+
+    it "set the value of the 'foo' variable" do
+      expect(execution_variables.foo).to eq foo
+    end
+  end
+end
+

data/spec/spec_helper.rb

@@ -0,0 +1,13 @@
+require "rubygems"
+require "bundler/setup"
+require "rspec"
+
+require "hive/messages"
+
+Dir["./spec/support/**/*.rb"].each {|f| require f}
+
+RSpec.configure do |config|
+  config.color_enabled = true
+  config.filter_run focus: true
+  config.run_all_when_everything_filtered = true
+end