issue693 detect type null in schema

Author: ElienVandermaesenVITO

openeo/internal/processes/parse.py

@@ -53,6 +53,9 @@ def get_enum_options(self):
                         result = copy.deepcopy(item["enum"])
                     else:
                         result += item["enum"]
+                elif "type" in item:
+                    if item["type"] == "null":
+                        result += [None]
         elif isinstance(self.schema,dict):
             if "enum" in self.schema:
                 result = self.schema["enum"]

openeo/rest/datacube.py

@@ -2773,7 +2773,7 @@ def sar_backscatter(
                 "coefficient"
             )
             schema = parameter.schema
-            coefficient_options = schema.get_enum_options() + [None]
+            coefficient_options = schema.get_enum_options()
             if coefficient == _UNSET:
                 coefficient = parameter.default
         except Exception as e:

tests/internal/processes/test_parse.py

@@ -55,22 +55,23 @@ def test_schema_accepts_geojson(schema, expected):
                         "mode",
                         "near",
                     ],
-                }
+                },
             ),
             ["average", "bilinear", "cubic", "cubicspline", "lanczos", "max", "med", "min", "mode", "near"],
         ),
         (
-            Schema([{"type": "string", "enum": ["replicate", "reflect", "reflect_pixel", "wrap"]}, {"type": "number"}]),
-            ["replicate", "reflect", "reflect_pixel", "wrap"],
+            Schema([{"type": "string", "enum": ["replicate", "reflect", "reflect_pixel", "wrap"]}, {"type": "null"}]),
+            ["replicate", "reflect", "reflect_pixel", "wrap", None],
         ),
         (
             Schema(
                 [
                     {"type": "string", "enum": ["replicate", "reflect"]},
+                    {"type": "null"},
                     {"type": "number", "enum": ["reflect_pixel", "wrap"]},
                 ]
             ),
-            ["replicate", "reflect", "reflect_pixel", "wrap"],
+            ["replicate", "reflect", None, "reflect_pixel", "wrap"],
         ),
     ],
 )