pact_broker-client 1.75.2 → 1.75.4

data/spec/lib/pact_broker/client/cli/custom_thor_spec.rb

@@ -1,174 +0,0 @@
-require 'pact_broker/client/cli/custom_thor'
-
-module PactBroker::Client::CLI
-
-  class Delegate
-    def self.call options; end
-  end
-
-  class TestThor < CustomThor
-    def self.exit_on_failure?
-      false
-    end
-
-    desc 'ARGUMENT', 'This is the description'
-    def test_default(argument)
-      Delegate.call(argument: argument)
-    end
-
-    desc '', ''
-    method_option :multi, type: :array
-    def test_multiple_options
-      Delegate.call(options)
-    end
-
-    desc '', ''
-    method_option :broker_base_url, required: true, aliases: "-b"
-    method_option :broker_username, aliases: "-u"
-    method_option :broker_password, aliases: "-p"
-    method_option :broker_token, aliases: "-k"
-    def test_using_env_vars
-      Delegate.call(options)
-    end
-
-    desc '', ''
-    method_option :broker_base_url, required: true, aliases: "-b"
-    method_option :broker_username, aliases: "-u"
-    method_option :broker_password, aliases: "-p"
-    method_option :broker_token, aliases: "-k"
-    def test_pact_broker_client_options
-      Delegate.call(pact_broker_client_options)
-    end
-
-    desc '', ''
-    ignored_and_hidden_potential_options_from_environment_variables
-    def test_without_parameters
-      Delegate.call(options)
-    end
-
-    default_command :test_default
-  end
-
-  describe CustomThor do
-    subject { TestThor.new }
-
-    it "converts options that are specified multiple times into a single array" do
-      expect(Delegate).to receive(:call).with({'multi' => ['one', 'two']})
-      TestThor.start(%w{test_multiple_options --multi one --multi two})
-    end
-
-    context "with broker configuration in the environment variables" do
-      before do
-        ENV['PACT_BROKER_BASE_URL'] = 'http://foo'
-        ENV['PACT_BROKER_USERNAME'] = 'username'
-        ENV['PACT_BROKER_PASSWORD'] = 'password'
-        ENV['PACT_BROKER_TOKEN'] = 'token'
-      end
-
-      it "populates the base URL from the environment variables" do
-        expect(Delegate).to receive(:call) do | options |
-          expect(options.broker_base_url).to eq 'http://foo'
-        end
-        TestThor.start(%w{test_using_env_vars})
-      end
-
-      it "does not override a value specifed on the command line" do
-        expect(Delegate).to receive(:call) do | options |
-          expect(options.broker_base_url).to eq 'http://bar'
-        end
-        TestThor.start(%w{test_using_env_vars --broker-base-url http://bar})
-      end
-
-      it "allows commands to be called that don't use the environment variables" do
-        expect(Delegate).to receive(:call)
-        TestThor.start(%w{test_without_parameters})
-      end
-    end
-
-    it "removes trailing slashes from the broker base url when passed as an arg" do
-      expect(Delegate).to receive(:call) do | options |
-        expect(options[:pact_broker_base_url]).to eq 'http://bar'
-      end
-      TestThor.start(%w{test_pact_broker_client_options --broker-base-url http://bar/})
-    end
-
-    it "removes trailing slashes from the broker base url when passed as an env var" do
-      ENV['PACT_BROKER_BASE_URL'] = 'http://bar/'
-      expect(Delegate).to receive(:call) do | options |
-        expect(options[:pact_broker_base_url]).to eq 'http://bar'
-      end
-      TestThor.start(%w{test_pact_broker_client_options})
-    end
-
-    describe "when someone copy pastes from Word and uses an em dash instead of a normal dash" do
-      before do
-        allow(TestThor).to receive(:exit_with_error_code)
-      end
-
-      it "exits with an error message" do
-        expect($stdout).to receive(:puts).with(/contains an em dash/)
-        expect(TestThor).to receive(:exit_with_error_code)
-        TestThor.start(["test_default", "\u2014\u2014bar"])
-      end
-    end
-
-    describe ".handle_help" do
-      context "when the last argument is --help or -h" do
-        it "turns it into the form that Thor expects, which is a really odd one" do
-          expect(TestThor.handle_help(["foo", "--help"])).to eq ["help", "foo"]
-          expect(TestThor.handle_help(["foo", "-h"])).to eq ["help", "foo"]
-          expect(TestThor.handle_help(["-h"])).to eq ["help"]
-          expect(TestThor.handle_help(["--help"])).to eq ["help"]
-        end
-      end
-    end
-
-    describe ".turn_muliple_tag_options_into_array" do
-      it "turns '--tag foo --tag bar' into '--tag foo bar'" do
-        input = %w{--ignore this --tag foo --tag bar --wiffle --that}
-        output = %w{--ignore this --tag foo bar --wiffle --that }
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq output
-      end
-
-      it "turns '--tag foo bar --tag meep' into '--tag foo meep bar'" do
-        input = %w{--ignore this --tag foo bar --tag meep --wiffle --that}
-        output = %w{--ignore this --tag foo meep bar --wiffle --that}
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq output
-      end
-
-      it "turns '--tag foo --tag bar wiffle' into '--tag foo bar wiffle' which is silly" do
-        input = %w{--ignore this --tag foo --tag bar wiffle}
-        output = %w{--ignore this --tag foo bar wiffle}
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq output
-      end
-
-      it "maintains '--tag foo bar wiffle'" do
-        input = %w{--ignore this --tag foo bar wiffle --meep}
-        output = %w{--ignore this --tag foo bar wiffle --meep}
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq output
-      end
-
-      it "turns '-t foo -t bar' into '-t foo bar'" do
-        input = %w{--ignore this -t foo -t bar --meep --that 1 2 3}
-        output = %w{--ignore this -t foo bar --meep --that 1 2 3}
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq output
-      end
-
-      it "turns '--tag=foo --tag=bar' into '--tag foo bar'" do
-        input = %w{--ignore this --tag=foo --tag=bar --wiffle --that}
-        output = %w{--ignore this --tag foo bar --wiffle --that }
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq output
-      end
-
-      it "doesn't change anything when there are no duplicate options" do
-        input = %w{--ignore this --taggy foo --blah bar --wiffle --that}
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq input
-      end
-
-      it "return an empty array when given an empty array" do
-        input = []
-        expect(TestThor.turn_muliple_tag_options_into_array(input)).to eq input
-      end
-    end
-  end
-end

data/spec/lib/pact_broker/client/cli/version_selector_options_parser_spec.rb

@@ -1,97 +0,0 @@
-require 'pact_broker/client/cli/version_selector_options_parser'
-
-module PactBroker
-  module Client
-    module CLI
-      describe VersionSelectorOptionsParser do
-
-        TEST_CASES = [
-          [
-            ["--pacticipant", "Foo", "--version", "1.2.3"],
-            [{ pacticipant: "Foo", version: "1.2.3" } ]
-          ],[
-            ["-a", "Foo", "-e", "1.2.3"],
-            [{ pacticipant: "Foo", version: "1.2.3" } ]
-          ],[
-            ["--pacticipant", "Foo"],
-            [{ pacticipant: "Foo" } ]
-          ],[
-            ["--pacticipant", "Foo", "Bar"],
-            [{ pacticipant: "Bar" } ]
-          ],[
-            ["--pacticipant", "Foo", "--pacticipant", "Bar", "--version", "1.2.3"],
-            [{ pacticipant: "Foo" }, { pacticipant: "Bar", version: "1.2.3" } ]
-          ],[
-            ["--pacticipant", "Foo", "--wrong", "Bar", "--version", "1.2.3"],
-            [{ pacticipant: "Foo", version: "1.2.3" } ]
-          ],[
-            ["--pacticipant", "the-thing", "--version", "1.2.3"],
-            [{ pacticipant: "the-thing", version: "1.2.3" } ]
-          ],[
-            ["--version", "1.2.3"],
-            [{ pacticipant: nil, version: "1.2.3" } ]
-          ],[
-            ["--pacticipant", "Foo", "--latest", "--pacticipant", "Bar"],
-            [{ pacticipant: "Foo", latest: true }, { pacticipant: "Bar" } ]
-          ],[
-            ["--pacticipant", "Foo", "--latest", "prod", "--pacticipant", "Bar"],
-            [{ pacticipant: "Foo", latest: true, tag: "prod"}, { pacticipant: "Bar" } ]
-          ],[
-            ["--pacticipant", "Foo", "--all", "prod", "--pacticipant", "Bar"],
-            [{ pacticipant: "Foo", tag: "prod"}, { pacticipant: "Bar" } ]
-          ],[
-            ["--pacticipant=Foo", "--version=1.2.3"],
-            [{ pacticipant: "Foo", version: "1.2.3" } ]
-          ],[
-            ["--pacticipant=Foo=Bar", "--version", "1.2.3"],
-            [{ pacticipant: "Foo=Bar", version: "1.2.3" } ]
-          ],[
-            ["--pacticipant", "Foo=Bar", "--version", "1.2.3"],
-            [{ pacticipant: "Foo=Bar", version: "1.2.3" } ]
-          ],[
-            ["--ignore", "Foo", "--version", "1.2.3"],
-            [{ pacticipant: "Foo", version: "1.2.3", ignore: true }]
-          ],[
-            ["--ignore", "Foo", "--ignore", "Bar", "--version", "1.2.3"],
-            [{ pacticipant: "Foo", ignore: true }, { pacticipant: "Bar", version: "1.2.3", ignore: true }]
-          ],[
-            ["--ignore", "Foo", "--pacticipant", "Bar", "--version", "1.2.3"],
-            [{ pacticipant: "Foo", ignore: true }, { pacticipant: "Bar", version: "1.2.3" }]
-          ],[
-            ["--pacticipant", "Foo", "--version", "1", "--version", "2"],
-            [{ pacticipant: "Foo", version: "2" }]
-          ],[
-            ["--pacticipant", "Foo", "--version", "2", "--latest"],
-            [{ pacticipant: "Foo", version: "2", latest: true }]
-          ],[
-            ["--pacticipant", "Foo", "--version", "2", "--latest", "--latest"],
-            [{ pacticipant: "Foo", version: "2", latest: true }]
-          ],[
-            ["--version", "2"],
-            [{ pacticipant: nil, version: "2" }]
-          ],[
-            ["--pacticipant", "Foo", "--branch", "main"],
-            [{ pacticipant: "Foo", branch: "main", latest: true }]
-          ],[
-            ["--branch", "main"],
-            [{ pacticipant: nil, branch: "main", latest: true }]
-          ],[
-            ["--pacticipant", "Foo", "--main-branch", "--pacticipant", "Bar", "--version", "1"],
-            [{ pacticipant: "Foo", main_branch: true, latest: true }, { pacticipant: "Bar", version: "1" }]
-          ],[
-            ["--main-branch"],
-            [{ pacticipant: nil, main_branch: true, latest: true }]
-          ]
-        ]
-
-        TEST_CASES.each do | input, output |
-
-          it "parses #{input.join(' ')}" do
-            expect(VersionSelectorOptionsParser.call(input)).to eq output
-          end
-
-        end
-      end
-    end
-  end
-end

data/spec/lib/pact_broker/client/deployments/record_deployment_spec.rb

@@ -1,224 +0,0 @@
-require 'pact_broker/client/deployments/record_deployment'
-
-module PactBroker
-  module Client
-    module Deployments
-      describe RecordDeployment do
-        describe ".call" do
-          let(:broker_base_url) { "http://broker" }
-          let(:index_body_hash) do
-            {
-              _links: {
-                "pb:environments" => { href: "#{broker_base_url}/environments" },
-                "pb:pacticipant-version" => {
-                  href: "#{broker_base_url}/pacticipants/{pacticipant}/versions/{version}"
-                }
-              }
-            }
-          end
-          let(:version_body_hash) do
-            {
-              "_links" => {
-                "pb:record-deployment" => [
-                  {
-                    name: "test",
-                    href: "#{broker_base_url}/record-deployment"
-                  }
-                ]
-              }
-            }
-          end
-          let(:environments_body_hash) do
-            {
-              "_links" => {
-                "pb:environments" => [
-                  {
-                    "name" => "test",
-                    "href" => ""
-                  }
-                ]
-              }
-
-            }
-          end
-          let(:record_deployment_body_hash) do
-            { "some" => "response" }
-          end
-          let(:index_headers) { { "Content-Type" => "application/hal+json" } }
-          let!(:index_request) do
-            stub_request(:get, broker_base_url).to_return(status: 200, body: index_body_hash.to_json, headers: index_headers  )
-          end
-          let!(:version_request) do
-            stub_request(:get, broker_base_url + "/pacticipants/Foo/versions/1").to_return(status: 200, body: version_body_hash.to_json, headers: { "Content-Type" => "application/hal+json" }  )
-          end
-
-          let!(:environments_request) do
-            stub_request(:get, "http://broker/environments").
-              with(headers: { 'Accept'=>'application/hal+json' }).
-              to_return(status: 200, body: environments_body_hash.to_json, headers: { "Content-Type" => "application/hal+json" })
-          end
-          let!(:record_deployment_request) do
-            stub_request(:post, "http://broker/record-deployment").
-              to_return(status: 200, body: record_deployment_body_hash.to_json , headers: {})
-          end
-
-          let(:application_instance) { "blue" }
-
-          let(:params) do
-            {
-              pacticipant_name: "Foo",
-              version_number: "1",
-              environment_name: "test",
-              application_instance: application_instance
-            }
-          end
-
-          let(:options) do
-            {
-              output: output,
-              verbose: true
-            }
-          end
-          let(:output) { "text" }
-          let(:pact_broker_client_options) { { pact_broker_base_url: broker_base_url} }
-
-          subject { RecordDeployment.call(params, options, pact_broker_client_options) }
-
-          context "when the pb:environments relation does not exist" do
-            let(:index_body_hash) do
-              {
-                "_links" => {}
-              }
-            end
-
-            it "returns an error response" do
-              expect(subject.success).to be false
-              expect(subject.message).to include "does not support"
-            end
-
-            context "when the server is PactFlow" do
-              let(:index_headers) { { "Content-Type" => "application/hal+json", "Pactflow-Something" => "foo" } }
-
-              it "returns an error response" do
-                expect(subject.message).to include "permission"
-                expect(subject.message).to include "does not support"
-              end
-            end
-          end
-
-          context "when the specified version does not exist" do
-            let!(:version_request) do
-              stub_request(:get, broker_base_url + "/pacticipants/Foo/versions/1").to_return(status: 404)
-            end
-
-            it "returns an error response" do
-              expect(subject.success).to be false
-              expect(subject.message).to include "Foo version 1 not found"
-            end
-          end
-
-          context "when the pacticipant version does not support recording deployments" do
-            let(:version_body_hash) do
-              {
-                "_links" => {}
-              }
-            end
-            it "returns an error response" do
-              expect(subject.success).to be false
-              expect(subject.message).to include "does not support"
-            end
-          end
-
-          context "when the specified environment is not available for recording a deployment" do
-            let(:version_body_hash) do
-              {
-                "_links" => {
-                  "pb:record-deployment" => [
-                    {
-                      "name" => "prod",
-                      "href" => ""
-                    }
-                  ]
-                }
-              }
-            end
-
-            context "when the environment does not exist" do
-              let(:environments_body_hash) do
-                {
-                  "_links" => {
-                    "pb:environments" => [
-                      {
-                        "name" => "prod",
-                        "href" => ""
-                      },{
-                        "name" => "uat",
-                        "href" => ""
-                      }
-                    ]
-                  }
-                }
-              end
-
-              it "returns an error response" do
-                expect(subject.success).to be false
-                expect(subject.message).to include "No environment found with name 'test'. Available options: prod"
-              end
-
-              context "when the server is PactFlow" do
-                let(:index_headers) { { "Content-Type" => "application/hal+json", "X-Pactflow-Sha" => "123" } }
-
-                it "returns a customised message" do
-                  expect(subject.message).to include "Environment 'test' is not an available option for recording a deployment of Foo"
-                  expect(subject.message).to include "permissions"
-                end
-              end
-            end
-
-            context "when the environment does exist" do
-              it "returns an error response" do
-                expect(subject.success).to be false
-                expect(subject.message).to include "Environment 'test' is not an available option for recording a deployment of Foo. Available options: prod"
-              end
-            end
-          end
-
-          context "when the output is json" do
-            let(:output) { "json" }
-
-            it "returns the JSON payload" do
-              expect(JSON.parse(subject.message)).to eq record_deployment_body_hash
-            end
-          end
-
-          context "when the response headers contain PactFlow" do
-            before do
-              allow_any_instance_of(RecordDeployment).to receive(:check_if_command_supported)
-              allow_any_instance_of(RecordDeployment).to receive(:check_environment_exists)
-              allow_any_instance_of(RecordDeployment).to receive(:record_action)
-              allow_any_instance_of(RecordDeployment).to receive(:index_resource).and_return(index_resource)
-            end
-
-            let(:response_headers) { { "X-Pactflow-Sha" => "abc" } }
-
-            let(:index_resource) do
-              double('PactBroker::Client::Hal::Entity', response: double('response', headers: response_headers) )
-            end
-
-            it "indicates the API was PactFlow" do
-              expect(subject.message).to include "Recorded deployment of Foo version 1 to test environment (application instance blue) in PactFlow"
-            end
-
-            context "when application_instance is nil" do
-              let(:application_instance) { nil }
-
-              it "does not include the application_instance in the result message" do
-                expect(subject.message).to include "Recorded deployment of Foo version 1 to test environment in"
-              end
-            end
-          end
-        end
-      end
-    end
-  end
-end