Skip to content

Commit bd3d315

Browse files
Merge pull request #114 from rosette-api/WS-3224-1.30-release
WS-3224: fix requests compatibility issue
2 parents c53aac2 + 7b110e8 commit bd3d315

File tree

3 files changed

+7
-21
lines changed

3 files changed

+7
-21
lines changed

examples/record_similarity.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,8 @@ def run(key, alt_url='https://api.rosette.com/rest/v1/'):
5050
"dob": "1993-04-16",
5151
"addr": "123 Roadlane Ave",
5252
"dob2": {
53-
"date": "1993/04/16"
53+
"date": "04161993",
54+
"format": "MMddyyyy"
5455
}
5556
},
5657
{
@@ -78,7 +79,8 @@ def run(key, alt_url='https://api.rosette.com/rest/v1/'):
7879
"date": "1993-04-16"
7980
},
8081
"addr": {
81-
"address": "123 Roadlane Ave"
82+
"houseNumber": "123",
83+
"road": "Roadlane Ave"
8284
},
8385
"dob2": {
8486
"date": "1993/04/16"

rosette/api.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -360,7 +360,7 @@ def __init__(self):
360360

361361
def validate(self):
362362
"""Internal. Do not use."""
363-
for option in "fields", "properties", "records": # required
363+
for option in ["records"]: # required
364364
if self[option] is None:
365365
raise RosetteException(
366366
"missingParameter",
@@ -528,7 +528,7 @@ def call(self, parameters, paramtype=None):
528528
request = requests.Request(
529529
'POST', url, files=files, headers=headers, params=payload)
530530
prepared_request = self.api.session.prepare_request(request)
531-
settings = self.api.session.merge_environment_settings(prepared_request.url, {}, {}, None, {})
531+
settings = self.api.session.merge_environment_settings(prepared_request.url, {}, {}, None, None)
532532
response = self.api.session.send(prepared_request, **settings)
533533
rdata = response.content
534534
response_headers = {"responseHeaders": dict(response.headers)}
@@ -671,7 +671,7 @@ def _make_request(self, operation, url, data, headers):
671671
operation, url, data=data, headers=headers, params=payload)
672672
prepared_request = self.session.prepare_request(request)
673673
# Take into account environment settings, e.g. HTTP_PROXY and HTTPS_PROXY
674-
settings = self.session.merge_environment_settings(prepared_request.url, {}, {}, None, {})
674+
settings = self.session.merge_environment_settings(prepared_request.url, {}, {}, None, None)
675675

676676
try:
677677
response = self.session.send(prepared_request, **settings)

tests/test_rosette_api.py

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -993,22 +993,6 @@ def test_for_record_similarity_required_parameters(api, json_response):
993993

994994
params = RecordSimilarityParameters()
995995

996-
with pytest.raises(RosetteException) as e_rosette:
997-
api.record_similarity(params)
998-
999-
assert e_rosette.value.status == 'missingParameter'
1000-
assert e_rosette.value.message == 'Required Record Similarity parameter is missing: fields'
1001-
1002-
params["fields"] = {}
1003-
1004-
with pytest.raises(RosetteException) as e_rosette:
1005-
api.record_similarity(params)
1006-
1007-
assert e_rosette.value.status == 'missingParameter'
1008-
assert e_rosette.value.message == 'Required Record Similarity parameter is missing: properties'
1009-
1010-
params["properties"] = {}
1011-
1012996
with pytest.raises(RosetteException) as e_rosette:
1013997
api.record_similarity(params)
1014998

0 commit comments

Comments
 (0)