Skip to content

Commit 3ebdac7

Browse files
committed
update processor test
2 parents 852dba8 + e9b1514 commit 3ebdac7

File tree

2 files changed

+9
-29
lines changed

2 files changed

+9
-29
lines changed

recordprocessor/tests/test_audit_table.py

Lines changed: 8 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -7,23 +7,20 @@
77

88
from tests.utils_for_recordprocessor_tests.mock_environment_variables import MOCK_ENVIRONMENT_DICT
99
from tests.utils_for_recordprocessor_tests.generic_setup_and_teardown import GenericSetUp, GenericTearDown
10-
from tests.utils_for_recordprocessor_tests.values_for_recordprocessor_tests import MockFileDetails, FileDetails
10+
from tests.utils_for_recordprocessor_tests.values_for_recordprocessor_tests import MockFileDetails
1111
from tests.utils_for_recordprocessor_tests.utils_for_recordprocessor_tests import (
1212
deserialize_dynamodb_types,
1313
add_entry_to_table,
14-
assert_audit_table_entry,
1514
)
1615

1716
# Ensure environment variables are mocked before importing from src files
1817
with patch.dict("os.environ", MOCK_ENVIRONMENT_DICT):
1918
from constants import (
2019
AUDIT_TABLE_NAME,
21-
AuditTableKeys,
2220
FileStatus,
2321
)
2422

2523
from audit_table import get_next_queued_file_details, change_audit_table_status_to_processed
26-
from errors import UnhandledAuditTableError
2724
from clients import REGION_NAME
2825

2926
dynamodb_client = boto3_client("dynamodb", region_name=REGION_NAME)
@@ -77,35 +74,18 @@ def test_get_next_queued_file_details(self):
7774
self.assertEqual(get_next_queued_file_details(queue_to_check), deserialize_dynamodb_types(expected_table_entry))
7875

7976
def test_change_audit_table_status_to_processed(self):
80-
# Checks audit table is updated correctly
81-
queue_to_check = "RAVS_RSV"
77+
"""Checks audit table correctly updates a record as processed"""
8278

8379
add_entry_to_table(MockFileDetails.rsv_ravs, file_status=FileStatus.QUEUED)
80+
add_entry_to_table(MockFileDetails.flu_emis, file_status=FileStatus.QUEUED)
8481
table_items = dynamodb_client.scan(TableName=AUDIT_TABLE_NAME).get("Items", [])
85-
# print(f"started off queued:{table_items}")
8682

8783
expected_table_entry = {**MockFileDetails.rsv_ravs.audit_table_entry, "status": {"S": FileStatus.PROCESSED}}
84+
8885
file_key = "RSV_Vaccinations_v5_X26_20210730T12000000.csv"
8986
message_id = "rsv_ravs_test_id_1"
90-
# print(f"file_key_checker: {file_key}")
91-
# print(f"message_id: {message_id}")
92-
result = change_audit_table_status_to_processed(file_key, message_id)
93-
print(result)
87+
88+
change_audit_table_status_to_processed(file_key, message_id)
9489
table_items = dynamodb_client.scan(TableName=AUDIT_TABLE_NAME).get("Items", [])
95-
# print(f"change to processed :{table_items}")
96-
97-
Actual_result = [
98-
{
99-
"message_id": {"S": "rsv_ravs_test_id_1"},
100-
"filename": {"S": "RSV_Vaccinations_v5_X26_20210730T12000000.csv"},
101-
"queue_name": {"S": "RAVS_RSV"},
102-
"timestamp": {"S": "20211120T12000000"},
103-
"status": {"S": "Processed"},
104-
}
105-
]
106-
Actual_logger = "The status of RSV_Vaccinations_v5_X26_20210730T12000000.csv file, with message id rsv_ravs_test_id_1, was successfully updated to Processed in the audit table"
107-
# self.assertEqual(result, deserialize_dynamodb_types(expected_table_entry))
108-
109-
# Check items in the dynamo table
110-
# table_items = dynamodb_client.scan(TableName=AUDIT_TABLE_NAME).get("Items", [])
111-
# print(f"table items 4 :{table_items}")
90+
91+
self.assertIn(expected_table_entry, table_items)

recordprocessor/tests/utils_for_recordprocessor_tests/values_for_recordprocessor_tests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ def __init__(self, vaccine_type: str, supplier: str, ods_code: str, file_number:
148148
self.vaccine_type = vaccine_type
149149
self.ods_code = ods_code
150150
self.supplier = supplier
151-
file_date_and_time_string = f"20000101T0000000{file_number}"
151+
self.file_date_and_time_string = f"20000101T0000000{file_number}"
152152
self.message_id = f"{vaccine_type.lower()}_{supplier.lower()}_test_id"
153153
self.message_id_order = f"{vaccine_type.lower()}_{supplier.lower()}_test_id_{file_number}"
154154
self.full_permissions_list = [f"{vaccine_type}_FULL"]

0 commit comments

Comments
 (0)