Skip to content

Commit 0f83a61

Browse files
committed
VED-457: Send correct E-Tag in update request.
1 parent 1931ff8 commit 0f83a61

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

e2e/test_create_immunization.py

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -37,30 +37,29 @@ def test_non_unique_identifier(self):
3737

3838
# Check that duplicate CREATE request is rejected
3939
self.assert_operation_outcome(self.default_imms_api.create_immunization(imms, expected_status_code=422), 422)
40+
self.assertEqual(res.headers["E-Tag"], "1")
4041

41-
# Check that duplice CREATE request is rejected after the event is updated
42+
# Check that duplicate CREATE request is rejected after the event is updated
4243
imms["id"] = imms_id # Imms fhir resource should include the id for update
43-
etag_version = int(res.headers["E-Tag"])
44-
print("E-Tag being sent:", etag_version)
45-
print("Update payload:", json.dumps(imms, indent=2, default=str))
46-
self.default_imms_api.update_immunization(imms_id, imms, headers={"E-Tag": str(etag_version)})
44+
self.default_imms_api.update_immunization(imms_id, imms)
4745
self.assertEqual(res.status_code, 200)
4846
del imms["id"] # Imms fhir resource should not include an id for create
4947
self.assert_operation_outcome(self.default_imms_api.create_immunization(imms, expected_status_code=422), 422)
5048

51-
# Check that duplice CREATE request is rejected after the event is updated then deleted
49+
# Check that duplicate CREATE request is rejected after the event is updated then deleted
5250
self.default_imms_api.delete_immunization(imms_id)
5351
self.assertEqual(self.default_imms_api.get_immunization_by_id(
5452
imms_id, expected_status_code=404).status_code, 404)
5553
self.assert_operation_outcome(self.default_imms_api.create_immunization(imms, expected_status_code=422), 422)
5654

57-
# Check that duplice CREATE request is rejected after the event is updated then deleted then reinstated
55+
# Check that duplicate CREATE request is rejected after the event is updated then deleted then reinstated
5856
imms["id"] = imms_id # Imms fhir resource should include the id for update
59-
self.default_imms_api.update_immunization(imms_id, imms, headers={"E-Tag": str(etag_version)})
57+
self.default_imms_api.update_immunization(imms_id, imms, headers={"E-Tag": "2"})
6058
res = self.default_imms_api.get_immunization_by_id(imms_id)
6159
self.assertEqual(res.status_code, 200)
6260
del imms["id"] # Imms fhir resource should not include an id for create
6361
self.assert_operation_outcome(self.default_imms_api.create_immunization(imms, expected_status_code=422), 422)
62+
self.assertEqual(res.headers["E-Tag"], "3")
6463

6564
def test_bad_nhs_number(self):
6665
"""it should reject the request if nhs-number does not exist"""

0 commit comments

Comments
 (0)