Merge pull request #1639 from Sage-Bionetworks/v25.8.1-rc-dev

Release `v25.8.1`

Author: SageGJ

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "schematicpy"
-version = "25.7.1"
+version = "25.8.1"
 description = "Package for biomedical data model and metadata ingress management"
 authors = [
     "Milen Nikolov <milen.nikolov@sagebase.org>",

schematic/schemas/create_json_schema.py

@@ -281,30 +281,28 @@ def _get_validation_rule_based_fields(
         # Schematic will use the implicit type if the explicit type isn't specified for now,
         #   but this behavior is deprecated and will be removed in the future by SCHEMATIC-326
         implicit_js_type = get_js_type_from_inputted_rules(validation_rules)
-        # If there is an explicit type set...
-        if explicit_js_type:
-            # and the implicit type conflicts with the explicit type, then an exception is raised
-            if explicit_js_type != implicit_js_type:
-                msg = (
-                    f"Property: '{name}', has explicit type: '{explicit_js_type}' "
-                    f"that conflicts with the implicit type: '{implicit_js_type}' "
-                    f"derived from its validation rules: {validation_rules}"
-                )
-                raise ValueError(msg)
-        # If there is no explicit type...
-        else:
-            # and there is an implicit type...
-            if implicit_js_type:
-                # then the implicit type is used...
-                js_type = implicit_js_type
-                # and a warning is raised since this behavior is deprecated
-                msg = (
-                    f"No explicit type set for property: '{name}', "
-                    "using validation rules to set the type. "
-                    "Using validation rules to set type is deprecated. "
-                    "You should set the columnType for this property in your data model."
-                )
-                warnings.warn(msg)
+        # If there is an explicit and implicit type set and the implicit type conflicts with the
+        # explicit type, then an exception is raised
+        if (
+            explicit_js_type
+            and implicit_js_type
+            and explicit_js_type != implicit_js_type
+        ):
+            msg = (
+                f"Property: '{name}', has explicit type: '{explicit_js_type}' "
+                f"that conflicts with the implicit type: '{implicit_js_type}' "
+                f"derived from its validation rules: {validation_rules}"
+            )
+            raise ValueError(msg)
+        if implicit_js_type:
+            js_type = implicit_js_type
+            msg = (
+                f"No explicit type set for property: '{name}', "
+                "using validation rules to set the type. "
+                "Using validation rules to set type is deprecated. "
+                "You should set the columnType for this property in your data model."
+            )
+            warnings.warn(msg)
 
         if ValidationRuleName.URL in validation_rule_names:
             js_format = JSONSchemaFormat.URI

schematic/version.py

@@ -1,3 +1,3 @@
 """Sets the version of the package"""
 # Version hardcoded see https://sagebionetworks.jira.com/browse/SCHEMATIC-229
-__version__ = "25.7.1"
+__version__ = "25.8.1"

schematic_api/Dockerfile

@@ -37,7 +37,7 @@ LABEL version=$TAG
 
 # run open ssl and generate certificate
 RUN apt update && \
-    apt-get install openssl && \
+    apt-get install -y openssl && \
     openssl req -x509 -nodes -days 365 \
     -subj  "/C=CA/ST=QC/O=Company" \
     -newkey rsa:2048 -keyout /etc/ssl/private/localhost.key \

tests/unit/test_create_json_schema.py

@@ -283,6 +283,16 @@ def test_get_validation_rule_based_fields_no_explicit_type(
             None,
             None,
         ),
+        (
+            ["list"],
+            JSONSchemaType.STRING,
+            JSONSchemaType.STRING,
+            True,
+            None,
+            None,
+            None,
+            None,
+        ),
         (
             ["inRange 50 100"],
             JSONSchemaType.NUMBER,
@@ -324,7 +334,7 @@ def test_get_validation_rule_based_fields_no_explicit_type(
             JSONSchemaFormat.URI,
         ),
     ],
-    ids=["No rules", "String", "InRange", "Regex", "Date", "URL"],
+    ids=["No rules", "String", "List string", "InRange", "Regex", "Date", "URL"],
 )
 def test_get_validation_rule_based_fields_with_explicit_type(
     validation_rules: list[str],