ci: re-enable pytest, splitting offline tests from live ones

The pytest workflow was disabled because almost every test fetches from
Yahoo and live fetches fail intermittently on CI IPs. But 60 of 211 tests
are offline unit tests that never touch the network; conftest.py auto-marks
the live ones `network`, so PRs run `pytest -m "not network"` while the live
suite moves to a scheduled job. data.py also retries empty 2xx bodies (the
throttling failure mode) under the existing `retries` config, enabled for
tests in context.py.

Author: dokson

.github/workflows/pytest-network.yml

@@ -0,0 +1,49 @@
+name: Pytest (live)
+
+on:
+  schedule:
+    - cron: '0 6 * * 1'
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  network:
+    # Live tests against Yahoo. Flaky by nature, so scheduled-only (off PRs).
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        python-version: ["3.6", "3.7", "3.8", "3.9", "3.10", "3.11", "3.12", "3.13"]
+
+    steps:
+    - name: Checkout code
+      # Verify: curl -s "https://api.github.com/repos/actions/checkout/commits?sha=v7&until=$(date -u -d '7 days ago' '+%Y-%m-%dT%H:%M:%SZ')&per_page=1" | jq -r '.[0].sha'
+      uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
+      with:
+        persist-credentials: false
+
+    - name: Install uv
+      # Verify: curl -s "https://api.github.com/repos/astral-sh/setup-uv/commits?sha=v9.0.0&until=$(date -u -d '7 days ago' '+%Y-%m-%dT%H:%M:%SZ')&per_page=1" | jq -r '.[0].sha'
+      uses: astral-sh/setup-uv@c771a70e6277c0a99b617c7a806ffedaca235ff9 # v9.0.0
+      with:
+        version: "latest"
+        enable-cache: true
+        cache-dependency-glob: "pyproject.toml"
+
+    - name: Set up Python ${{ matrix.python-version }}
+      run: uv python install ${{ matrix.python-version }}
+
+    - name: Create virtual environment
+      run: uv venv
+
+    - name: Install dependencies
+      run: uv pip install -e ".[dev]"
+
+    - name: Run non-cache tests
+      run: uv run pytest -m network --ignore tests/test_cache.py --ignore tests/test_price_repair.py
+
+    - name: Run cache tests
+      run: |
+        uv run pytest tests/test_cache.py::TestCache
+        uv run pytest tests/test_cache.py::TestCacheNoPermission

.github/workflows/pytest.yml.disabled .github/workflows/pytest.yml.github/workflows/pytest.yml.disabled renamed to .github/workflows/pytest.yml

@@ -3,16 +3,16 @@ name: Pytest
 on:
   pull_request:
     branches:
-      - master
       - main
       - dev
 
+permissions:
+  contents: read
+
 jobs:
-  test:
+  offline:
+    # Deterministic tests with no Yahoo access.
     runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        python-version: ["3.6", "3.7", "3.8", "3.9", "3.10", "3.11", "3.12", "3.13"]
 
     steps:
     - name: Checkout code
@@ -29,14 +29,14 @@ jobs:
         enable-cache: true
         cache-dependency-glob: "pyproject.toml"
 
-    - name: Set up Python ${{ matrix.python-version }}
-      run: uv python install ${{ matrix.python-version }}
+    - name: Set up Python
+      run: uv python install 3.12
 
     - name: Create virtual environment
       run: uv venv
 
     - name: Install dependencies
       run: uv pip install -e ".[dev]"
 
-    - name: Run tests
-      run: uv run pytest tests/ -x -q --tb=short --ignore=tests/test_price_repair.py
+    - name: Run offline tests
+      run: uv run pytest -m "not network"

pyproject.toml

@@ -79,6 +79,9 @@ yfinance = ["pricing.proto", "pricing_pb2.py"]
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
+markers = [
+    "network: requires live Yahoo Finance network access",
+]
 
 [tool.setuptools]
 include-package-data = true

tests/conftest.py

@@ -0,0 +1,15 @@
+import pytest
+
+# Live tests import tests.context; the rest are offline unit tests. Auto-mark
+# the live ones so PR CI can run `-m "not network"`.
+_OFFLINE_MODULES = {
+    "test_auth", "test_data", "test_http",
+    "test_live", "test_screener", "test_utils",
+}
+
+
+def pytest_collection_modifyitems(config, items):
+    for item in items:
+        module = item.module.__name__.rsplit(".", 1)[-1]
+        if module not in _OFFLINE_MODULES:
+            item.add_marker(pytest.mark.network)

tests/context.py

@@ -5,9 +5,13 @@
 import sys
 import os
 import yfinance
+from yfinance.config import YfConfig
 # from requests_ratelimiter import LimiterSession
 # from pyrate_limiter import Duration, RequestRate, Limiter
 
+# Retry throttled/empty responses from Yahoo on CI IPs.
+YfConfig.network.retries = 2
+
 _parent_dp = os.path.abspath(os.path.join(os.path.dirname(__file__), '..'))
 _src_dp = _parent_dp
 sys.path.insert(0, _src_dp)

yfinance/data.py

@@ -27,6 +27,11 @@ def _is_transient_error(exception):
     }
     return error_type_name in transient_error_types
 
+
+def _is_empty_response(response):
+    """Empty 2xx body (Yahoo throttling); retryable."""
+    return response.status_code < 400 and not response.content
+
 cache_maxsize = 64
 
 
@@ -448,6 +453,9 @@ def _make_request(self, url, request_method, body=None, params=None, timeout=30,
         for attempt in range(YfConfig.network.retries + 1):
             try:
                 response = request_method(**request_args)
+                if _is_empty_response(response) and attempt < YfConfig.network.retries:
+                    _time.sleep(2 ** attempt)
+                    continue
                 break
             except Exception as e:
                 if _is_transient_error(e) and attempt < YfConfig.network.retries: