diff --git a/test/test_helpers.py b/test/test_helpers.py index 5e4246cbf86f9fbb59659a3d8003d421a055c9f3..623c0bf8b1bec735ab422e31c0e5783b5c711a3a 100644 --- a/test/test_helpers.py +++ b/test/test_helpers.py @@ -135,39 +135,38 @@ class TestPrepareHost: @mock.patch("os.getlogin", return_value="testUser") @mock.patch("os.path.exists", return_value=True) def test_prepare_host(self, mock_host, mock_user, mock_path): - path = prepare_host() - assert path == "/home/testUser/machinelearningtools/data/toar_daily/" - path = prepare_host() - assert path == "/home/testUser/Data/toar_daily/" - path = prepare_host() - assert path == "/home/testUser/Data/toar_daily/" - path = prepare_host() - assert path == "/p/project/cjjsc42/testUser/DATA/toar_daily/" - path = prepare_host() - assert path == "/p/home/jusers/testUser/juwels/intelliaq/DATA/toar_daily/" - path = prepare_host() - assert path == '/home/testUser/machinelearningtools/data/toar_daily/' + assert prepare_host() == "/home/testUser/machinelearningtools/data/toar_daily/" + assert prepare_host() == "/home/testUser/Data/toar_daily/" + assert prepare_host() == "/home/testUser/Data/toar_daily/" + assert prepare_host() == "/p/project/cjjsc42/testUser/DATA/toar_daily/" + assert prepare_host() == "/p/home/jusers/testUser/juwels/intelliaq/DATA/toar_daily/" + assert prepare_host() == '/home/testUser/machinelearningtools/data/toar_daily/' @mock.patch("socket.gethostname", return_value="NotExistingHostName") @mock.patch("os.getlogin", return_value="zombie21") - def test_error_handling(self, mock_user, mock_host): + def test_error_handling_unknown_host(self, mock_user, mock_host): with pytest.raises(OSError) as e: prepare_host() assert "unknown host 'NotExistingHostName'" in e.value.args[0] + + @mock.patch("os.getlogin", return_value="zombie21") + @mock.patch("src.helpers.check_path_and_create", side_effect=PermissionError) + def test_error_handling(self, mock_cpath, mock_user): # if "runner-6HmDp9Qd-project-2411-concurrent" not in platform.node(): - mock_host.return_value = "linux-aa9b" + # mock_host.return_value = "linux-aa9b" with pytest.raises(NotADirectoryError) as e: prepare_host() - assert "does not exist for host 'linux-aa9b'" in e.value.args[0] + assert PyTestRegex(r"path '.*' does not exist for host '.*'\.") == e.value.args[0] with pytest.raises(NotADirectoryError) as e: prepare_host(False) - assert "does not exist for host 'linux-aa9b'" in e.value.args[0] + # assert "does not exist for host 'linux-aa9b'" in e.value.args[0] + assert PyTestRegex(r"path '.*' does not exist for host '.*'\.") == e.value.args[0] @mock.patch("socket.gethostname", side_effect=["linux-aa9b", "ZAM144", "zam347", "jrtest", "jwtest", "runner-6HmDp9Qd-project-2411-concurrent"]) @mock.patch("os.getlogin", side_effect=OSError) @mock.patch("os.path.exists", return_value=True) - def test_os_error(self, mock_host, mock_user, mock_path): + def test_os_error(self, mock_path, mock_user, mock_host): path = prepare_host() assert path == "/home/default/machinelearningtools/data/toar_daily/" path = prepare_host()