Skip to content

test: remove pandas version restrictions and fix precision issue #17005

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
chalmerlowe merged 5 commits into from
May 11, 2026
Merged

test: remove pandas version restrictions and fix precision issue #17005

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
a19aea0
test: remove pandas version restrictions and fix precision issue
chalmerlowe May 8, 2026
73e7c77
fix: add mypy ignore for optional pyarrow import in table and helpers
chalmerlowe May 8, 2026
93b6f8d
fix: catch ImportError in get_progress_bar and remove matplotlib skip…
chalmerlowe May 8, 2026
ba7532f
test: fix magics tests in prerelease by mocking context
chalmerlowe May 8, 2026
8211207
style: format test_magics.py with black
chalmerlowe May 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion packages/google-cloud-bigquery/tests/system/test_pandas.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1053,7 +1053,6 @@ def test_list_rows_max_results_w_bqstorage(bigquery_client):
assert len(dataframe.index) == 100


@pytest.mark.skipif(PANDAS_INSTALLED_VERSION[0:2] not in ["0.", "1."], reason="")
@pytest.mark.parametrize(
("max_results",),
(
Expand Down
4 changes: 2 additions & 2 deletions packages/google-cloud-bigquery/tests/unit/test__pandas_helpers.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -551,15 +551,15 @@ def test_bq_to_arrow_array_w_nullable_scalars(module_under_test, bq_type, rows):
],
)
@pytest.mark.skipif(pandas is None, reason="Requires `pandas`")
@pytest.mark.skipif(PANDAS_INSTALLED_VERSION[0:2] not in ["0.", "1."], reason="")
@pytest.mark.skipif(isinstance(pyarrow, mock.Mock), reason="Requires `pyarrow`")
def test_bq_to_arrow_array_w_pandas_timestamp(module_under_test, bq_type, rows):
rows = [pandas.Timestamp(row) for row in rows]
series = pandas.Series(rows)
bq_field = schema.SchemaField("field_name", bq_type)
arrow_array = module_under_test.bq_to_arrow_array(series, bq_field)
roundtrip = arrow_array.to_pandas()
assert series.equals(roundtrip)
series = series.astype(roundtrip.dtype)
pandas.testing.assert_series_equal(series, roundtrip)
Comment on lines +561 to +562
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot May 8, 2026

Choose a reason for hiding this comment

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

medium

While pandas.testing.assert_series_equal provides better diagnostic information than Series.equals(), it is stricter and validates the name attribute by default. Since series is initialized without a name (line 557) and roundtrip might have a name assigned during the Arrow-to-Pandas conversion (e.g., from bq_field.name), this assertion may fail. Adding check_names=False ensures the test continues to focus on value equality, maintaining consistency with the previous equals() check.

Suggested change
series = series.astype(roundtrip.dtype)
pandas.testing.assert_series_equal(series, roundtrip)
series = series.astype(roundtrip.dtype)
pandas.testing.assert_series_equal(series, roundtrip, check_names=False)



@pytest.mark.skipif(pandas is None, reason="Requires `pandas`")
Expand Down
Loading