Adjust testtools usage

requirements-dev.txt

@@ -1,5 +1,7 @@
 black~=22.0
+fixtures
 stestr
+testtools
 pylint~=2.16.2
 astroid~=2.14.2  # Must be kept aligned to what pylint wants
 jinja2==3.0.3
@@ -20,4 +22,4 @@ coverage>=5.5
 ipykernel<=6.21.3
 jupyter-client<=8.0.3
 ipython<8.13.0 ; python_version<"3.9"  # for python 3.8 compatibility
-sphinx-remove-toctrees
+sphinx-remove-toctrees

test/base.py

@@ -22,10 +22,10 @@
 from typing import Any, Callable, Optional
 
 import fixtures
+import testtools
 import uncertainties
 from qiskit.utils.deprecation import deprecate_func
 
-import qiskit_experiments.warnings
 from qiskit_experiments.framework import (
     ExperimentDecoder,
     ExperimentEncoder,
@@ -35,14 +35,14 @@
 from .extended_equality import is_equivalent
 
 # Fail tests that take longer than this
-TEST_TIMEOUT = os.environ.get("TEST_TIMEOUT", 60)
-
-
-# If testtools is installed use that as a (mostly) drop in replacement for
+TEST_TIMEOUT = int(os.environ.get("TEST_TIMEOUT", 60))
+# Use testtools by default as a (mostly) drop in replacement for
 # unittest's TestCase. This will enable the fixtures used for capturing stdout
 # stderr, and pylogging to attach the output to stestr's result stream.
-if qiskit_experiments.warnings.HAS_TESTTOOLS:
-    import testtools
+USE_TESTTOOLS = os.environ.get("QE_USE_TESTOOLS", "TRUE") == "TRUE"
+
+
+if USE_TESTTOOLS:
 
     class BaseTestCase(testtools.TestCase):
         """Base test class."""
@@ -61,7 +61,11 @@ class BaseTestCase(testtools.TestCase):
     class BaseTestCase(unittest.TestCase):
         """Base test class."""
 
-        pass
+        def useFixture(self, fixture):
+            """Shim so that useFixture can be called in subclasses
+
+            The actual fixture is not used when using unittest.
+            """
 
 
 # TODO: copy enforce_subclasses_call decorator?