Revert to earlier version of pyright

tests/cogs/test_lfg_cog.py

@@ -74,7 +74,7 @@ async def test_lfg_fully_seated(
 
         with mock_operations(lfg_action, users=[other_player]):
             message = MagicMock(spec=discord.Message)
-            message.id = game.posts[0].message_xid  # type: ignore
+            message.id = game.posts[0].message_xid
             lfg_action.safe_get_partial_message.return_value = message
 
             await self.run(cog.lfg)

tests/cogs/test_lfg_cog_concurrency.py

@@ -52,11 +52,11 @@ async def test_concurrent_lfg_requests_different_channels(self, bot: SpellBot) -
         messages_out_of_order = False
         message_xid: int | None = None
         for game in games:  # pragma: no cover
-            if message_xid and game.posts[0].message_xid != message_xid + 1:  # type: ignore
+            if message_xid and game.posts[0].message_xid != message_xid + 1:
                 # At least one game is out of order, this is good!
                 messages_out_of_order = True
                 break
-            message_xid = game.posts[0].message_xid  # type: ignore
+            message_xid = game.posts[0].message_xid
         assert messages_out_of_order
 
     async def test_concurrent_lfg_requests_same_channel(
@@ -108,9 +108,9 @@ def get_next_message(*args: Any, **kwargs: Any) -> discord.Message:
         messages_out_of_order = False
         message_xid: int | None = None
         for game in games:  # pragma: no cover
-            if message_xid is not None and game.posts[0].message_xid != message_xid + 1:  # type: ignore
+            if message_xid is not None and game.posts[0].message_xid != message_xid + 1:
                 # At least one game is out of order, this is good!
                 messages_out_of_order = True
                 break
-            message_xid = game.posts[0].message_xid  # type: ignore
+            message_xid = game.posts[0].message_xid
         assert messages_out_of_order

tests/cogs/test_lfg_cog_points.py

@@ -58,7 +58,7 @@ async def test_points(
         user = self.factories.user.create(xid=self.interaction.user.id)
         other_user = self.factories.user.create(xid=self.interaction.user.id + 1, game=game)
         other_player = mock_discord_object(other_user)
-        message = MagicMock(spec=discord.Message, id=game.posts[0].message_xid)  # type: ignore
+        message = MagicMock(spec=discord.Message, id=game.posts[0].message_xid)
         self.interaction.original_response.return_value = message
 
         with patch.object(
@@ -165,7 +165,7 @@ async def test_points_when_message_not_found(
         user = self.factories.user.create(xid=self.interaction.user.id)
         other_user = self.factories.user.create(xid=self.interaction.user.id + 1, game=game)
         other_player = mock_discord_object(other_user)
-        message = MagicMock(spec=discord.Message, id=game.posts[0].message_xid)  # type: ignore
+        message = MagicMock(spec=discord.Message, id=game.posts[0].message_xid)
         self.interaction.original_response.return_value = message
 
         with patch.object(
@@ -247,7 +247,7 @@ async def test_points_when_not_in_game(
         self.factories.user.create(xid=self.interaction.user.id)
         other_user = self.factories.user.create(xid=self.interaction.user.id + 1, game=game)
         other_player = mock_discord_object(other_user)
-        message = MagicMock(spec=discord.Message, id=game.posts[0].message_xid)  # type: ignore
+        message = MagicMock(spec=discord.Message, id=game.posts[0].message_xid)
         self.interaction.original_response.return_value = message
 
         outside_user = self.factories.user.create(xid=self.interaction.user.id + 2)