updated models to support background images

This was mainly added so that a background image could be applied
to a protocol/input for the recently added cavas attribute at
ReproNim/reproschema-ui#363

Potentially allows for background use in other activities b/c
it was easier to minimally implement and/or one could add
a background image/logo or something for their own protocol.

Author: bendhouseart

reproschema/cli.py

@@ -302,3 +302,9 @@ def reproschema2fhir(reproschema_questionnaire, output):
 
         with open(output_path / f"{file_name}/{file_name}.json", "w+") as f:
             f.write(json.dumps(fhir_questionnaire))
+
+
+if __name__ == "__main__":
+    import sys
+
+    main(prog_name="reproschema")

reproschema/models/model.py

@@ -10,7 +10,13 @@
 from pydantic.version import VERSION as PYDANTIC_VERSION
 
 if int(PYDANTIC_VERSION[0]) >= 2:
-    from pydantic import BaseModel, ConfigDict, Field, field_validator
+    from pydantic import (
+        BaseModel,
+        ConfigDict,
+        Field,
+        field_validator,
+        model_validator,
+    )
 else:
     from pydantic import BaseModel, Field, validator
 metamodel_version = "None"
@@ -178,6 +184,9 @@ class AdditionalProperty(Thing):
     An object to describe the various properties added to assessments and Items.
     """
 
+    # Override parent's extra="forbid" to allow extra fields
+    model_config = ConfigDict(extra="allow")
+
     allow: Optional[List[AllowedType]] = Field(
         default_factory=list,
         title="allow",
@@ -229,6 +238,14 @@ class AdditionalProperty(Thing):
         title="UI",
         description="An element to control UI specifications. Originally @nest in jsonld, but using a class in the model.",
     )
+
+    # Add backgroundImage as an explicit field to allow it
+    backgroundImage: Optional[str] = Field(
+        None,
+        title="backgroundImage",
+        description="Background image for drawing activities.",
+    )
+
     id: Optional[str] = Field(
         None,
         description="A unique identifier for an entity. Must be either a CURIE shorthand for a URI or a complete URI.",

reproschema/models/tests/test_schema.py

@@ -4,6 +4,7 @@
 
 import pytest
 from pyld import jsonld
+from pydantic import ValidationError
 
 from ...jsonldutils import load_file
 from ...utils import start_server, stop_server
@@ -176,3 +177,88 @@ def test_item(tmp_path, server_http_kwargs):
     )
     del data_comp["@context"]
     assert item_dict == data_comp
+
+
+def test_canvas_activity_allows_extra_fields():
+    """Test that canvas activities allow backgroundImage (now an explicit field)."""
+    activity_dict = {
+        "category": "Activity",
+        "id": "canvas_activity.jsonld",
+        "prefLabel": {"en": "Canvas Drawing Activity"},
+        "description": {"en": "A canvas drawing activity"},
+        "schemaVersion": "1.0.0-rc4",
+        "version": "0.0.1",
+        "ui": {
+            "inputType": "canvas",
+            "addProperties": [
+                {
+                    "isAbout": "item1",
+                    "variableName": "canvas_item",
+                    "backgroundImage": "./images/drawaclock.png",
+                }
+            ],
+        },
+    }
+
+    # This should work fine
+    activity_obj = Activity(**activity_dict)
+    assert (
+        activity_obj.ui.addProperties[0].backgroundImage
+        == "./images/drawaclock.png"
+    )
+
+
+def test_background_image_always_allowed():
+    """Test that backgroundImage is always allowed regardless of inputType."""
+    activity_dict = {
+        "category": "Activity",
+        "id": "radio_activity_with_background.jsonld",
+        "prefLabel": {"en": "Radio Button Activity with Background"},
+        "description": {"en": "A radio button activity with background image"},
+        "schemaVersion": "1.0.0-rc4",
+        "version": "0.0.1",
+        "ui": {
+            "inputType": "radio",
+            "addProperties": [
+                {
+                    "isAbout": "item1",
+                    "variableName": "radio_item",
+                    # backgroundImage should be allowed for all activities
+                    "backgroundImage": "./images/some_image.png",
+                }
+            ],
+        },
+    }
+
+    # This should work fine - backgroundImage is now an explicit field
+    activity_obj = Activity(**activity_dict)
+    assert (
+        activity_obj.ui.addProperties[0].backgroundImage
+        == "./images/some_image.png"
+    )
+
+
+def test_activity_without_extra_fields_works():
+    """Test that activities without extra fields work normally regardless of input type."""
+    activity_dict = {
+        "category": "Activity",
+        "id": "normal_activity.jsonld",
+        "prefLabel": {"en": "Normal Activity"},
+        "description": {"en": "A normal activity without extra fields"},
+        "schemaVersion": "1.0.0-rc4",
+        "version": "0.0.1",
+        "ui": {
+            "inputType": "radio",
+            "addProperties": [
+                {
+                    "isAbout": "item1",
+                    "variableName": "normal_item",
+                    # No extra fields
+                }
+            ],
+        },
+    }
+
+    # This should work fine
+    activity_obj = Activity(**activity_dict)
+    assert activity_obj.ui.addProperties[0].variableName == "normal_item"