Skip to content

Commit

Permalink
Remove another test for this.
Browse files Browse the repository at this point in the history
  • Loading branch information
JoeZiminski committed Apr 11, 2024
1 parent d42da14 commit 846b50e
Showing 1 changed file with 0 additions and 35 deletions.
35 changes: 0 additions & 35 deletions tests/tests_integration/test_configs.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,41 +94,6 @@ def test_bad_path_syntax(self, project, bad_pattern, path_type, tmp_path):

assert "must contain the full folder path with no " in str(e.value)

def test_additional_error_text_when_ssh_used(
self, no_cfg_project, non_existent_path, existent_path
):
"""
If SSH is used as `connection_method`, if `local_path` does not exist
an extra message is printed to warn to check the `central_path`, because
it cannot be checked.
Currently if SSH is used and the central path does not exist,
no error is raised because it is not possible to check.
"""
with pytest.raises(BaseException) as e:
no_cfg_project.make_config_file(
non_existent_path / no_cfg_project.project_name,
existent_path / no_cfg_project.project_name,
"ssh",
central_host_id="fake_id",
central_host_username="fake_username",
)

assert (
"Also make sure the central_path` is correct, as datashuttle "
"cannot check it via SSH at this stage." in str(e.value)
)

# This should not raise an error, even though the path does not
# exist, because it is not possible to check over SSH.
no_cfg_project.make_config_file(
existent_path / no_cfg_project.project_name,
non_existent_path / no_cfg_project.project_name,
"ssh",
central_host_id="fake_id",
central_host_username="fake_username",
)

def test_no_ssh_options_set_on_make_config_file(self, no_cfg_project):
"""
Check that program will assert if not all ssh options
Expand Down

0 comments on commit 846b50e

Please sign in to comment.