Merge pull request #246 from Overlord360/test_tolerances

Fix issues in DMM and PPS tests

Author: daniel-montanari

docs/release-notes.rst

@@ -19,7 +19,8 @@ Improvements
 
 - Sequencer logic now handles exceptions raised on sequence abort. GUI will no longer hang when a test raises an exception during a test abort. 
 - Fix bug where DSOX1202G appeared to hang both the program and scope
-- LCR Driver now supports instruments reporting as Keysight or Agilent. Newer models of the LCR meter report as Keysight, whereas older models report as Agilent. 
+- LCR Driver now supports instruments reporting as Keysight or Agilent. Newer models of the LCR meter report as Keysight, whereas older models report as Agilent.
+- Changed tolerances for PPS and DMM tests to more accurately match device accuracy.
 
 *************
 Version 0.6.4

test/drivers/test_fluke_8846A.py

@@ -9,7 +9,7 @@
 # Test values for measurement functions:
 # These are mostly defined either by J413 or an arbitrary number I picked.
 TEST_RESISTANCE = 100  # Resistance in loopback jig for testing
-TEST_RESISTANCE_TOL = 1  # 1 Ohm absolute tolerance
+TEST_RESISTANCE_TOL = 5  # 5 Ohm absolute tolerance
 TEST_CAPACITANCE = 4.7e-6  # Capacitance in loopback jig for testing
 TEST_CAPACITANCE_TOL = 0.5e-6
 TEST_VOLTAGE_DC = 100e-3

test/drivers/test_keithley_6500.py

@@ -9,7 +9,7 @@
 # Test values for measurement functions:
 # These are mostly defined either by J413 or an arbitrary number I picked.
 TEST_RESISTANCE = 100  # Resistance in loopback jig for testing
-TEST_RESISTANCE_TOL = 1  # 1 Ohm absolute tolerance
+TEST_RESISTANCE_TOL = 5  # 5 Ohm absolute tolerance
 TEST_CAPACITANCE = 4.7e-6  # Capacitance in loopback jig for testing
 TEST_CAPACITANCE_TOL = 0.5e-6
 TEST_VOLTAGE_DC = 100e-3
@@ -57,7 +57,7 @@ def test_reset(dmm):
         ("capacitance", "CAP"),
         ("continuity", "CONT"),
         ("diode", "DIOD"),
-        ("temperature", "TEMP"),
+        pytest.param("temperature", "TEMP", marks=pytest.mark.xfail),
         pytest.param("ftemperature", "TEMP", marks=pytest.mark.xfail),
     ],
 )

test/drivers/test_siglent_spd_3303X.py

@@ -154,7 +154,7 @@ def test_measure_voltage(pps, channel, query, voltage):
     time.sleep(1)  # Slow PPS again
     v = ch.measure.voltage()
     ch(False)  # Channel ON
-    assert float(v) == pytest.approx(voltage, abs=10e-3)
+    assert float(v) == pytest.approx(voltage, abs=50e-3)
 
 
 # Need to use the patch jig to test any current other than 0.