chore: Fix two types of warnings in unit tests by chalmerlowe · Pull Request #2183 · googleapis/python-bigquery

@google-labs-jules

This commit addresses two warnings that appear when running unit tests:

1.  `PytestRemovedIn9Warning` in `tests/unit/test_opentelemetry_tracing.py`:
    Removed a `@pytest.mark.skipif` decorator from a fixture. The skip condition
    is already present on the test methods using the fixture.

2.  `FutureWarning` in `tests/unit/test_client.py`:
    Updated calls to `client.query()` to include `job_retry=None` when `job_id`
    is also specified. This is to avoid ambiguity as BigQuery cannot retry a
    failed job with the exact same ID.

@chalmerlowe requested a review from a team

May 14, 2025 13:55

chalmerlowe

@chalmerlowe

chalmerlowe

@chalmerlowe