Factoring out 'dry' run. This doesn't really make sense to have as we don't have a concept of a pre-flight request. Plugin tests should mock out their particular dependencies.

This commit is contained in:
kevgliss 2015-07-08 16:40:46 -07:00
parent 5156371913
commit 1a2e437b33
1 changed files with 0 additions and 6 deletions

View File

@ -306,9 +306,6 @@ class CloudCAPlugin(IssuerPlugin):
:param data: :param data:
:return: :return:
""" """
if self.dry_run:
endpoint += '?dry_run=1'
data = dumps(dict(data.items() + get_auth_data(data['caName']).items())) data = dumps(dict(data.items() + get_auth_data(data['caName']).items()))
# we set a low timeout, if cloudca is down it shouldn't bring down # we set a low timeout, if cloudca is down it shouldn't bring down
@ -323,9 +320,6 @@ class CloudCAPlugin(IssuerPlugin):
:param endpoint: :param endpoint:
:return: :return:
""" """
if self.dry_run:
endpoint += '?dry_run=1'
response = self.session.get(self.url + endpoint, timeout=10, verify=self.ca_bundle) response = self.session.get(self.url + endpoint, timeout=10, verify=self.ca_bundle)
return process_response(response) return process_response(response)