Skip to content

Commit cffad67

Browse files
Merge pull request #427 from OP-TED/feature/TED-1141
Update supra_notice_validator.py
2 parents f19c3bd + 00df331 commit cffad67

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

ted_sws/supra_notice_manager/services/supra_notice_validator.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
from ted_sws.core.model.supra_notice import SupraNoticeValidationReport, DailySupraNotice
99
from ted_sws.data_manager.adapters.notice_repository import NoticeRepository
1010
from ted_sws.data_manager.adapters.supra_notice_repository import DailySupraNoticeRepository
11+
from ted_sws.event_manager.services.log import log_technical_error
1112
from ted_sws.notice_fetcher.adapters.ted_api import TedAPIAdapter, RequestAPI, TedRequestAPI
1213
from ted_sws.notice_validator.services.validation_summary_runner import generate_validation_summary_report_notices
1314

@@ -42,7 +43,8 @@ def validate_and_update_daily_supra_notice(notice_publication_day: day_type, mon
4243
validation_report = supra_notice.validation_report or SupraNoticeValidationReport(object_data="")
4344
missing_notice_ids = api_notice_ids - fetched_notice_ids
4445
if len(missing_notice_ids):
45-
validation_report.missing_notice_ids = missing_notice_ids
46+
validation_report.missing_notice_ids = list(missing_notice_ids)
47+
log_technical_error(message=f"Supra notice for date [{notice_publication_day}] don't fetch notices with ids=[{missing_notice_ids}]")
4648

4749
supra_notice.validation_report = validation_report
4850
repo.update(daily_supra_notice=supra_notice)

0 commit comments

Comments
 (0)