PB-1210: Set default file path for assets - #patch

app/stac_api/models.py

@@ -628,6 +628,19 @@ def filename(self):
     def __str__(self):
         return self.name
 
+    def get_asset_path(self):
+        # Method must be implemented by Asset and CollectionAsset separately.
+        raise NotImplementedError("get_asset_path() not implemented")
+
+    def save(self, *args, **kwargs):
+        # Default file value to the asset path.
+        #
+        # This is the behaviour when creating an asset via PUT API endpoint.
+        # But we need to set this here so it also applies in the admin UI.
+        if not bool(self.file):
+            self.file = self.get_asset_path()
+        super().save(*args, **kwargs)
+
     def delete(self, *args, **kwargs):  # pylint: disable=signature-differs
         try:
             super().delete(*args, **kwargs)

app/tests/test_admin_page.py

@@ -272,6 +272,9 @@ def _create_asset(self, item, extra=None):
             Asset.objects.filter(item=item, name=data["name"]).exists(),
             msg="Admin page asset added not found in DB"
         )
+        asset = Asset.objects.get(item=item, name=data["name"])
+        # Assert that the filename is set to the value in name
+        self.assertEqual(asset.filename, data['name'])
 
         data = {
             "item": item.id,