Fix: versions in conditions are not validated

bananas_api/helpers/api_schema.py

@@ -144,6 +144,20 @@ def validate_conditions(self, data, **kwargs):
                     f"Condition can only mark the first client-version this version does or doesn't work for;"
                     f" expected '>= VERSION' or '< VERSION', got '{data[0]}'."
                 )
+
+            if data[0].startswith(">="):
+                condition = data[0][3:]
+            else:
+                condition = data[0][2:]
+
+            # Check if all parts of the version are integers
+            try:
+                [int(p) for p in condition.split(".")]
+            except ValueError:
+                raise ValidationError(
+                    f"Versions in a condition should be a stable release in the form of '12.0' or '1.8.0',"
+                    f" got '{condition}'."
+                )
         else:
             if not data[0].startswith(">= "):
                 raise ValidationError(
@@ -156,6 +170,24 @@ def validate_conditions(self, data, **kwargs):
                     f" expected '< VERSION', got '{data[0]}'."
                 )
 
+            # Check if all parts of the version are integers
+            try:
+                [int(p) for p in data[0][3:].split(".")]
+            except ValueError:
+                raise ValidationError(
+                    f"Versions in a condition should be a stable release in the form of '12.0' or '1.8.0',"
+                    f" got '{data[0][3:]}'."
+                )
+
+            # Check if all parts of the version are integers
+            try:
+                [int(p) for p in data[1][2:].split(".")]
+            except ValueError:
+                raise ValidationError(
+                    f"Versions in a condition should be a stable release in the form of '12.0' or '1.8.0',"
+                    f" got '{data[1][2:]}'."
+                )
+
 
 class VersionMinimized(Global):
     read_only = ["upload_date", "md5sum_partial", "filesize", "license", "availability"]

regression/002_success_new_version.yaml

@@ -10,6 +10,11 @@ steps:
   tags:
   - "tag1"
   - "tag2"
+  compatibility:
+  - name: vanilla
+    conditions:
+    - ">= 12.0"
+    - "< 13.0"
 - file-upload: valid2.grf
 - file-upload: readme.txt
 - file-upload: changelog.txt
@@ -22,6 +27,11 @@ steps:
   tags:
   - "tag1"
   - "tag2"
+  compatibility:
+  - name: vanilla
+    conditions:
+    - ">= 12.0"
+    - "< 13.0"
   md5sum-partial: "fc03854a"
   content-type: "newgrf"
   unique-id: "4e4d4c01"

regression/953_condition_invalid_version.yaml

@@ -0,0 +1,11 @@
+steps:
+- api: user/login
+- api: new-package/start
+- file-upload: valid.grf
+- api: new-package/update
+  compatibility:
+  - name: vanilla
+    conditions:
+    - ">= 1.9.0rc1"
+  error:
+    compatibility.conditions: "Versions in a condition should be a stable release in the form of '12.0' or '1.8.0', got '1.9.0rc1'."

regression/954_condition_invalid_version_second.yaml

@@ -0,0 +1,12 @@
+steps:
+- api: user/login
+- api: new-package/start
+- file-upload: valid.grf
+- api: new-package/update
+  compatibility:
+  - name: vanilla
+    conditions:
+    - ">= 1.8.0"
+    - "< 1.9.0rc1"
+  error:
+    compatibility.conditions: "Versions in a condition should be a stable release in the form of '12.0' or '1.8.0', got '1.9.0rc1'."