remove unnecessary test fixtures for message pubsub

canvas-medical · Jan 24, 2025 · 1be3823 · 1be3823
1 parent 9659288
commit 1be3823
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/plugin_runner/tests/test_plugin_runner.py b/plugin_runner/tests/test_plugin_runner.py
@@ -278,9 +278,8 @@ async def test_handle_plugin_event_returns_expected_result(
 
 
 @pytest.mark.asyncio
-@pytest.mark.parametrize("install_test_plugin", ["example_plugin"], indirect=True)
 async def test_reload_plugins_event_handler_successfully_publishes_message(
-    install_test_plugin: Path, plugin_runner: PluginRunner
+    plugin_runner: PluginRunner,
 ) -> None:
     """Test ReloadPlugins Event handler successfully publishes a message with restart action."""
     with patch(
@@ -299,10 +298,7 @@ async def test_reload_plugins_event_handler_successfully_publishes_message(
 
 
 @pytest.mark.asyncio
-@pytest.mark.parametrize("install_test_plugin", ["test_module_imports_plugin"], indirect=True)
-async def test_synchronize_plugins_calls_install_and_load_plugins(
-    install_test_plugin: Path, plugin_runner: PluginRunner
-) -> None:
+async def test_synchronize_plugins_calls_install_and_load_plugins() -> None:
     """Test that synchronize_plugins calls install_plugins and load_plugins."""
     with (
         patch("plugin_runner.plugin_runner.get_client", new_callable=MagicMock) as mock_get_client,