Update live API healthcheck workflow to use new test target and add streaming marker to player game tests

Author: DoraFgr

.github/workflows/live-api-healthcheck.yml

@@ -27,7 +27,11 @@ jobs:
         run: make setup
 
       - name: Run tests against live API
-        run: make test_live_api
+        # Streaming endpoints can intermittently hang or time out for many
+        # minutes. To keep the healthcheck fast and reliable we skip tests
+        # marked with the `streaming` marker in this workflow and run the
+        # non-streaming test target instead.
+        run: make test_live_no_streaming
         id: test
         continue-on-error: true
 

CHANGELOG.rst

@@ -6,6 +6,7 @@ To be released
 
 
 * Deprecate Python 3.9 support - minimum required version is now Python 3.10+. This does not mean the library will not work with Python 3.9, but it will not be tested against it anymore.
+* Added test_live_no_streaming Makefile target and updated live-api-healthcheck GitHub Actions workflow to use it.
 * Updated typing and client behavior for the ``opening_explorer``:
   - Added new typed dicts ``PlayerOpeningStatistic`` and ``MastersOpeningStatistic`` in ``berserk.types.opening_explorer``.
   - Updated ``client.opening_explorer`` method signatures to return the new types.

Makefile

@@ -36,7 +36,10 @@ test_record: ## run tests with pytest and record http requests
 	uv run pytest --record-mode=once tests
 
 test_live_api: ## run tests with live API (no cassettes)
-	uv run pytest --disable-recording --throttle-time=1.0 tests
+	uv run pytest --disable-recording --throttle-time=5.0 tests
+
+test_live_no_streaming: ## run live tests but skip streaming-marked tests
+	uv run pytest --disable-recording --throttle-time=5.0 -m "not streaming" tests
 
 typecheck: ## run type checking with pyright
 	uv run pyright berserk integration/local.py $(ARGS)

pyproject.toml

@@ -67,3 +67,8 @@ build-backend = "uv_build"
 
 [tool.uv.build-backend]
 module-root = ""
+
+[tool.pytest.ini_options]
+markers = [
+    "streaming: test that use a streaming endpoint.",
+]

tests/clients/test_opening_explorer.py

@@ -76,6 +76,7 @@ def test_export(self):
 class TestPlayerGames:
     @pytest.mark.vcr
     @pytest.mark.default_cassette("TestPlayerGames.results.yaml")
+    @pytest.mark.streaming
     def test_wait_for_last_results(self):
         result = Client().opening_explorer.get_player_games(
             player="evachesss", color="white", wait_for_indexing=True
@@ -84,6 +85,7 @@ def test_wait_for_last_results(self):
 
     @pytest.mark.vcr
     @pytest.mark.default_cassette("TestPlayerGames.results.yaml")
+    @pytest.mark.streaming
     def test_get_first_result_available(self):
         result = Client().opening_explorer.get_player_games(
             player="evachesss",
@@ -94,6 +96,7 @@ def test_get_first_result_available(self):
 
     @pytest.mark.vcr
     @pytest.mark.default_cassette("TestPlayerGames.results.yaml")
+    @pytest.mark.streaming
     def test_stream(self):
         iterator = Client().opening_explorer.stream_player_games(
             player="evachesss",