Skip to content

GH-55458: [Python] Mark NumPy-dependent tests - #55483

Open
09Catho wants to merge 1 commit into
apache:mainfrom
09Catho:fix/numpy-test-markers-55458
Open

09Catho wants to merge 1 commit into
apache:mainfrom
09Catho:fix/numpy-test-markers-55458

Conversation

@09Catho

@09Catho 09Catho commented Nov 10, 2026

Copy link
Copy Markdown

Rationale for this change

Four tests use NumPy but are missing pytest.mark.numpy. Without that marker, the no-NumPy job does not exercise them and the tests can fail with an AttributeError when an off-by-default group runs without NumPy.

What changes are included in this PR?

Add pytest.mark.numpy to the three large-memory conversion tests and the CUDA-backed test_non_cpu_array test named in #55458.

Are these changes tested?

  • python -m py_compile python/pyarrow/tests/test_convert_builtin.py python/pyarrow/tests/test_array.py passed.
  • An AST check confirmed all four target functions have the NumPy marker.
  • git diff --check passed.
  • The focused pytest commands could not collect because this shallow source checkout does not contain the generated pyarrow.lib extension. No pytest result is claimed.

Are there any user-facing changes?

No. This only corrects test selection for environments without NumPy.

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

Copilot AI balanced review requested due to automatic review settings November 10, 2026 16:52

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #55458 has been automatically assigned in GitHub to PR creator.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants