STAC updates (#1105)

* STAC: use asset keys if they exist; update native STAC item response media type evaluation
* fix test

Author: tomkralidis

pycsw/core/metadata.py

@@ -1890,7 +1890,7 @@ def _parse_stac_resource(context, repos, record):
             }
 
             if link.get('title') is not None:
-                new_link['name'] = link.get('title')
+                new_link['name'] = key
                 new_link['description'] = link.get('title')
             if link.get('type') is not None:
                 new_link['protocol'] = link.get('type')

pycsw/ogc/api/records.py

@@ -1128,7 +1128,7 @@ def record2json(record, url, collection, mode='ogcapi-records'):
     :returns: `dict` of record GeoJSON
     """
 
-    if record.metadata_type == 'application/json':
+    if record.metadata_type in ['application/json', 'application/geo+json']:
         rec = json.loads(record.metadata)
         if rec.get('stac_version') is not None and rec['type'] == 'Feature' and mode == 'stac-api':
             LOGGER.debug('Returning native STAC representation')

pycsw/stac/api.py

@@ -565,7 +565,12 @@ def links2stacassets(collection, record):
         if 'assets' not in record:
             record['assets'] = {}
 
-        for count, value in enumerate(links_assets):
-            record['assets'][count] = value
+        for count, asset in enumerate(links_assets):
+            if 'name' in asset:
+                asset_key = asset.pop('name')
+            else:
+                asset_key = count
+
+            record['assets'][asset_key] = asset
 
     return record

tests/functionaltests/suites/stac_api/test_stac_api_functional.py

@@ -247,6 +247,12 @@ def test_item(config):
     assert content['stac_version'] == '1.0.0'
     assert content['collection'] == 'S2MSI2A'
 
+    assert content['geometry']['coordinates'][0][0][0] == 16.33660021997006
+
+    assert 'assets' in content
+    assert 'B02' in content['assets']
+    assert 'product-metadata' in content['assets']
+
     for link in content['links']:
         assert 'href' in link
         assert 'rel' in link
@@ -275,7 +281,7 @@ def test_json_transaction(config, sample_collection, sample_item,
 
     assert content['id'] == '20201211_223832_CS2'
     assert content['geometry'] is None
-    assert content['properties']['datetime'] == '2020-12-11T22:38:32Z'
+    assert content['properties']['datetime'] == '2020-12-11T22:38:32.125000Z'
     assert content['collection'] == 'metadata:main'
 
     # update item

tests/unittests/test_util.py

@@ -1,8 +1,10 @@
 # =================================================================
 #
 # Authors: Ricardo Garcia Silva <ricardo.garcia.silva@gmail.com>
+# Authors: Tom Kralidis
 #
 # Copyright (c) 2017 Ricardo Garcia Silva
+# Copyright (c) 2025 Tom Kralidis
 #
 # Permission is hereby granted, free of charge, to any person
 # obtaining a copy of this software and associated documentation
@@ -148,7 +150,7 @@ def test_wkt2geom(wkt, bounds, expected):
         "-10.0, 10.0, 30.0, 15.0",
         "POLYGON((-10.00 10.00, -10.00 15.00, 30.00 15.00, "
         "30.00 10.00, -10.00 10.00))"
-    ),
+    )
 ])
 def test_bbox2wktpolygon(bbox, expected):
     result = util.bbox2wktpolygon(bbox)
@@ -367,6 +369,20 @@ def test_programmatic_import_with_invalid_path(invalid_import_path):
     result = util.programmatic_import(invalid_import_path)
     assert result is None
 
+
 def test_sanitize_url():
     result = util.sanitize_db_connect("postgresql://username:password@localhost/pycsw")
     assert result == "postgresql://***:***@localhost/pycsw"
+
+
+def test_str2bool():
+    assert util.str2bool('true')
+    assert util.str2bool(True)
+    assert util.str2bool('1')
+    assert util.str2bool('yes')
+    assert util.str2bool('on')
+    assert not util.str2bool('0')
+    assert not util.str2bool('false')
+    assert not util.str2bool(False)
+    assert not util.str2bool('off')
+    assert not util.str2bool('no')