Tests on prerelease and semver check (#119)

Fix bug for semver check with the poetry syntax of pre release versions and test the pre-releases/ branches
mercator-ocean · Aug 29, 2024 · 84d2dd3 · 84d2dd3
1 parent d1359b1
commit 84d2dd3
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 11 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -2,7 +2,10 @@ name: Run tests
 
 on:
   pull_request:
-    branches: [ "main", "copernicusmarine-toolbox-v2" ]
+    branches:
+      - "main"
+      - "copernicusmarine-toolbox-v2"
+      - "pre-releases/**"
 
 jobs:
   tests:

diff --git a/copernicusmarine/core_functions/versions_verifier.py b/copernicusmarine/core_functions/versions_verifier.py
@@ -43,16 +43,23 @@ def _check_version(function_name: str, staging: bool):
             function_name
         ]:
             required_version = marine_data_store_versions[service]
-            if not semver.Version.parse(client_version).match(
-                required_version
-            ):
-                logger.debug(
-                    f"Client version {client_version} is not compatible with "
-                    f"{service}. Service needs version {required_version}."
-                )
-                logger.error(
-                    f"Client version {client_version} is not compatible with current "
-                    "backend service. Please update to the latest client version."
+            try:
+                if not semver.Version.parse(client_version).match(
+                    required_version
+                ):
+                    logger.debug(
+                        f"Client version {client_version} is not compatible with "
+                        f"{service}. Service needs version {required_version}."
+                    )
+                    logger.error(
+                        f"Client version {client_version} is not "
+                        f"compatible with current backend service. "
+                        f"Please update to the latest client version."
+                    )
+            except ValueError:
+                logger.warning(
+                    f"Using a pre-release or a non-official version "
+                    f"of the client. Client version: {client_version}"
                 )
 
     @staticmethod