Merge pull request #1 from bknueven/mis_bk

fixing _get_constraint test
DLWoodruff · Mar 1, 2024 · 099eb8b · 099eb8b
2 parents a24432b + 20e4196
commit 099eb8b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/pyomo/contrib/mis/__init__.py b/pyomo/contrib/mis/__init__.py
@@ -10,5 +10,3 @@
 #  ___________________________________________________________________________
 
 from pyomo.contrib.mis.mis import compute_infeasibility_explanation
-# so the tests can find it
-from pyomo.contrib.mis.mis import _get_constraint
diff --git a/pyomo/contrib/mis/tests/test_mis.py b/pyomo/contrib/mis/tests/test_mis.py
@@ -12,6 +12,7 @@
 import pyomo.common.unittest as unittest
 import pyomo.environ as pyo
 import pyomo.contrib.mis as mis
+from pyomo.contrib.mis.mis import _get_constraint
 from pyomo.common.tempfiles import TempfileManager
 
 import logging
@@ -42,9 +43,8 @@ def test_write_mis_ipopt(self):
 
     def test__get_constraint_errors(self):
         # A not-completely-cyincal way to get the coverage up.
-        m = _get_infeasible_model()  # not modified, but who cares?
-        fct = getattr(mis, "_get_constraint")
-        # fct = mis._get_constraint
+        m = _get_infeasible_model()  # not modified
+        fct = _get_constraint
 
         m.foo_slack_plus_ = pyo.Var()
         self.assertRaises(RuntimeError, fct, m, m.foo_slack_plus_)