Adding tests, removed first space

Co-authored-by: Julien Guts <[email protected]>

Author: 2 people

src/mkdocs_git_authors_plugin/git/page.py

@@ -256,7 +256,7 @@ def _process_git_log(self, sha, commit) -> None:
                 # skip author as already available in Commit object
                 continue
 
-            result = re.search(r"Co-authored-by:(.*) <(.*)>", line)
+            result = re.search(r"Co-authored-by: (.*) <(.*)>", line)
             if result is not None and result.group(1) != "" and result.group(2) != "":
                 # Extract co-authors from the commit
                 co_author = self.repo().author(result.group(1), result.group(2))

tests/basic_setup/docs/page_with_co_authors.md

@@ -1,7 +1,3 @@
-# Test with co-authors
+# Test with co-authors mentioned
 
 Page authors: {{ git_page_authors }}
-
-----
-
-Site authors: {{ git_site_authors }}

tests/test_basic.py

@@ -45,8 +45,6 @@
 ]
 
 
-
-
 def build_docs_setup(mkdocs_path, output_path) -> Result:
     runner = CliRunner()
     return runner.invoke(
@@ -144,6 +142,25 @@ def test_ignore_authors_working(tmp_path) -> None:
     assert not re.search("Julien", contents)
 
 
+def test_co_authors_working(tmp_path) -> None:
+    result = build_docs_setup("tests/basic_setup/mkdocs_ignore_authors.yml", tmp_path)
+    assert (
+        result.exit_code == 0
+    ), f"'mkdocs build' command failed. Error: {result.stdout}"
+
+    page_file = tmp_path / "page_with_co_authors/index.html"
+    assert page_file.exists(), f"{page_file} does not exist"
+
+    contents = page_file.read_text()
+    assert re.search("<span class='git-page-authors", contents)
+    assert re.search(
+        "<a href='mailto:12074690\[email protected]'>Fabien Pozzobon</a>",
+        contents,
+    )
+    assert re.search("<a href='mailto:[email protected]'>John Doe</a>", contents)
+    assert not re.search("Julien", contents)
+
+
 def test_exclude_working_with_genfiles(tmp_path) -> None:
     """
     A warning for uncommited files should not show up
@@ -331,11 +348,12 @@ def test_mkapi_v20x(tmp_path) -> None:
     assert True
 
 
-@pytest.mark.skipif(sys.version_info < (3, 7) or sys.version_info > (3, 9), reason="Requires Python 3.7 or higher")
 @pytest.mark.skipif(
-    not (
-        Version(mkdocs.__version__) < Version("1.6")
-    ),
+    sys.version_info < (3, 7) or sys.version_info > (3, 9),
+    reason="Requires Python 3.7 or higher",
+)
+@pytest.mark.skipif(
+    not (Version(mkdocs.__version__) < Version("1.6")),
     reason="Requires mkdocs  >= 1.6",
 )
 def test_mkapi_v1(tmp_path) -> None:
@@ -354,12 +372,12 @@ def test_mkapi_v1(tmp_path) -> None:
         contents,
     )
 
+
 def test_custom_docs_dir(tmp_path):
 
     testproject_path = tmp_path / "testproject"
     shutil.copytree("tests/custom_docs_dir", testproject_path)
 
-
     # init git inside the docs directory
     with working_directory(str(testproject_path / "documentation")):
         # setup git history
@@ -380,4 +398,3 @@ def test_custom_docs_dir(tmp_path):
 
     contents = index_file.read_text()
     assert re.search("<span class='git-page-authors", contents)
-

tests/test_util.py

@@ -129,7 +129,7 @@ def setup_commit_history(testproject_path):
         repo.git.commit(message="homepage", author=author)
         repo.git.add("docs/page_with_co_authors.md")
         repo.git.commit(
-            message="co-authors commit\nCo-authored-by: Test Person <[email protected]>\nCo-authored-by: John Doe <jdoe@john.com>",
+            message="co-authors commit\nCo-authored-by: Test Person <[email protected]>\nCo-authored-by: Roger Doe <rdoe@john.com>",
             author=author,
         )
         os.chdir(str(cwd))