Skip to content

Commit

Permalink
Fix test_report_runner.py (#4003)
Browse files Browse the repository at this point in the history
  • Loading branch information
originalsouth authored Jan 1, 2025
1 parent 8730e18 commit 39130d0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion rocky/tests/integration/test_report_runner.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import json
from datetime import date

from reports.runner.report_runner import LocalReportRunner

Expand Down Expand Up @@ -89,7 +90,7 @@ def test_run_report_task(octopoes_api_connector: OctopoesAPIConnector, report_ru
assert len(subreports) == 2

assert report.name == "Concatenated report for 2 objects"
assert "DNS Report for a.example.com in 2024" in {x.name for x in subreports}
assert f"DNS Report for a.example.com in {date.today().year}" in {x.name for x in subreports}

# FIXME: the naming logic in reports/views/mixins.py 107-112 is not right. We expect to find example.com in this
# set, but instead only find a.example.com because when ooi_name is 'example.com', the check:
Expand Down

0 comments on commit 39130d0

Please sign in to comment.