Skip to content

Commit 521b2cf

Browse files
committed
Use lower case for submitter role
Signed-off-by: Martin Vrachev <[email protected]>
1 parent 74458b4 commit 521b2cf

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

tests/unit/email/test_init.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1338,7 +1338,7 @@ def test_send_removed_project_release_email_to_maintainer(
13381338
"release": release.version,
13391339
"release_date": release.created.strftime("%Y-%m-%d"),
13401340
"submitter": stub_submitter_user.username,
1341-
"submitter_role": "Owner",
1341+
"submitter_role": "owner",
13421342
"recipient_role_descr": "a maintainer",
13431343
}
13441344

@@ -1348,7 +1348,7 @@ def test_send_removed_project_release_email_to_maintainer(
13481348
body_renderer.assert_(release="0.0.0")
13491349
body_renderer.assert_(release_date=release.created.strftime("%Y-%m-%d"))
13501350
body_renderer.assert_(submitter=stub_submitter_user.username)
1351-
body_renderer.assert_(submitter_role="Owner")
1351+
body_renderer.assert_(submitter_role="owner")
13521352
body_renderer.assert_(recipient_role_descr="a maintainer")
13531353

13541354
assert pyramid_request.task.calls == [
@@ -1442,7 +1442,7 @@ def test_send_removed_project_release_emai_to_owner(
14421442
"release": release.version,
14431443
"release_date": release.created.strftime("%Y-%m-%d"),
14441444
"submitter": stub_submitter_user.username,
1445-
"submitter_role": "Owner",
1445+
"submitter_role": "owner",
14461446
"recipient_role_descr": "an owner",
14471447
}
14481448

@@ -1452,7 +1452,7 @@ def test_send_removed_project_release_emai_to_owner(
14521452
body_renderer.assert_(release="0.0.0")
14531453
body_renderer.assert_(release_date=release.created.strftime("%Y-%m-%d"))
14541454
body_renderer.assert_(submitter=stub_submitter_user.username)
1455-
body_renderer.assert_(submitter_role="Owner")
1455+
body_renderer.assert_(submitter_role="owner")
14561456
body_renderer.assert_(recipient_role_descr="an owner")
14571457

14581458
assert pyramid_request.task.calls == [

warehouse/email/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -242,7 +242,7 @@ def send_removed_project_release_email(
242242
"release": release.version,
243243
"release_date": release.created.strftime("%Y-%m-%d"),
244244
"submitter": submitter_name,
245-
"submitter_role": submitter_role,
245+
"submitter_role": submitter_role.lower(),
246246
"recipient_role_descr": recipient_role_descr,
247247
}
248248

0 commit comments

Comments
 (0)