coverage: refine cuda.core coverage tests comments

Author: rluo8

cuda_core/tests/test_tensor_map.py

@@ -309,14 +309,9 @@ def test_invalid_data_type(self, dev, skip_if_no_tma):
             )
 
     def test_as_tensor_map_host_view_rejected_without_tma(self):
-        """``as_tensor_map`` on a non-device-accessible (host) view fails
-        gracefully with a clear error, without needing TMA-capable hardware.
-
-        This drives the real ``as_tensor_map`` -> ``_from_tiled`` path: every
-        keyword is assembled and the options are coerced before the
-        device-accessibility guard rejects the host pointer, so no monkeypatching
-        is required to cover the forwarding logic.
-        """
+        """``as_tensor_map`` rejects a non-device-accessible (host) view with a
+        clear error, exercising the ``as_tensor_map`` -> ``_from_tiled`` path
+        without needing TMA-capable hardware."""
         host = np.zeros((64, 64), dtype=np.float32)
         view = StridedMemoryView.from_any_interface(host, stream_ptr=-1)
         with pytest.raises(ValueError, match="device-accessible"):

cuda_core/tests/test_utils.py

@@ -1077,11 +1077,9 @@ def test_view_as_cai_device_pointer_and_stream_ordering(init_cuda):
     ``stream`` differs from the consumer stream.
 
     This only exercises the code path and checks *device* correctness (ptr,
-    device_id, shape); it does NOT verify stream-order correctness -- that would
-    need many queued kernels on the producer stream plus an observable
-    dependency on the consumer side. Uses a synthetic CAI object backed by a
-    genuine device allocation, so the cupy/numba-only device branch is exercised
-    without those optional deps.
+    device_id, shape); it does NOT verify stream-order correctness. Uses a
+    synthetic CAI object backed by a genuine device allocation, so the
+    cupy/numba-only device branch is exercised without those optional deps.
     """
     dev = init_cuda
     buffer = dev.memory_resource.allocate(64, stream=dev.default_stream)
@@ -1108,7 +1106,7 @@ def test_strided_memory_view_init_cai_path_deprecated(init_cuda):
     """The deprecated ``StridedMemoryView(obj)`` constructor routes a CAI-only
     object through the CAI branch (warn + ``view_as_cai``), not the DLPack one."""
     obj = _make_cuda_array_interface_obj(shape=(4,), strides=None, typestr="<f4", data=(0, False))
-    with pytest.deprecated_call(match="CUDA-array-interface-supporting"):
+    with pytest.deprecated_call(match="CUDA-array-interface-supporting object is deprecated"):
         view = StridedMemoryView(obj, stream_ptr=-1)
     assert view.is_device_accessible is True
     assert view.shape == (4,)

cuda_core/tests/test_utils_dlpack.py

@@ -2,12 +2,9 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-"""DLPack-focused tests for ``StridedMemoryView``.
-
-Split out of ``test_utils.py`` (which had grown large): export/import
-round-trips, capsule + deleter paths, ``from_dlpack`` error handling, and the
+"""DLPack tests for ``StridedMemoryView``: export/import round-trips, capsule +
+deleter paths, ``from_dlpack`` error handling, and the
 ``__dlpack_c_exchange_api__`` C exchange-API helpers driven through ctypes.
-CAI-only behavior stays in ``test_utils.py``.
 """
 
 import ctypes
@@ -167,12 +164,8 @@ def test_from_dlpack_typeerror_fallback_unversioned_import():
 #
 # Drive the C function pointers exposed by the capsule the way a native
 # consumer would, exercising the StridedMemoryView exchange-API implementation.
-#
-# The C functions report failure by setting a Python error and returning -1.
-# Defining the pointers with PYFUNCTYPE (Python calling convention) lets ctypes
-# propagate that real exception (TypeError/RuntimeError/NotImplementedError)
-# instead of wrapping it in a SystemError, so the tests assert the meaningful
-# type directly.
+# Pointers use PYFUNCTYPE so a failing call raises its real Python exception
+# (TypeError/RuntimeError/NotImplementedError).
 # ---------------------------------------------------------------------------
 
 _PyCapsule_GetPointer = ctypes.pythonapi.PyCapsule_GetPointer
@@ -312,13 +305,11 @@ def test_dlpack_c_exchange_api_to_py_object_null_tensor():
 
 
 def test_dlpack_c_exchange_api_managed_tensor_allocator_not_supported():
-    """``managed_tensor_allocator`` is unsupported (NotImplementedError).
-
-    The implementation sets a Python error even when no ``SetError`` callback is
-    passed, so with PYFUNCTYPE ctypes surfaces the NotImplementedError directly.
-    """
+    """Covers the ``managed_tensor_allocator`` entry point, which is unsupported
+    and only ever raises NotImplementedError (StridedMemoryView never allocates)."""
     api = _get_exchange_api()
     out = ctypes.c_void_p(123)
     with pytest.raises(NotImplementedError, match="not supported"):
+        # Currently sets a Python error when `SetError` isn't passed.
         api.managed_tensor_allocator(None, ctypes.byref(out), None, None)
     assert not out.value  # set to NULL before the error