Skip to content

Commit 63f0e3e

Browse files
Merge pull request #142 from elliottmurray/ssl_verify
Allowed https communication to mock. Didn't fix tests
2 parents 60c9f5a + cd43bd0 commit 63f0e3e

File tree

4 files changed

+23
-11
lines changed

4 files changed

+23
-11
lines changed

.travis.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,12 +13,13 @@ rvm: "2.2"
1313

1414
sudo: false
1515

16-
install: pip install -r requirements_dev.txt
16+
install:
17+
- pip install -r requirements_dev.txt
1718

1819
script:
1920
- flake8
2021
- pydocstyle pact
21-
- tox
22+
- tox -e test
2223
- if [[ $TRAVIS_PYTHON_VERSION == "3.5" ]]; then make package && pip install ./dist/pact-python-*.tar.gz && make e2e; fi
2324

2425
before_deploy:

pact/pact.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -201,12 +201,13 @@ def setup(self):
201201
"""Configure the Mock Service to ready it for a test."""
202202
try:
203203
resp = requests.delete(
204-
self.uri + '/interactions', headers=self.HEADERS)
204+
self.uri + '/interactions', headers=self.HEADERS, verify=False)
205205

206206
assert resp.status_code == 200, resp.text
207207
resp = requests.put(
208208
self.uri + '/interactions',
209209
headers=self.HEADERS,
210+
verify=False,
210211
json={"interactions": self._interactions})
211212

212213
assert resp.status_code == 200, resp.text
@@ -288,10 +289,10 @@ def verify(self):
288289
self._interactions = []
289290
resp = requests.get(
290291
self.uri + '/interactions/verification',
291-
headers=self.HEADERS)
292+
headers=self.HEADERS, verify=False)
292293
assert resp.status_code == 200, resp.text
293294
resp = requests.post(
294-
self.uri + '/pact', headers=self.HEADERS)
295+
self.uri + '/pact', headers=self.HEADERS, verify=False)
295296
assert resp.status_code == 200, resp.text
296297

297298
def with_request(self, method, path, body=None, headers=None, query=None):
@@ -362,8 +363,10 @@ def _wait_for_server_start(self):
362363
"""
363364
s = requests.Session()
364365
retries = Retry(total=9, backoff_factor=0.1)
365-
s.mount('http://', HTTPAdapter(max_retries=retries))
366-
resp = s.get(self.uri, headers=self.HEADERS)
366+
http_mount = 'https://' if self.ssl else 'http://'
367+
s.mount(http_mount, HTTPAdapter(max_retries=retries))
368+
369+
resp = s.get(self.uri, headers=self.HEADERS, verify=False)
367370
if resp.status_code != 200:
368371
self._process.terminate()
369372
self._process.communicate()

tests/test_pact.py

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -375,12 +375,14 @@ def setUp(self):
375375
.will_respond_with(200, body='success'))
376376

377377
self.delete_call = call('delete', 'http://localhost:1234/interactions',
378-
headers={'X-Pact-Mock-Service': 'true'})
378+
headers={'X-Pact-Mock-Service': 'true'},
379+
verify=False)
379380

380381
self.put_interactions_call = call(
381382
'put', 'http://localhost:1234/interactions',
382383
data=None,
383384
headers={'X-Pact-Mock-Service': 'true'},
385+
verify=False,
384386
json={'interactions': [{
385387
'response': {'status': 200, 'body': 'success'},
386388
'request': {'path': '/path', 'method': 'GET'},
@@ -570,7 +572,9 @@ def test_wait_for_server_start_success(self):
570572
session.mount.assert_called_once_with(
571573
'http://', self.mock_HTTPAdapter.return_value)
572574
session.get.assert_called_once_with(
573-
'http://localhost:1234', headers={'X-Pact-Mock-Service': 'true'})
575+
'http://localhost:1234',
576+
headers={'X-Pact-Mock-Service': 'true'},
577+
verify=False)
574578
self.mock_HTTPAdapter.assert_called_once_with(
575579
max_retries=self.mock_Retry.return_value)
576580
self.mock_Retry.assert_called_once_with(total=9, backoff_factor=0.1)
@@ -588,7 +592,9 @@ def test_wait_for_server_start_failure(self):
588592
session.mount.assert_called_once_with(
589593
'http://', self.mock_HTTPAdapter.return_value)
590594
session.get.assert_called_once_with(
591-
'http://localhost:1234', headers={'X-Pact-Mock-Service': 'true'})
595+
'http://localhost:1234',
596+
headers={'X-Pact-Mock-Service': 'true'},
597+
verify=False)
592598
self.mock_HTTPAdapter.assert_called_once_with(
593599
max_retries=self.mock_Retry.return_value)
594600
self.mock_Retry.assert_called_once_with(total=9, backoff_factor=0.1)
@@ -611,12 +617,14 @@ def setUp(self):
611617
'get', 'http://localhost:1234/interactions/verification',
612618
allow_redirects=True,
613619
headers={'X-Pact-Mock-Service': 'true'},
620+
verify=False,
614621
params=None)
615622

616623
self.post_publish_pacts_call = call(
617624
'post', 'http://localhost:1234/pact',
618625
data=None,
619626
headers={'X-Pact-Mock-Service': 'true'},
627+
verify=False,
620628
json=None)
621629

622630
def test_success(self):

tox.ini

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@ envlist=py{34,35,36,37,38}-{test,install}
33
[testenv]
44
deps=
55
test: -rrequirements_dev.txt
6-
ommands=
6+
commands=
77
test: nosetests
88
install: python -c "import pact"

0 commit comments

Comments
 (0)