✅ (tests/src/cmds/core/test_user.py): Fix tests patch DB call

src/cmds/core/user.py

@@ -95,7 +95,7 @@ async def kick(self, ctx: ApplicationContext, user: Member, reason: str, evidenc
             )
 
         await ctx.guild.kick(user=member, reason=reason)
-        infraction_reason = f"{ctx.user.name} was kicked on {datetime.now()} for {reason}"
+        infraction_reason = f"{ctx.user.name} was kicked on {datetime.now().strftime('%Y-%m-%d %H:%M:%S')} for {reason}"
         await add_infraction(ctx.guild, member, 0, infraction_reason, ctx.user)
         return await ctx.respond(f"{member.name} got the boot!")
 

tests/src/cmds/core/test_user.py

@@ -1,3 +1,4 @@
+from datetime import datetime
 from unittest.mock import AsyncMock, patch
 
 import pytest
@@ -10,9 +11,10 @@ class TestUserCog:
     """Test the `User` cog."""
 
     @pytest.mark.asyncio
-    async def test_kick_success(self, ctx, bot):
+    async def test_kick_success(self, ctx, guild, bot, session):
         ctx.user = helpers.MockMember(id=1, name="Test Moderator")
         user_to_kick = helpers.MockMember(id=2, name="User to Kick", bot=False)
+        ctx.guild = guild
         ctx.guild.kick = AsyncMock()
         bot.get_member_or_user = AsyncMock(return_value=user_to_kick)
 
@@ -21,19 +23,23 @@ async def test_kick_success(self, ctx, bot):
         user_to_kick.name = "User to Kick"
 
         with (
+            patch('src.cmds.core.user.add_infraction', new_callable=AsyncMock) as add_infraction_mock,
             patch('src.cmds.core.user.add_evidence_note', new_callable=AsyncMock) as add_evidence_mock,
             patch('src.cmds.core.user.member_is_staff', return_value=False)
         ):
             cog = user.UserCog(bot)
             await cog.kick.callback(cog, ctx, user_to_kick, "Violation of rules")
 
-            add_evidence_mock.assert_called_once_with(user_to_kick.id, "kick", "Violation of rules", None, ctx.user.id)
+            reason = "Violation of rules"
+            add_evidence_mock.assert_called_once_with(user_to_kick.id, "kick", reason, None, ctx.user.id)
+            add_infraction_mock.assert_called_once_with(
+                ctx.guild, user_to_kick, 0, f"{ctx.user.name} was kicked on {datetime.now().strftime('%Y-%m-%d %H:%M:%S')} for {reason}", ctx.user
+            )
 
             # Assertions
             ctx.guild.kick.assert_called_once_with(user=user_to_kick, reason="Violation of rules")
             ctx.respond.assert_called_once_with("User to Kick got the boot!")
 
-
     def test_setup(self, bot):
         """Test the setup method of the cog."""
         # Invoke the command