feat: apply changes related to tests from Top-gg-Community#87

Author: null8626

.github/workflows/release.yml

@@ -9,7 +9,7 @@ jobs:
       - uses: actions/checkout@v5
       - uses: actions/setup-python@v6
         with:
-          python-version: 3.13
+          python-version: 3.14
       - name: Install dependencies
         run: python3 -m pip install build twine
       - name: Build and publish

.github/workflows/test.yml

@@ -9,14 +9,14 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: [ 3.9, '3.10', 3.11, 3.12, 3.13 ]
+        python-version: [ '3.10', 3.11, 3.12, 3.13, 3.14 ]
     steps:
       - uses: actions/checkout@v5
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v6
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install
-        run: python -m pip install .[dev]
+        run: python -m pip install . pytest mock pytest-mock pytest-asyncio
       - name: Test with pytest
         run: pytest

MANIFEST.in

@@ -1,18 +1,11 @@
 prune .github
-prune .pytest_cache
 prune .ruff_cache
-prune build
 prune docs
 prune examples
-prune scripts
 prune tests
-
-exclude .coverage
 exclude .gitignore
 exclude .readthedocs.yml
-exclude ISSUE_TEMPLATE.md
-exclude mypy.ini
-exclude PULL_REQUEST_TEMPLATE.md
-exclude pytest.ini
 exclude ruff.toml
-exclude LICENSE
+exclude LICENSE
+exclude ISSUE_TEMPLATE.md
+exclude PULL_REQUEST_TEMPLATE.md

tests/test_autopost.py

@@ -27,19 +27,19 @@ def autopost(session: ClientSession) -> AutoPoster:
 async def test_AutoPoster_breaks_autopost_loop_on_401(
     mocker: MockerFixture, session: ClientSession
 ) -> None:
-    response = mock.Mock("reason, status")
-    response.reason = "Unauthorized"
-    response.status = 401
-
     mocker.patch(
-        "topgg.DBLClient.post_guild_count", side_effect=HTTPException(response, {})
+        "topgg.DBLClient.post_guild_count",
+        side_effect=HTTPException("Unauthorized", 401),
     )
 
     callback = mock.Mock()
     autopost = DBLClient(MOCK_TOKEN, session=session).autopost().stats(callback)
+
     assert isinstance(autopost, AutoPoster)
     assert not isinstance(autopost.stats()(callback), AutoPoster)
 
+    autopost._interval = 1
+
     with pytest.raises(HTTPException):
         await autopost.start()
 
