Skip to content

Conversation

@Reranko05
Copy link

@Reranko05 Reranko05 commented Jan 19, 2026

This PR reorganizes Python array tests by moving NumPy-specific tests out of
python/pyarrow/tests/test_array.py into a new file,
python/pyarrow/tests/test_array_numpy.py.

The goal is to keep test_array.py focused on core Arrow array behavior while
isolating tests that depend on NumPy, which is an optional runtime dependency.

What this PR does:

  • Moves tests explicitly marked with @pytest.mark.numpy into a separate file
  • Keeps test logic, assertions, and coverage unchanged
  • Reduces the size and scope of test_array.py
  • Improves test organization and maintainability

What this PR does NOT do:

  • No behavioral or functional changes
  • No new tests added
  • No changes to NumPy or Arrow semantics

Related issue: GH-43510

Comment on lines 1631 to 1646
















Copy link
Member

Choose a reason for hiding this comment

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

Can you remove these empty lines?

Copy link
Author

Choose a reason for hiding this comment

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

Removed the extra blank lines. Thanks for pointing it out.

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting review Awaiting review labels Jan 20, 2026
@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Jan 20, 2026
@Reranko05 Reranko05 requested a review from raulcd January 20, 2026 16:26
Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

There are several linting failures, that's why the CI job is failing, can you take a look?
https://github.com/apache/arrow/actions/runs/21179073114/job/60943377596?pr=48899

@Reranko05
Copy link
Author

Hello @raulcd, I will look into it and get back to you.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants