Qiskit / QiskitIBMRuntime.jl / 38293801067

10%

push

github

feat: add Base.show() for all classes + tests (#35)

* Document how to run the service tests in the README

* Add Base.show for Service, Job, and Backend

Service, Job, and Backend wrap raw pointers, so Julia's default show leaks an
address that changes between runs.  Define compact show methods so they display
as Service(...), Job(...), and Backend(<name>).  Backend already had a
text/plain method; adding the compact method (used when a Backend is printed
inside a collection) also fixes BackendSearchResults, whose elements
previously leaked their pointers.

Samples and BackendSearchResults are AbstractVectors and already inherit the
standard array printing, so no methods are added for them.

Assert in the pointer constructors (@assert ptr !== C_NULL): NULL instances are
not reachable through the public API, and guarding the constructor protects
every method rather than just display.

* fix: make NULL-pointer guards in constructors actually fire

The @assert ptr !== C_NULL guards were no-ops: !== is identity
comparison, and C_NULL is a Ptr{Nothing} while these fields are
Ptr{QkrtBackend}, Ptr{QkrtJob}, etc. Different pointer types are never
identical, so the assertions passed even for NULL.

Use != (address comparison) so the guards actually catch NULL. This
matters most for Backend: qkrt_backend_search_results_least_busy returns
NULL for an empty result set, so least_busy on an empty search previously
built a Backend around NULL and segfaulted on the first property access.

Also drop the now-redundant Backend text/plain show method (identical to
the compact method, and Base falls back to it) and the redundant
!occursin("Ptr{", ...) test assertion.

0 of 9 new or added lines in 1 file covered. (0.0%)

13 of 133 relevant lines covered (9.77%)

0.47 hits per line