@@ -50,21 +50,21 @@ async def test_AutoPoster_breaks_autopost_loop_on_401(
 @pytest.mark.asyncio
 async def test_AutoPoster_raises_missing_stats(autopost: AutoPoster) -> None:
     with pytest.raises(
-        TopGGException, match="you must provide a callback that returns the stats."
+        TopGGException, match="You must provide a callback that returns the stats."
     ):
         await autopost.start()
 
 
 @pytest.mark.asyncio
 async def test_AutoPoster_raises_already_running(autopost: AutoPoster) -> None:
     autopost.stats(mock.Mock()).start()
-    with pytest.raises(TopGGException, match="the autopost is already running."):
+    with pytest.raises(TopGGException, match="The autoposter is already running."):
         await autopost.start()
 
 
 @pytest.mark.asyncio
 async def test_AutoPoster_interval_too_short(autopost: AutoPoster) -> None:
-    with pytest.raises(ValueError, match="interval must be greated than 900 seconds."):
+    with pytest.raises(ValueError, match="interval must be greater than 900 seconds."):
         autopost.set_interval(50)
 
 
@@ -73,10 +73,7 @@ async def test_AutoPoster_error_callback(
     mocker: MockerFixture, autopost: AutoPoster
 ) -> None:
     error_callback = mock.Mock()
-    response = mock.Mock("reason, status")
-    response.reason = "Internal Server Error"
-    response.status = 500
-    side_effect = HTTPException(response, {})
+    side_effect = HTTPException("Internal Server Error", 500)
 
     mocker.patch("topgg.DBLClient.post_guild_count", side_effect=side_effect)
     task = autopost.on_error(error_callback).stats(mock.Mock()).start()

tests/test_client.py

@@ -1,54 +1,65 @@
 import mock
 import pytest
+from aiohttp import ClientSession
 
 import topgg
 
 
 MOCK_TOKEN = ".eyJfdCI6IiIsImlkIjoiMzY0ODA2MDI5ODc2NTU1Nzc2In0=."
 
 
+@pytest.fixture
+def session() -> ClientSession:
+    return mock.Mock(ClientSession)
+
+
+@pytest.fixture
+def client(session: ClientSession) -> topgg.DBLClient:
+    return topgg.DBLClient(MOCK_TOKEN, session=session)
+
+
 @pytest.mark.asyncio
-async def test_DBLClient_post_guild_count_with_no_args():
-    client = topgg.DBLClient(MOCK_TOKEN)
-    with pytest.raises(TypeError, match="stats or guild_count must be provided."):
+async def test_DBLClient_post_guild_count_with_no_args(client: topgg.DBLClient):
+    with pytest.raises(ValueError, match="Got an invalid server count. Got None."):
         await client.post_guild_count()
 
 
 @pytest.mark.asyncio
-async def test_DBLClient_get_weekend_status(monkeypatch):
-    client = topgg.DBLClient(MOCK_TOKEN)
+async def test_DBLClient_get_weekend_status(monkeypatch, client: topgg.DBLClient):
     monkeypatch.setattr("topgg.DBLClient._DBLClient__request", mock.AsyncMock())
     await client.get_weekend_status()
     client._DBLClient__request.assert_called_once()
 
 
 @pytest.mark.asyncio
-async def test_DBLClient_post_guild_count(monkeypatch):
-    client = topgg.DBLClient(MOCK_TOKEN)
+async def test_DBLClient_post_guild_count(monkeypatch, client: topgg.DBLClient):
     monkeypatch.setattr("topgg.DBLClient._DBLClient__request", mock.AsyncMock())
     await client.post_guild_count(guild_count=123)
     client._DBLClient__request.assert_called_once()
 
 
 @pytest.mark.asyncio
-async def test_DBLClient_get_guild_count(monkeypatch):
-    client = topgg.DBLClient(MOCK_TOKEN)
-    monkeypatch.setattr("topgg.DBLClient._DBLClient__request", mock.AsyncMock(return_value={}))
+async def test_DBLClient_get_guild_count(monkeypatch, client: topgg.DBLClient):
+    monkeypatch.setattr(
+        "topgg.DBLClient._DBLClient__request", mock.AsyncMock(return_value={})
+    )
     await client.get_guild_count()
     client._DBLClient__request.assert_called_once()
 
 
 @pytest.mark.asyncio
-async def test_DBLClient_get_bot_votes(monkeypatch):
-    client = topgg.DBLClient(MOCK_TOKEN)
-    monkeypatch.setattr("topgg.DBLClient._DBLClient__request", mock.AsyncMock(return_value=[]))
+async def test_DBLClient_get_bot_votes(monkeypatch, client: topgg.DBLClient):
+    monkeypatch.setattr(
+        "topgg.DBLClient._DBLClient__request", mock.AsyncMock(return_value=[])
+    )
     await client.get_bot_votes()
     client._DBLClient__request.assert_called_once()
 
 
 @pytest.mark.asyncio
-async def test_DBLClient_get_user_vote(monkeypatch):
-    client = topgg.DBLClient(MOCK_TOKEN)
-    monkeypatch.setattr("topgg.DBLClient._DBLClient__request", mock.AsyncMock(return_value={"voted": 1}))
+async def test_DBLClient_get_user_vote(monkeypatch, client: topgg.DBLClient):
+    monkeypatch.setattr(
+        "topgg.DBLClient._DBLClient__request", mock.AsyncMock(return_value={"voted": 1})
+    )
     await client.get_user_vote(1234)
     client._DBLClient__request.assert_called_once()

tests/test_data_container.py

@@ -15,18 +15,15 @@ def data_container() -> DataContainerMixin:
 
 async def _async_callback(
     text: str = data(str), number: int = data(int), mapping: dict = data(dict)
-):
-    ...
+): ...
 
 
 def _sync_callback(
     text: str = data(str), number: int = data(int), mapping: dict = data(dict)
-):
-    ...
+): ...
 
 
-def _invalid_callback(number: float = data(float)):
-    ...
+def _invalid_callback(number: float = data(float)): ...
 
 
 @pytest.mark.asyncio