Remove optional tests in favor of returns or throws

Open-EO · Oct 30, 2023 · 0155298 · 0155298
1 parent 83ddc2c
commit 0155298
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 11 deletions.
diff --git a/tests/README.md b/tests/README.md
@@ -210,16 +210,15 @@ properties:
           type: object
           additionalProperties:
             description: An argument, can be of any type
-        optional:
-          description: Marks optional test so a failure produces only a warning.
-          type: boolean
-          default: false
         returns:
           description: The return value, can be of any type
         delta:
           description: If set to a positive number the equality of the actual return value and the expected return value is checked against a delta value to circumvent problems with floating-point inaccuracy.
           type: number
         throws:
+          description: >-
+            Specifies whether the execution is meant to throw an exception.
+            If used in combination with a return value, it makes the test effectively optional.
           oneOf:
             - description: Specify an exception name from the process specification
               type: string

diff --git a/tests/array_append.json5 b/tests/array_append.json5
@@ -76,9 +76,8 @@
       "returns": [
         "a",
         "b"
-        "3"
-      ],
-      "optional": true
+        "c"
+      ]
     },
     {
       "arguments": {
@@ -94,7 +93,7 @@
         "3"
       ],
       // mixing data types is optional
-      "optional": true
+      "throws": true
     },
     {
       "arguments": {

diff --git a/tests/array_concat.json5 b/tests/array_concat.json5
@@ -42,7 +42,7 @@
       ]
     },
     {
-      // Concatenates two arrays containing different data type, may not always be supported.
+      // Concatenates two arrays containing different data type
       "arguments": {
         "array1": [
           "a",
@@ -59,7 +59,8 @@
         1,
         2
       ],
-      "optional": true
+      // May not always be supported
+      "throws": true
     },
     {
       // Check that a normal array and a labeled array lead to a normal array (labels get dropped)

diff --git a/tests/is_nodata.json5 b/tests/is_nodata.json5
@@ -14,7 +14,7 @@
       },
       "returns": false,
       // Optional to avoid an error in case the no-data value is 1
-      "optional": true
+      "throws": true
     },
     {
       "arguments": {