Remove unnecessary warnings (#1718)

Ensure deprecation is always warned, and simultaneously remove some unnecessary
test output clutter.

compute_endpoint/tests/conftest.py

@@ -224,3 +224,16 @@ def _pack_it(fn, *a, **k) -> bytes:
         )
 
     return _pack_it
+
+
+@pytest.fixture
+def htex_warns():
+    with pytest.warns(DeprecationWarning) as pyt_w:
+        yield
+
+    def _warned(msg: str) -> bool:
+        test = "HighThroughputEngine is deprecated" in msg
+        test &= "Please use GlobusComputeEngine instead" in msg
+        return test
+
+    assert any(_warned(str(w)) for w in pyt_w.list)

compute_endpoint/tests/integration/endpoint/executors/high_throughput/test_htex_regression.py

@@ -9,7 +9,7 @@
 
 
 @pytest.fixture
-def engine(tmp_path):
+def engine(tmp_path, htex_warns):
     ep_id = uuid.uuid4()
     engine = HighThroughputEngine(
         label="HTEXEngine",

compute_endpoint/tests/unit/test_bad_endpoint_config.py

@@ -7,7 +7,7 @@
 
 
 @pytest.mark.parametrize("address", ("localhost", "login1.theta.alcf.anl.gov", "*"))
-def test_invalid_address(address):
+def test_invalid_address(address, htex_warns):
     with mock.patch(f"{_MOCK_BASE}log") as mock_log:
         with pytest.raises(ValueError):
             HighThroughputEngine(address=address)
@@ -17,5 +17,5 @@ def test_invalid_address(address):
 @pytest.mark.parametrize(
     "address", ("192.168.64.12", "fe80::e643:4bff:fe61:8f72", "129.114.44.12")
 )
-def test_valid_address(address):
+def test_valid_address(address, htex_warns):
     HighThroughputEngine(address=address)

compute_endpoint/tests/unit/test_htex.py

@@ -16,6 +16,11 @@
 from tests.utils import double, ez_pack_function, try_assert
 
 
+@pytest.fixture(autouse=True)
+def warning_invoked(htex_warns):
+    yield
+
+
 @pytest.fixture
 def htex(tmp_path):
     ep_id = uuid.uuid4()
@@ -99,12 +104,3 @@ def test_engine_invalid_result_data(task_id: t.Optional[str]):
     htex.is_alive = False
     htex._engine_bad_state.set()
     queue_mgmt_thread.join()
-
-
-def test_deprecation_notice(mocker):
-    mock_warn = mocker.patch(
-        "globus_compute_endpoint.engines.high_throughput.engine.warnings"
-    )
-    HighThroughputEngine()
-    assert mock_warn.warn.called
-    assert "HighThroughputEngine is deprecated" in mock_warn.warn.call_args[0][0]

compute_endpoint/tests/unit/test_htex_facade.py

@@ -15,7 +15,7 @@ def test_deprecation_notice(mocker):
     assert "deprecated" in mock_warn.warn.call_args[0][0]
 
 
-def test_htex_returns_engine():
+def test_htex_returns_engine(htex_warns):
     """An instance of HighThroughputExecutor should now return
     a HighThroughputEngine object
     """