diff --git a/app/daily_update/templates/report_body.py b/app/daily_update/templates/report_body.py index 236a627..ac46944 100644 --- a/app/daily_update/templates/report_body.py +++ b/app/daily_update/templates/report_body.py @@ -6,7 +6,7 @@ from app.daily_update import announce MOD_TODO_URL = "https://check.arxiv.org/q/todo" -MOD_TODO_TITLE = "Your moderation todo queue" +MOD_TODO_TITLE = "Your moderation to-do queue" def announce_line() -> str: """the header line about when everything not on hold will be announced""" @@ -21,7 +21,7 @@ def announce_line() -> str: class Section(str, Enum): """the parts of the report, listed in the order they appear. the value is the heading used in the email.""" HOLD = "On Hold" - NEW = "New" + NEW = "Scheduled for announcement" CROSS = "Cross Lists" NOTHING_TO_REPORT = "You have no new activity or submissions on Hold today!" diff --git a/tests/daily_update/test_process.py b/tests/daily_update/test_process.py index 68350ea..74bacdc 100644 --- a/tests/daily_update/test_process.py +++ b/tests/daily_update/test_process.py @@ -125,7 +125,7 @@ def test_cross_does_not_reach_a_category_it_already_lives_in(sends): def test_mod_hold_lands_in_the_hold_section(sends): body = sends["digest-cat@example.com"]["body"] - hold_part = body.split("New:")[0] + hold_part = body.split("Scheduled for announcement:")[0] assert "submit/210" in hold_part #a rep on hold — the only way a replacement appears at all @@ -251,13 +251,13 @@ def test_the_whole_html_digest_for_one_moderator(sends): assert sends["digest-cat@example.com"]["html_body"] == ( '
Daily moderator report for cs.AI
\n' 'If no further actions are taken, all submissions below not currently on hold will be announced at 09-03 20:00 EDT.
\n' - '\n' + '\n' '07-27 15:00 EDT cs.AI Frank Franky submit/210
\n'
'On Mod Hold
\n'
'Mod Hold Author
\n'
'Proposals: none
07-27 18:00 EDT cs.AI Frank Franky submit/213
\n'
'A Discussed Paper
\n'
'Talky Author
\n'
diff --git a/tests/daily_update/test_report.py b/tests/daily_update/test_report.py
index edc0ba6..317ccbf 100644
--- a/tests/daily_update/test_report.py
+++ b/tests/daily_update/test_report.py
@@ -52,7 +52,7 @@ def _mod(labels=("cs.AI",), categories=("cs.AI",)) -> DigestMod:
# literals, so a reorder, a retitle or a dropped section fails here
def test_sections_are_in_this_order_with_these_headings():
- assert [section.value for section in Section] == ["On Hold", "New", "Cross Lists"]
+ assert [section.value for section in Section] == ["On Hold", "Scheduled for announcement", "Cross Lists"]
# ── header ──────────────────────────────────────────────────────────────────
@@ -63,7 +63,7 @@ def test_header_text_is_the_whole_layout():
"\n"
f"{announce_line()}\n"
"\n"
- "Your moderation todo queue: https://check.arxiv.org/q/todo\n"
+ "Your moderation to-do queue: https://check.arxiv.org/q/todo\n"
)
@@ -71,7 +71,7 @@ def test_header_html_is_the_whole_layout():
assert render_header("cs.AI").html == (
"
Daily moderator report for cs.AI
\n" f"{announce_line()}
\n" - '\n' + '\n' ) @@ -132,7 +132,7 @@ def test_a_section_with_content_still_shows_none_under_the_empty_ones(): "On Hold:\n" f" none\n" "\n" - "New:\n" + "Scheduled for announcement:\n" "a new one\n" "\n" "Cross Lists:\n" @@ -144,7 +144,7 @@ def test_a_section_with_content_still_shows_none_under_the_empty_ones(): assert body_html == ( "none
\n" - "a new one
\n" "none
\n" @@ -154,7 +154,7 @@ def test_a_section_with_content_still_shows_none_under_the_empty_ones(): def test_sections_appear_in_this_order(): entries = {section: [Rendered(f"{section.value} entry\n", "\n")] for section in Section} text, _ = render_body("cs.AI", entries) - positions = [text.index(t) for t in ("On Hold:", "New:", "Cross Lists:")] + positions = [text.index(t) for t in ("On Hold:", "Scheduled for announcement:", "Cross Lists:")] assert positions == sorted(positions)