fix regex bug

Author: andrewelamb

synapseclient/models/schema_organization.py

@@ -1161,7 +1161,9 @@ def _check_semantic_version(self, version: str) -> None:
         Raises:
             ValueError: If the string is not a correct semantic version
         """
-        if not re.match("^(\d+)\.(\d+)\.([1-9]\d*)$", version):
+        if version == "0.0.0":
+            raise ValueError("Schema version must start at '0.0.1' or higher")
+        if not re.match("^(\d+)\.(\d+)\.(\d+)$", version):
             raise ValueError(
                 (
                     "Schema version must be a semantic version with no letters "

tests/unit/synapseclient/models/synchronous/unit_test_schema_organization.py

@@ -125,6 +125,25 @@ def test_fill_from_dict(self) -> None:
         assert js.created_by == "2"
         assert js.uri == "org.name-name"
 
+    @pytest.mark.parametrize("version", ["1.0.0", "0.0.1", "0.1.0"])
+    def test_check_semantic_version(self, version: str) -> None:
+        "Tests that only correct versions are allowed"
+        js = JSONSchema()
+        js._check_semantic_version(version)
+
+    def test_check_semantic_version_with_exceptions(self) -> None:
+        "Tests that only correct versions are allowed"
+        js = JSONSchema()
+        with pytest.raises(
+            ValueError, match="Schema version must start at '0.0.1' or higher"
+        ):
+            js._check_semantic_version("0.0.0")
+        with pytest.raises(
+            ValueError,
+            match="Schema version must be a semantic version with no letters and a major, minor and patch version",
+        ):
+            js._check_semantic_version("0.0.1.rc")
+
 
 class TestCreateSchemaRequest:
     @pytest.mark.parametrize(