Add enhydris_api_client.put_timeseries() (fixes DEV-130)

Author: aptiko

docs/enhydris_api_client.rst

@@ -67,11 +67,13 @@ Reference
    .. method:: list_timeseries(self, station_id, timeseries_group_id)
                get_timeseries(self, station_id, timeseries_group_id, timeseries_id)
                post_timeseries(self, station_id, timeseries_group_id, data)
+               put_timeseries(self, station_id, timeseries_group_id, timeseries_id, data)
                delete_timeseries(self, station_id, timeseries_group_id, timeseries_id)
 
-      Methods that create, retrieve or delete time series. Similar to
-      the ones for station. :meth:`~EnhydrisApiClient.list_timeseries`
-      returns a list of dictionaries.
+      Methods that create, retrieve, update or delete time series.
+      Similar to the ones for station.
+      :meth:`~EnhydrisApiClient.list_timeseries` returns a list of
+      dictionaries.
 
    .. method:: read_tsdata(self, station_id, timeseries_group_id, timeseries_id, start_date=None, end_date=None, timezone=None)
               .post_tsdata(self, station_id, timeseries_group_id, timeseries_id, ts)

pyproject.toml

@@ -79,4 +79,4 @@ skip = ["_version.py"]
 extra_standard_library = ["cpython", "libc"]
 
 [tool.pyright]
-ignore = ["osgeo.*", "django.contrib.gis.*"]
+ignore = ["osgeo.*"]

src/enhydris_api_client/__init__.py

@@ -244,6 +244,23 @@ def post_timeseries(
         self.check_response()
         return self.response.json()["id"]
 
+    def put_timeseries(
+        self,
+        station_id: int,
+        timeseries_group_id: int,
+        timeseries_id: int,
+        data: JSONDict,
+    ) -> None:
+        self.response = self.session.put(
+            urljoin(
+                self.base_url,
+                f"api/stations/{station_id}/timeseriesgroups/{timeseries_group_id}/"
+                f"timeseries/{timeseries_id}/",
+            ),
+            data=data,
+        )
+        self.check_response()
+
     def delete_timeseries(
         self, station_id: int, timeseries_group_id: int, timeseries_id: int
     ) -> None:

src/evaporation/py.typed

@@ -87,6 +87,17 @@ def test_returns_id(self) -> None:
         self.assertEqual(self.data, 43)
 
 
+@mock_session()
+class PutTimeseriesTestCase(TestCase):
+    def test_makes_request(self, m: mock.MagicMock) -> None:
+        client = EnhydrisApiClient("https://mydomain.com")
+        client.put_timeseries(41, 42, 43, data={"location": "Syria"})
+        m.return_value.put.assert_called_once_with(
+            "https://mydomain.com/api/stations/41/timeseriesgroups/42/timeseries/43/",
+            data={"location": "Syria"},
+        )
+
+
 class FailedPostTimeseriesTestCase(TestCase):
     @mock_session(**{"post.return_value.status_code": 404})
     def test_raises_exception_on_error(self, m: mock.MagicMock) -> None:

src/hspatial/py.typed

@@ -13,7 +13,7 @@
 
 import numpy as np
 import pandas as pd
-from django.contrib.gis.gdal import GDALRaster  # type: ignore
+from django.contrib.gis.gdal import GDALRaster
 from django.contrib.gis.geos import Point as GeoDjangoPoint
 from osgeo import gdal, ogr, osr
 
@@ -154,7 +154,7 @@ def setUp(self) -> None:
         )
 
         # Our grid represents a 70x150m area, lower-left co-ordinates (0, 0).
-        self.dataset.geotransform = (0, 10, 0, 70, 0, -10)
+        self.dataset.geotransform = [0, 10, 0, 70, 0, -10]
 
         self.target_band = self.dataset.bands[1]