✅ Add tests for planetary coordinates

src/ipyaladin/utils/_coordinate_parser.py

@@ -71,7 +71,7 @@ def _from_name_on_planet(string: str, body: str) -> SkyCoord:
     """
     url = (
         f"https://alasky.cds.unistra.fr/planetary-features/resolve"
-        f"?identifier={string}&body={body}&threshold=0.7&format=json"
+        f"?identifier={string.replace(' ', '+')}&body={body}&threshold=0.7&format=json"
     )
     request = requests.get(url)
     if request.status_code != OK_STATUS_CODE:

src/test/test_aladin.py

@@ -1,7 +1,8 @@
+import requests
 from astropy.coordinates import Angle, SkyCoord
 import numpy as np
 import pytest
-from typing import Callable
+from typing import Callable, Dict
 
 from ipyaladin import Aladin
 from ipyaladin.utils._coordinate_parser import parse_coordinate_string
@@ -18,6 +19,40 @@ def mock_sesame(monkeypatch: Callable) -> None:
     monkeypatch.setattr(SkyCoord, "from_name", lambda _: SkyCoord(0, 0, unit="deg"))
 
 
+class MockResponse:
+    """Mock response object for requests.get."""
+
+    def __init__(self) -> None:
+        self.status_code = 200
+
+    def json(self) -> Dict:
+        """Return a mock JSON response."""
+        return {
+            "data": [
+                [
+                    None,
+                    None,
+                    None,
+                    None,
+                    None,
+                    0,
+                    0,
+                    None,
+                    None,
+                    None,
+                    None,
+                    "",
+                ]
+            ]
+        }
+
+
+@pytest.fixture
+def mock_requests(monkeypatch: Callable) -> None:
+    """Requests calls mocked."""  # noqa: D401
+    monkeypatch.setattr(requests, "get", lambda _: MockResponse())
+
+
 test_aladin_string_target, _ = zip(*test_is_coordinate_string_values)
 
 
@@ -37,6 +72,35 @@ def test_aladin_string_target_set(target: str, mock_sesame: Callable) -> None:
     assert np.isclose(aladin.target.icrs.dec.deg, parsed_target.icrs.dec.deg)
 
 
+test_aladin_planetary_string_target = [
+    "Olympus Mons",
+    "Terra Sabaea",
+    "Promethei Terra",
+]
+
+
+@pytest.mark.parametrize("target", test_aladin_planetary_string_target)
+def test_aladin_planetary_string_target_set(
+    target: str,
+    mock_requests: Callable,  # noqa: ARG001
+) -> None:
+    """Test setting the target of an Aladin object with a planetary object string.
+
+    Parameters
+    ----------
+    target : str
+        The target string.
+
+    """
+    aladin._survey_body = "mars"
+    aladin.target = target
+    parsed_target = parse_coordinate_string(target, body="mars")
+    print(aladin.target)
+    print(parsed_target)
+    assert np.isclose(aladin.target.icrs.ra.deg, parsed_target.icrs.ra.deg)
+    assert np.isclose(aladin.target.icrs.dec.deg, parsed_target.icrs.dec.deg)
+
+
 @pytest.mark.parametrize("target", test_aladin_string_target)
 def test_aladin_sky_coord_target_set(target: str, mock_sesame: Callable) -> None:  # noqa: ARG001
     """Test setting and getting the target of an Aladin object with a SkyCoord object.