fix: Ordinal is now a required property

Author: Alan Christie

decoder/job-definition-schema.yaml

@@ -415,6 +415,7 @@ definitions:
         minimum: 1
     required:
     - name
+    - ordinal
 
   # A test input, a string or array of strings
   test-input:

tests/test_validate_job_schema.py

@@ -257,7 +257,10 @@ def test_validate_test_run_groups():
     demo_job: Dict[str, Any] = text["jobs"]["demo"]
     demo_job["tests"] = {
         "basic-1": {
-            "run-groups": [{"name": "demo-1"}, {"name": "demo-2", "ordinal": 1}],
+            "run-groups": [
+                {"name": "demo-1", "ordinal": 1},
+                {"name": "demo-2", "ordinal": 1},
+            ],
         },
     }
 
@@ -266,3 +269,20 @@ def test_validate_test_run_groups():
 
     # Assert
     assert error is None
+
+
+def test_validate_test_run_groups_without_ordinal():
+    # Arrange
+    text: Dict[str, Any] = deepcopy(_MINIMAL)
+    demo_job: Dict[str, Any] = text["jobs"]["demo"]
+    demo_job["tests"] = {
+        "basic-1": {
+            "run-groups": [{"name": "demo-1"}, {"name": "demo-2"}],
+        },
+    }
+
+    # Act
+    error = decoder.validate_job_schema(text)
+
+    # Assert
+    assert error == "'ordinal' is a required property"