Skip to content

Commit

Permalink
Merge pull request #35 from pact-foundation/fix-broker-credentials
Browse files Browse the repository at this point in the history
Fix the use of broker credentials
  • Loading branch information
mefellows authored Jun 27, 2017
2 parents 41fa3df + d60f37f commit 69caa40
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions pact/test/test_verify.py
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ def test_password_from_env_var(self):
self.mock_Popen.return_value.returncode = 0
result = self.runner.invoke(verify.main, self.default_opts)
self.assertEqual(result.exit_code, 0)
self.assertProcess(*self.default_call + ['--pact-broker-password=pwd'])
self.assertProcess(*self.default_call + ['--broker-password=pwd'])
self.mock_Popen.return_value.communicate.assert_called_once_with(
timeout=30)

Expand All @@ -135,8 +135,8 @@ def test_all_options(self):
'--pact-urls=./pacts/consumer-provider.json',
'--provider-states-url=http=//localhost/provider-states',
'--provider-states-setup-url=http://localhost/provider-states/set',
'--pact-broker-username=user',
'--pact-broker-password=pass')
'--broker-username=user',
'--broker-password=pass')
self.mock_Popen.return_value.communicate.assert_called_once_with(
timeout=60)

Expand Down
4 changes: 2 additions & 2 deletions pact/verify.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,8 +74,8 @@ def main(base_url, pact_urls, states_url, states_setup_url, username,
'--pact-urls': ','.join(pact_urls),
'--provider-states-url': states_url,
'--provider-states-setup-url': states_setup_url,
'--pact-broker-username': username,
'--pact-broker-password': password
'--broker-username': username,
'--broker-password': password
}

command = [VERIFIER_PATH] + [
Expand Down

0 comments on commit 69caa40

Please sign in to comment.