Use lower case for submitter role

Signed-off-by: Martin Vrachev <[email protected]>

Author: MVrachev

tests/unit/email/test_init.py

@@ -1143,14 +1143,14 @@ def test_removed_project_email_to_maintainer(
         assert result == {
             "project": "test_project",
             "submitter": stub_submitter_user.username,
-            "submitter_role": "Owner",
+            "submitter_role": "owner",
             "recipient_role_descr": "a maintainer",
         }
 
         subject_renderer.assert_(project="test_project")
         body_renderer.assert_(project="test_project")
         body_renderer.assert_(submitter=stub_submitter_user.username)
-        body_renderer.assert_(submitter_role="Owner")
+        body_renderer.assert_(submitter_role="owner")
         body_renderer.assert_(recipient_role_descr="a maintainer")
 
         assert pyramid_request.task.calls == [
@@ -1235,14 +1235,14 @@ def test_removed_project_email_to_owner(
         assert result == {
             "project": "test_project",
             "submitter": stub_submitter_user.username,
-            "submitter_role": "Owner",
+            "submitter_role": "owner",
             "recipient_role_descr": "an owner",
         }
 
         subject_renderer.assert_(project="test_project")
         body_renderer.assert_(project="test_project")
         body_renderer.assert_(submitter=stub_submitter_user.username)
-        body_renderer.assert_(submitter_role="Owner")
+        body_renderer.assert_(submitter_role="owner")
         body_renderer.assert_(recipient_role_descr="an owner")
 
         assert pyramid_request.task.calls == [
@@ -1338,7 +1338,7 @@ def test_send_removed_project_release_email_to_maintainer(
             "release": release.version,
             "release_date": release.created.strftime("%Y-%m-%d"),
             "submitter": stub_submitter_user.username,
-            "submitter_role": "Owner",
+            "submitter_role": "owner",
             "recipient_role_descr": "a maintainer",
         }
 
@@ -1348,7 +1348,7 @@ def test_send_removed_project_release_email_to_maintainer(
         body_renderer.assert_(release="0.0.0")
         body_renderer.assert_(release_date=release.created.strftime("%Y-%m-%d"))
         body_renderer.assert_(submitter=stub_submitter_user.username)
-        body_renderer.assert_(submitter_role="Owner")
+        body_renderer.assert_(submitter_role="owner")
         body_renderer.assert_(recipient_role_descr="a maintainer")
 
         assert pyramid_request.task.calls == [
@@ -1442,7 +1442,7 @@ def test_send_removed_project_release_emai_to_owner(
             "release": release.version,
             "release_date": release.created.strftime("%Y-%m-%d"),
             "submitter": stub_submitter_user.username,
-            "submitter_role": "Owner",
+            "submitter_role": "owner",
             "recipient_role_descr": "an owner",
         }
 
@@ -1452,7 +1452,7 @@ def test_send_removed_project_release_emai_to_owner(
         body_renderer.assert_(release="0.0.0")
         body_renderer.assert_(release_date=release.created.strftime("%Y-%m-%d"))
         body_renderer.assert_(submitter=stub_submitter_user.username)
-        body_renderer.assert_(submitter_role="Owner")
+        body_renderer.assert_(submitter_role="owner")
         body_renderer.assert_(recipient_role_descr="an owner")
 
         assert pyramid_request.task.calls == [

warehouse/email/__init__.py

@@ -224,7 +224,7 @@ def send_removed_project_email(
     return {
         "project": project_name,
         "submitter": submitter_name,
-        "submitter_role": submitter_role,
+        "submitter_role": submitter_role.lower(),
         "recipient_role_descr": recipient_role_descr,
     }
 
@@ -242,7 +242,7 @@ def send_removed_project_release_email(
         "release": release.version,
         "release_date": release.created.strftime("%Y-%m-%d"),
         "submitter": submitter_name,
-        "submitter_role": submitter_role,
+        "submitter_role": submitter_role.lower(),
         "recipient_role_descr": recipient_role_descr,
     }