Skip to content

Commit

Permalink
feat(cli): change cli option names to match pact-provider-verifier
Browse files Browse the repository at this point in the history
BREAKING CHANGE: nobody will have used the cli I released yesterday, so I'm not sorry.
  • Loading branch information
bethesque committed Oct 1, 2017
1 parent 8b2f8a5 commit 0682662
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
9 changes: 5 additions & 4 deletions lib/pact_broker/client/cli/publish.rb
Original file line number Diff line number Diff line change
Expand Up @@ -12,11 +12,12 @@ class PactPublicationError < ::Thor::Error; end

class Publish < CustomThor
desc 'PACT_DIRS_OR_FILES ...', "Publish pacts to a Pact Broker."
method_option :consumer_version, required: true, aliases: "-v", desc: "The consumer application version"
method_option :consumer_app_version, required: true, aliases: "-a", desc: "The consumer application version"
method_option :broker_base_url, required: true, aliases: "-b", desc: "The base URL of the Pact Broker"
method_option :username, aliases: "-u", desc: "Basic auth username for Pact Broker"
method_option :password, aliases: "-p", desc: "Basic auth password for Pact Broker"
method_option :broker_username, aliases: "-n", desc: "Pact Broker basic auth username"
method_option :broker_password, aliases: "-p", desc: "Pact Broker basic auth password"
method_option :tag, aliases: "-t", type: :array, banner: "TAG", desc: "Tag name for consumer version. Can be specified multiple times."
method_option :verbose, aliases: "-v", desc: "Verbose output", :required => false

def broker(*pact_files)
validate(pact_files)
Expand Down Expand Up @@ -45,7 +46,7 @@ def publish_pacts pact_files
PactBroker::Client::PublishPacts.call(
options[:broker_base_url],
file_list(pact_files),
options[:consumer_version],
options[:consumer_app_version],
tags,
pact_broker_client_options
)
Expand Down
2 changes: 1 addition & 1 deletion spec/lib/pact_broker/client/cli/publish_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ module PactBroker::Client::CLI
let(:minimum_valid_options) do
{
pact_dir: 'spec/pacts',
consumer_version: '1.2.3',
consumer_app_version: '1.2.3',
broker_base_url: 'http://pact-broker'
}
end
Expand Down

0 comments on commit 0682662

Please sign in to comment.