Add malformed co-authors to the test, to check they aren't detected with the regex.

itsluketwist · itsluketwist · commit 3cb7ecf4e03c · 2022-08-23T19:50:00.000+01:00
diff --git a/test/test_commit.py b/test/test_commit.py
@@ -515,8 +515,12 @@ def test_commit_co_authors(self):
         commit.message = """Commit message
 
 Co-authored-by: Test User 1 <602352+test@users.noreply.github.com>
-Co-authored-by: test_user_2 <another_user-email@.github.com>"""
+Co-authored-by: test_user_2 <another_user-email@github.com>
+Co_authored_by: test_user_x <test@github.com>
+Co-authored-by: test_user_y <poorly formatted email @github.com>
+Co-authored-by: test_user_3 <test_user_3@github.com>"""
         assert commit.co_authors == [
             Actor("Test User 1", "602352+test@users.noreply.github.com"),
-            Actor("test_user_2", "another_user-email@.github.com"),
+            Actor("test_user_2", "another_user-email@github.com"),
+            Actor("test_user_3", "test_user_3@github.com"),
         ]

-Original file line number
+Diff line change
         commit.message = """Commit message
 Co-authored-by: Test User 1 <[email protected]>
 -Co-authored-by: test_user_2 <[email protected]>"""
 +Co-authored-by: test_user_2 <[email protected]>
 +Co_authored_by: test_user_x <[email protected]>
 +Co-authored-by: test_user_y <poorly formatted email @github.com>
 +Co-authored-by: test_user_3 <[email protected]>"""
         assert commit.co_authors == [
             Actor("Test User 1", "[email protected]"),
 -            Actor("test_user_2", "[email protected]"),
 +            Actor("test_user_2", "[email protected]"),
 +            Actor("test_user_3", "[email protected]"),
+        ]