Cover all cases in _make_finite and _widget_guess_initial_step

scikit-hep · Dec 19, 2024 · b2f370b · b2f370b
1 parent 3167f0f
commit b2f370b
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/tests/test_qtwidget.py b/tests/test_qtwidget.py
@@ -70,23 +70,26 @@ def assert_call(self):
         mw1.parameters[0].fit.click()
 
     # check changing of limits
-    mw1.parameters[0].fit.click()
-    mw1.parameters[0].tmin.setValue(-1)
-    mw1.parameters[0].tmax.setValue(1)
+    m = Minuit(cost, 0, 0)
+    m.limits["a"] = (-2, 2)
+    mw2 = qtinteractive(m, plot)
+    qtbot.addWidget(mw2)
+    mw2.parameters[0].tmin.setValue(-1)
+    mw2.parameters[0].tmax.setValue(1)
     assert_allclose(m.limits["a"], (-1, 1), atol=1e-5)
     with plot.assert_call():
-        mw1.parameters[0].tmin.setValue(0.5)
+        mw2.parameters[0].tmin.setValue(0.5)
     assert_allclose(m.limits["a"], (0.5, 1), atol=1e-5)
     assert_allclose(m.values, (0.5, 0), atol=1e-5)
-    mw1.parameters[0].tmin.setValue(2)
+    mw2.parameters[0].tmin.setValue(2)
     assert_allclose(m.limits["a"], (0.5, 1), atol=1e-5)
     assert_allclose(m.values, (0.5, 0), atol=1e-5)
-    mw1.parameters[0].tmin.setValue(-1)
+    mw2.parameters[0].tmin.setValue(-1)
     with plot.assert_call():
-        mw1.parameters[0].tmax.setValue(0)
+        mw2.parameters[0].tmax.setValue(0)
     assert_allclose(m.limits["a"], (-1, 0), atol=1e-5)
     assert_allclose(m.values, (0, 0), atol=1e-5)
-    mw1.parameters[0].tmax.setValue(-2)
+    mw2.parameters[0].tmax.setValue(-2)
     assert_allclose(m.limits["a"], (-1, 0), atol=1e-5)
     assert_allclose(m.values, (0, 0), atol=1e-5)