Skip to content

Commit

Permalink
Merge pull request #62 from dhoomakethu/master
Browse files Browse the repository at this point in the history
 Add support to publish verification result to pact broker
  • Loading branch information
matthewbalvanz-wf authored Nov 20, 2017
2 parents 08cd592 + c27a7a9 commit 7c7bc7d
Showing 1 changed file with 24 additions and 4 deletions.
28 changes: 24 additions & 4 deletions pact/verify.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,19 @@
help='The duration in seconds we should wait to confirm verification'
' process was successful. Defaults to 30.',
type=int)
def main(base_url, pact_url, pact_urls, states_url, states_setup_url, username,
password, timeout):
@click.option(
'provider_app_version', '-a', '--provider-app-version',
help='The provider application version, '
'required for publishing verification results'
)
@click.option(
'publish_verification_results', '-r', '--publish-verification-results',
default=False,
help='Publish verification results to the broker',
is_flag=True)
def main(base_url, pact_url, pact_urls, states_url,
states_setup_url, username, password, timeout, provider_app_version,
publish_verification_results):
"""
Verify one or more contracts against a provider service.
Expand Down Expand Up @@ -96,10 +107,19 @@ def main(base_url, pact_url, pact_urls, states_url, states_setup_url, username,
'--broker-username': username,
'--broker-password': password
}

command = [VERIFIER_PATH] + [
'{}={}'.format(k, v) for k, v in options.items() if v]

if publish_verification_results:
if not provider_app_version:
click.echo(
error
+ 'Provider application version is required '
+ 'to publish verification results to broker'
)
raise click.Abort()
command.extend(["--provider-app-version",
provider_app_version,
"--publish-verification-results"])
p = subprocess.Popen(command)
p.communicate(timeout=timeout)
sys.exit(p.returncode)
Expand Down

0 comments on commit 7c7bc7d

Please sign in to comment.