Fix typechecking of tests

Author: jenshnielsen

tests/parameter/test_combined_parameter_extended.py

@@ -31,8 +31,9 @@ def test_combine_creates_combined_parameter(
     def test_combine_with_label_and_unit(self, two_params: list[Parameter]) -> None:
         """combine() passes label and unit through."""
         cp = combine(*two_params, name="xy", label="X and Y", unit="V")
-        assert cp.parameter.label == "X and Y"
-        assert cp.parameter.unit == "V"
+        # cp.parameter is a parameter like object but these attributes are dynamically added
+        assert cp.parameter.label == "X and Y"  # pyright: ignore[reportFunctionMemberAccess]
+        assert cp.parameter.unit == "V"  # pyright: ignore[reportFunctionMemberAccess]
 
     def test_combine_with_aggregator(self, two_params: list[Parameter]) -> None:
         """combine() passes aggregator through."""
@@ -108,15 +109,17 @@ def test_units_deprecated(
         with caplog.at_level(logging.WARNING):
             cp = CombinedParameter(two_params, name="xy", units="mV")
         assert any("`units` is deprecated" in msg for msg in caplog.messages)
-        assert cp.parameter.unit == "mV"
+        # cp.parameter is a parameter like object but these attributes are dynamically added
+        assert cp.parameter.unit == "mV"  # pyright: ignore[reportFunctionMemberAccess]
 
     def test_units_deprecated_unit_takes_precedence(
         self, two_params: list[Parameter], caplog: pytest.LogCaptureFixture
     ) -> None:
         """When both unit and units are given, unit takes precedence."""
         with caplog.at_level(logging.WARNING):
             cp = CombinedParameter(two_params, name="xy", unit="V", units="mV")
-        assert cp.parameter.unit == "V"
+        # cp.parameter is a parameter like object but these attributes are dynamically added
+        assert cp.parameter.unit == "V"  # pyright: ignore[reportFunctionMemberAccess]
 
     def test_invalid_name_raises(self, two_params: list[Parameter]) -> None:
         """Invalid parameter name raises ValueError."""

tests/test_channel_extended.py

@@ -178,7 +178,7 @@ class OtherChannel(InstrumentChannel):
     ct1 = ch_instr.channels
     ct2 = ChannelTuple(ch_instr, "other", OtherChannel)
     with pytest.raises(TypeError, match="same type"):
-        ct1 + ct2
+        _ = ct1 + ct2
 
 
 @pytest.mark.serial
@@ -190,7 +190,7 @@ def test_channel_tuple_add_different_parent() -> None:
         ct1 = instr1.channels[0:1]
         ct2 = instr2.channels[0:1]
         with pytest.raises(ValueError, match="same parent"):
-            ct1 + ct2
+            _ = ct1 + ct2
     finally:
         instr1.close()
         instr2.close()

tests/test_instrument_extended.py

@@ -116,8 +116,8 @@ def test_ask_wraps_exception() -> None:
 @pytest.mark.serial
 def test_close_all() -> None:
     """close_all should remove all registered instruments."""
-    DummyInstrument(name="closeall1", gates=["g1"])
-    DummyInstrument(name="closeall2", gates=["g2"])
+    _ = DummyInstrument(name="closeall1", gates=["g1"])
+    _ = DummyInstrument(name="closeall2", gates=["g2"])
 
     assert Instrument.exist("closeall1")
     assert Instrument.exist("closeall2")