Revert "Add the SshAutoTransport transport plugin (aiidateam#6154)"

This reverts commit 71422eb.

Author: khsrali

pyproject.toml

@@ -191,7 +191,6 @@ requires-python = '>=3.9'
 'core.local' = 'aiida.transports.plugins.local:LocalTransport'
 'core.ssh' = 'aiida.transports.plugins.ssh:SshTransport'
 'core.ssh_async' = 'aiida.transports.plugins.ssh_async:AsyncSshTransport'
-'core.ssh_auto' = 'aiida.transports.plugins.ssh_auto:SshAutoTransport'
 
 [project.entry-points.'aiida.workflows']
 'core.arithmetic.add_multiply' = 'aiida.workflows.arithmetic.add_multiply:add_multiply'

src/aiida/transports/plugins/ssh_auto.py

@@ -977,13 +977,9 @@ def time_use_login_shell(authinfo, auth_params, use_login_shell, iterations) ->
 # comment on 'core.ssh_async':
 # It is important that 'ssh localhost' is functional in your test environment.
 # It should connect without asking for a password.
-# comment on 'core.ssh_auto':
-# It is important that no other options (except for `--safe-interval`) have to be specified for this transport type.
-@pytest.mark.parametrize(
-    'transport_type, config', [('core.ssh_auto', []), ('core.ssh_async', ['--machine-or-host', 'localhost'])]
-)
+@pytest.mark.parametrize('transport_type, config', [('core.ssh_async', ['--machine-or-host', 'localhost'])])
 def test_computer_setup_with_various_transport(run_cli_command, aiida_computer, transport_type, config):
-    """Test setup of computer with ``core.ssh_auto`` and ``core.ssh_async`` entry points.
+    """Test setup of computer with ``core.ssh_async`` entry points.
 
     pass any config option the setup needs in the parameter section``.
     """

tests/cmdline/commands/test_computer.py

@@ -18,6 +18,7 @@
 from aiida.engine.daemon import execmanager
 from aiida.manage import get_manager
 from aiida.orm import CalcJobNode, FolderData, PortableCode, RemoteData, SinglefileData
+from aiida.plugins import entry_point
 from aiida.transports.plugins.local import LocalTransport
 
 
@@ -41,7 +42,7 @@ def file_hierarchy_simple():
     }
 
 
-@pytest.fixture(params=('core.local', 'core.ssh'))
+@pytest.fixture(params=entry_point.get_entry_point_names('aiida.transports'))
 def node_and_calc_info(aiida_localhost, aiida_computer_ssh, aiida_code_installed, request):
     """Return a ``CalcJobNode`` and associated ``CalcInfo`` instance."""
 

tests/engine/daemon/test_execmanager.py

@@ -59,26 +59,16 @@ def tmp_path_local(tmp_path_factory):
 )
 def custom_transport(request, tmp_path_factory, monkeypatch) -> Transport:
     """Fixture that parametrizes over all the registered implementations of the ``CommonRelaxWorkChain``."""
-    plugin = TransportFactory(request.param)
-
     if request.param == 'core.ssh':
         kwargs = {'machine': 'localhost', 'timeout': 30, 'load_system_host_keys': True, 'key_policy': 'AutoAddPolicy'}
-    elif request.param == 'core.ssh_auto':
-        kwargs = {'machine': 'localhost'}
-        # The transport config is store in a independent temporary path per test to not mix up
-        # with the files under operating.
-        filepath_config = tmp_path_factory.mktemp('transport') / 'config'
-        monkeypatch.setattr(plugin, 'FILEPATH_CONFIG', filepath_config)
-        if not filepath_config.exists():
-            filepath_config.write_text('Host localhost')
     elif request.param == 'core.ssh_async':
         kwargs = {
             'machine': 'localhost',
         }
     else:
         kwargs = {}
 
-    return plugin(**kwargs)
+    return TransportFactory(request.param)(**kwargs)
 
 
 def test_is_open(custom_transport):