Standardize use of in_testing_environment(), building_documentation()

doc/OnlineDocs/conf.py

@@ -301,4 +301,7 @@ def check_output(self, want, got, optionflags):
     asl_available = False
     ma27_available = False
     mumps_available = False
+
+from pyomo.common.flags import in_testing_environment
+in_testing_environment(True)
 '''

pyomo/common/dependencies.py

@@ -19,9 +19,10 @@
 from types import ModuleType
 from typing import List
 
-from .deprecation import deprecated, deprecation_warning, in_testing_environment
+import pyomo
+from .deprecation import deprecated, deprecation_warning
 from .errors import DeferredImportError
-
+from .flags import in_testing_environment, building_documentation
 
 SUPPRESS_DEPENDENCY_WARNINGS = False
 
@@ -240,12 +241,12 @@ def UnavailableClass(unavailable_module):
 
     class UnavailableMeta(type):
         def __getattr__(cls, name):
-            if 'sphinx' in sys.modules:
+            if building_documentation():
                 # If we are building documentation, avoid the
                 # DeferredImportError (we will still raise one if
                 # someone attempts to *create* an instance of this
                 # class)
-                super().__getattr__(name)
+                return getattr(super(), name)
             raise DeferredImportError(
                 unavailable_module._moduleunavailable_message(
                     f"The class attribute '{cls.__name__}.{name}' is not available "

pyomo/common/deprecation.py

@@ -228,7 +228,7 @@ def deprecation_warning(
     logger.warning(msg)
 
 
-if in_testing_environment():
+if in_testing_environment() or building_documentation():
     deprecation_warning.emitted_warnings = None
 else:
     deprecation_warning.emitted_warnings = set()

pyomo/common/log.py

@@ -28,21 +28,19 @@
 from pyomo.version.info import releaselevel
 from pyomo.common.deprecation import deprecated
 from pyomo.common.fileutils import PYOMO_ROOT_DIR
+from pyomo.common.flags import in_testing_environment
 from pyomo.common.formatting import wrap_reStructuredText
 
 _indentation_re = re.compile(r'\s*')
 
-_RTD_URL = "https://pyomo.readthedocs.io/en/%s/errors.html" % (
-    'stable'
-    if (releaselevel == 'final' or 'sphinx' in sys.modules or 'Sphinx' in sys.modules)
-    else 'latest'
-)
-
 
 def RTD(_id):
     _id = str(_id).lower()
     assert _id[0] in 'wex'
-    return f"{_RTD_URL}#{_id}"
+    return "https://pyomo.readthedocs.io/en/%s/errors.html#%s" % (
+        'stable' if (releaselevel == 'final' or in_testing_environment()) else 'latest',
+        _id,
+    )
 
 
 _DEBUG = logging.DEBUG

pyomo/contrib/simplemodel/__init__.py

@@ -9,7 +9,8 @@
 #  This software is distributed under the 3-clause BSD License.
 #  ___________________________________________________________________________
 
-from pyomo.common.deprecation import deprecation_warning, in_testing_environment
+from pyomo.common.deprecation import deprecation_warning
+from pyomo.common.flags import in_testing_environment
 
 try:
     deprecation_warning(

pyomo/pysp/__init__.py

@@ -11,7 +11,8 @@
 
 import logging
 import sys
-from pyomo.common.deprecation import deprecation_warning, in_testing_environment
+from pyomo.common.deprecation import deprecation_warning
+from pyomo.common.flags import in_testing_environment
 
 try:
     # Warn the user