fix(searcher): respect get-file-locations --server parameter (#155)

Fixes issue where `get-file-locations --server https://example.org`
would ignore the custom server and revert to the default HTTPS server.

Closes #153

Author: tiborsimko

cernopendata_client/searcher.py

@@ -201,7 +201,7 @@ def get_files_list(
     if searcher_protocol == "http":
         new_server = server
     elif searcher_protocol == "https":
-        new_server = SERVER_HTTPS_URI
+        new_server = server
 
     for file_ in record_json["metadata"].get("files", []):
         files_list.append(
@@ -225,7 +225,7 @@ def get_files_list(
         else:
             files_list.append(
                 (
-                    f"{SERVER_HTTPS_URI}/record/{record_json['metadata']['recid']}/file_index/{file_['key']}",
+                    f"{new_server}/record/{record_json['metadata']['recid']}/file_index/{file_['key']}",
                     file_["size"],
                     "",
                 )

tests/test_cli_get_file_locations.py

@@ -106,3 +106,14 @@ def test_get_file_locations_with_https_server_xrootd_protocol():
     )
     assert test_result.exit_code == 0
     assert "root://eospublic.cern.ch//eos/" in test_result.output
+
+
+def test_get_file_locations_with_custom_https_server():
+    """Test `get-file-locations --server` command for custom https server."""
+    test_get_file_locations = CliRunner()
+    test_result = test_get_file_locations.invoke(
+        get_file_locations, ["--recid", 5500, "--server", "https://opendata-qa.cern.ch"]
+    )
+    assert test_result.exit_code == 0
+    assert "https://opendata-qa.cern.ch" in test_result.output
+    assert "https://opendata.cern.ch" not in test_result.output