Fix timestamp format generated by collection_temporal_extent (#279)

* fix timestamp format in collection_temporal_extent

* add test for temporal extent format

Author: hrodmn

src/pgstac/migrations/pgstac.0.9.0-unreleased.sql

@@ -237,6 +237,18 @@ END;
 $function$
 ;
 
+CREATE OR REPLACE FUNCTION pgstac.collection_temporal_extent(id text)
+ RETURNS jsonb
+ LANGUAGE sql
+ IMMUTABLE PARALLEL SAFE
+ SET search_path TO 'pgstac', 'public'
+AS $function$
+    SELECT to_jsonb(array[array[min(datetime), max(datetime)]])
+    FROM items WHERE collection=$1;
+;
+$function$
+;
+
 CREATE OR REPLACE FUNCTION pgstac.update_collection_extents()
  RETURNS void
  LANGUAGE sql

src/pgstac/migrations/pgstac.unreleased.sql

@@ -2362,7 +2362,7 @@ CREATE OR REPLACE FUNCTION collection_bbox(id text) RETURNS jsonb AS $$
 $$ LANGUAGE SQL IMMUTABLE PARALLEL SAFE SET SEARCH_PATH TO pgstac, public;
 
 CREATE OR REPLACE FUNCTION collection_temporal_extent(id text) RETURNS jsonb AS $$
-    SELECT to_jsonb(array[array[min(datetime)::text, max(datetime)::text]])
+    SELECT to_jsonb(array[array[min(datetime), max(datetime)]])
     FROM items WHERE collection=$1;
 ;
 $$ LANGUAGE SQL IMMUTABLE PARALLEL SAFE SET SEARCH_PATH TO pgstac, public;

src/pgstac/sql/003a_items.sql

@@ -335,7 +335,7 @@ CREATE OR REPLACE FUNCTION collection_bbox(id text) RETURNS jsonb AS $$
 $$ LANGUAGE SQL IMMUTABLE PARALLEL SAFE SET SEARCH_PATH TO pgstac, public;
 
 CREATE OR REPLACE FUNCTION collection_temporal_extent(id text) RETURNS jsonb AS $$
-    SELECT to_jsonb(array[array[min(datetime)::text, max(datetime)::text]])
+    SELECT to_jsonb(array[array[min(datetime), max(datetime)]])
     FROM items WHERE collection=$1;
 ;
 $$ LANGUAGE SQL IMMUTABLE PARALLEL SAFE SET SEARCH_PATH TO pgstac, public;

src/pgstac/tests/basic/crud_functions.sql

@@ -63,3 +63,7 @@ SELECT get_setting_bool('update_collection_extent');
 SELECT update_item((SELECT content || '{"properties":{"datetime":"2024-01-01 00:00:00Z"}}'::jsonb  FROM test_items LIMIT 1));
 
 SELECT content->'extent' FROM collections WHERE id='pgstactest-crudtest';
+
+-- check formatting of temporal extent
+SELECT collection_temporal_extent('pgstactest-crudtest');
+

src/pgstac/tests/basic/crud_functions.sql.out

@@ -100,3 +100,7 @@ SELECT update_item((SELECT content || '{"properties":{"datetime":"2024-01-01 00:
 SELECT content->'extent' FROM collections WHERE id='pgstactest-crudtest';
  {"spatial": {"bbox": [[-85.3792495727539, 30.933948516845703, -85.30819702148438, 31.003555297851562]]}, "temporal": {"interval": [["2023-02-01T00:00:00+00:00", "2023-02-01T00:00:00+00:00"]]}}
 
+-- check formatting of temporal extent
+SELECT collection_temporal_extent('pgstactest-crudtest');
+ [["2023-02-01T00:00:00+00:00", "2024-01-01T00:00:00+00:00"]]
+