Merge pull request #167 from pact-foundation/feat/add_env_vars_verify

feat: added env vars for broker verify

Author: elliottmurray

pact/cli/verify.py

@@ -36,10 +36,12 @@
     help='URL to send PUT requests to setup a given provider state.')
 @click.option(
     'username', '--pact-broker-username',
+    envvar='PACT_BROKER_USERNAME',
     help='Username for Pact Broker basic authentication.')
 @click.option(
     'broker_base_url', '--pact-broker-url',
     default='',
+    envvar='PACT_BROKER_BASE_URL',
     help='Base URl for the Pact Broker instance to publish pacts to.')
 @click.option(
     'consumer_version_tag', '--consumer-version-tag',

tests/cli/test_verify.py

@@ -183,10 +183,12 @@ def test_failed_verification(self, mock_isfile, mock_wrapper):
                                         timeout=30,
                                         verbose=False)
 
-    @patch.dict(os.environ, {'PACT_BROKER_PASSWORD': 'pwd'})
+    @patch.dict(os.environ, {'PACT_BROKER_PASSWORD': 'pwd',
+                             'PACT_BROKER_USERNAME': 'broker_user',
+                             'PACT_BROKER_BASE_URL': 'http://broker/'})
     @patch("pact.verify_wrapper.VerifyWrapper.call_verify")
     @patch("pact.verify_wrapper.isfile", return_value=True)
-    def test_password_from_env_var(self, mock_isfile, mock_wrapper):
+    def test_broker_creds_from_env_var(self, mock_isfile, mock_wrapper):
         mock_wrapper.return_value = 0, None
 
         result = self.runner.invoke(verify.main, self.simple_pact_opts)
@@ -197,6 +199,8 @@ def test_password_from_env_var(self, mock_isfile, mock_wrapper):
                                 provider='provider',
                                 provider_base_url='http://localhost',
                                 broker_password='pwd',
+                                broker_username='broker_user',
+                                broker_url='http://broker/',
                                 timeout=30,
                                 verbose=False)