Remove AssetsEndpoint.update

Author: rycwo

metafold/assets.py

@@ -115,30 +115,6 @@ def create(
             fp.close()
         return Asset(**r.json())
 
-    def update(
-        self, asset_id: str,
-        f: Union[str, bytes, PathLike, IO[bytes]],
-        project_id: Optional[str] = None,
-    ) -> Asset:
-        """Update an asset.
-
-        Args:
-            asset_id: ID of asset to update.
-            f: File-like object (opened in binary mode) or path to file on disk.
-            project_id: Asset project ID.
-
-        Returns:
-            Updated asset resource.
-        """
-        project_id = self._client.project_id(project_id)
-        fp: IO[bytes] = _open_file(f)
-        try:
-            url = f"/projects/{project_id}/assets/{asset_id}"
-            r: Response = self._client.patch(url, files={"file": fp})
-        finally:
-            fp.close()
-        return Asset(**r.json())
-
     def delete(self, asset_id: str, project_id: Optional[str] = None) -> None:
         """Delete an asset.
 

metafold/jobs.py

@@ -38,7 +38,10 @@ class Job:
         default=None,
     )
     state: str
-    assets: list[Asset] = field(converter=_assets)
+    assets: Optional[list[Asset]] = field(
+        converter=lambda v: optional(_assets)(v),
+        default=None,
+    )
     meta: dict[str, Any]
 
 

tests/test_assets.py

@@ -171,19 +171,6 @@ def test_create_asset(client):
     )
 
 
-def test_update_asset(client):
-    with open(test_file, "rb") as f:
-        a = client.assets.update("1", f)
-    assert a == Asset(
-        id="1",
-        filename="test.png",
-        size=67,
-        checksum="sha256:089ad5bf4831b6758e9907db43bc5ebba2e9248a9929dad6132c49932e538278",
-        created=default_dt,
-        modified=default_dt,
-    )
-
-
 def test_delete_asset(client):
     # FIXME: Assert something
     client.assets.delete("1")

tests/test_jobs.py

@@ -39,7 +39,6 @@
         },
         "created": "Mon, 01 Jan 2024 00:00:00 GMT",
         "state": "success",
-        "assets": [asset_json],
         "meta": None,
     },
     {
@@ -51,7 +50,6 @@
         },
         "created": "Mon, 01 Jan 2024 00:00:00 GMT",
         "state": "success",
-        "assets": [asset_json],
         "meta": None,
     },
     {
@@ -63,7 +61,6 @@
         },
         "created": "Mon, 01 Jan 2024 00:00:00 GMT",
         "state": "success",
-        "assets": [asset_json],
         "meta": None,
     },
 ]
@@ -103,7 +100,8 @@ def do_GET(self):
             self.send_response(HTTPStatus.OK)
             self.send_header("Content-Type", "application/json")
             self.end_headers()
-            payload = job_list[-1]
+            payload = deepcopy(job_list[-1])
+            payload["assets"] = [asset_json]
             self.wfile.write(json.dumps(payload).encode())
         elif u.path == "/projects/1/jobs/1/status":
             global poll_count
@@ -145,7 +143,10 @@ def do_PATCH(self):
             self.send_header("Content-Type", "application/json")
             self.end_headers()
             payload = deepcopy(job_list[-1])
-            payload["name"] = "baz"
+            payload.update({
+                "name": "baz",
+                "assets": [asset_json],
+            })
             self.wfile.write(json.dumps(payload).encode())
         else:
             self.send_error(HTTPStatus.NOT_FOUND)