From 937d0aa0a44030a3135ab97e65bff64e940b18d4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ra=C3=BAl=20Cumplido?= <raulcumplido@gmail.com>
Date: Mon, 26 Aug 2024 19:25:37 +0200
Subject: [PATCH] Add comment on why we duplicate test_basics_np_required for
 float16

---
 python/pyarrow/tests/test_scalars.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/python/pyarrow/tests/test_scalars.py b/python/pyarrow/tests/test_scalars.py
index e1862e6d43c92..3f4a53c473e7e 100644
--- a/python/pyarrow/tests/test_scalars.py
+++ b/python/pyarrow/tests/test_scalars.py
@@ -100,6 +100,9 @@ def test_basics(value, ty, klass, pickle_module):
     assert wr() is None
 
 
+# This test is a copy of test_basics but only for float16 (HalfFloatScalar)
+# which currently requires a numpy scalar to create it. The test collection
+# fails if numpy is used on the parametrization when not present.
 @pytest.mark.numpy
 def test_basics_np_required(pickle_module):
     value, ty, klass = np.float16(1.0), pa.float16(), pa.HalfFloatScalar