Merge pull request #110 from discord-modmail/hotfix-patch-embed

hotfix: patch embeds throughout the entire test suite

Author: onerandomusername

tests/conftest.py

@@ -1,6 +1 @@
 import pytest
-
-
-def pytest_report_header(config) -> str:
-    """Pytest headers."""
-    return "package: modmail"

tests/modmail/conftest.py

@@ -0,0 +1,14 @@
+import pytest
+
+
+def pytest_report_header(config) -> str:
+    """Pytest headers."""
+    return "package: modmail"
+
+
+@pytest.fixture(autouse=True, scope="package")
+def patch_embeds():
+    """Run the patch embed method. This is normally run by modmail.__main__, which is not run for testing."""
+    import modmail.utils.embeds
+
+    modmail.utils.embeds.patch_embed()

tests/modmail/utils/test_embeds.py

@@ -1,3 +1,5 @@
+import unittest.mock
+
 import discord
 import pytest
 
@@ -10,9 +12,10 @@ def test_patch_embed() -> None:
     from modmail.utils.embeds import __init__ as init
     from modmail.utils.embeds import original_init
 
-    assert discord.Embed.__init__ == original_init
-    patch_embed()
-    assert discord.Embed.__init__ == init
+    with unittest.mock.patch.object(discord.Embed, "__init__", original_init):
+        assert discord.Embed.__init__ == original_init
+        patch_embed()
+        assert discord.Embed.__init__ == init
 
 
 @pytest.mark.dependency(depends_on="patch_embed")