Skip to content

chore(sqlalchemy-bigquery): restore complete coverage#17770

Merged
ohmayr merged 7 commits into
mainfrom
chore-sqlalchemy-bigquery-coverage-100
Jul 21, 2026
Merged

chore(sqlalchemy-bigquery): restore complete coverage#17770
ohmayr merged 7 commits into
mainfrom
chore-sqlalchemy-bigquery-coverage-100

Conversation

@ohmayr

@ohmayr ohmayr commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

This PR restores coverage for sqlalchemy-bigquery

Towards #17459 🦕

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request increases the test coverage requirement to 100% in .coveragerc and noxfile.py, omits test directories from coverage, and adds a unit test for the deprecated dbapi method. The review feedback suggests improving the unit test by verifying that calling the deprecated dbapi() method correctly raises a deprecation warning.

Comment thread packages/sqlalchemy-bigquery/tests/unit/test_engine.py
@ohmayr
ohmayr force-pushed the fix-sqlalchemy-bigquery-geography-bug branch from e41196b to 2102681 Compare July 20, 2026 08:49
@ohmayr
ohmayr force-pushed the chore-sqlalchemy-bigquery-coverage-100 branch from 3757ee2 to 5f11a2e Compare July 20, 2026 08:50
@ohmayr
ohmayr force-pushed the fix-sqlalchemy-bigquery-geography-bug branch from 1ebe446 to 2e4345c Compare July 20, 2026 18:33
Base automatically changed from fix-sqlalchemy-bigquery-geography-bug to main July 21, 2026 15:46
@ohmayr
ohmayr marked this pull request as ready for review July 21, 2026 18:46
@ohmayr
ohmayr requested review from a team as code owners July 21, 2026 18:46
@ohmayr
ohmayr requested review from GarrettWu and removed request for a team July 21, 2026 18:46
@ohmayr ohmayr changed the title chore(sqlalchemy-bigquery): restore fail_under=100 in .coveragerc and… chore(sqlalchemy-bigquery): restore complete coverage Jul 21, 2026
@ohmayr
ohmayr enabled auto-merge (squash) July 21, 2026 20:00
@ohmayr
ohmayr merged commit d4b76fa into main Jul 21, 2026
39 checks passed
@ohmayr
ohmayr deleted the chore-sqlalchemy-bigquery-coverage-100 branch July 21, 2026 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants