Make the updating keywords in the editing experiment accept insensible searching and add some controlled keywords tests.

Author: EstelleDa

src/mavedb/routers/experiments.py

@@ -459,6 +459,7 @@ async def update_experiment(
     item.raw_read_identifiers = raw_read_identifiers
 
     if item_update.keywords:
+        keywords: list[ExperimentControlledKeywordAssociation] = []
         all_labels_none = all(k.keyword.label is None for k in item_update.keywords)
         if all_labels_none is False:
             # Users may choose part of keywords from dropdown menu. Remove not chosen keywords from the list.
@@ -467,10 +468,18 @@ async def update_experiment(
                 validate_keyword_list(filtered_keywords)
             except ValidationError as e:
                 raise HTTPException(status_code=422, detail=str(e))
-            try:
-                await item.set_keywords(db, filtered_keywords)
-            except Exception as e:
-                raise HTTPException(status_code=500, detail=f"Invalid keywords: {str(e)}")
+            for upload_keyword in filtered_keywords:
+                try:
+                    description = upload_keyword.description
+                    controlled_keyword = search_keyword(db, upload_keyword.keyword.key, upload_keyword.keyword.label)
+                    experiment_controlled_keyword = ExperimentControlledKeywordAssociation(
+                        controlled_keyword=controlled_keyword,
+                        description=description,
+                    )
+                    keywords.append(experiment_controlled_keyword)
+                except ValueError as e:
+                    raise HTTPException(status_code=422, detail=str(e))
+        item.keyword_objs = keywords
 
     item.modified_by = user_data.user
 

tests/helpers/constants.py

@@ -427,7 +427,7 @@
         "special": False,
         "description": "Description",
     },
-    {"key": "Delivery method", "label": "Other", "special": False, "description": "Description"},
+    {"key": "Delivery Method", "label": "Other", "special": False, "description": "Description"},
     {
         "key": "Phenotypic Assay Mechanism",
         "label": "Other",
@@ -442,6 +442,13 @@
         "special": False,
         "description": "Description",
     },
+    {
+        "key": "Phenotypic Assay Profiling Strategy",
+        "label": "Shotgun sequencing",
+        "code": None,
+        "special": False,
+        "description": "Description",
+    },
 ]
 
 TEST_KEYWORDS = [
@@ -470,7 +477,7 @@
         },
     },
     {
-        "keyword": {"key": "Delivery method", "label": "Other", "special": False, "description": "Description"},
+        "keyword": {"key": "Delivery Method", "label": "Other", "special": False, "description": "Description"},
         "description": "Details of delivery method",
     },
 ]
@@ -492,7 +499,7 @@
     "methodText": "Methods",
     "keywords": [
         {
-            "keyword": {"key": "Delivery method", "label": "Other", "special": False, "description": "Description"},
+            "keyword": {"key": "Delivery Method", "label": "Other", "special": False, "description": "Description"},
             "description": "Details of delivery method",
         },
     ],
@@ -572,7 +579,7 @@
     "keywords": [
         {
             "recordType": "ExperimentControlledKeyword",
-            "keyword": {"key": "Delivery method", "label": "Other", "special": False, "description": "Description"},
+            "keyword": {"key": "Delivery Method", "label": "Other", "special": False, "description": "Description"},
             "description": "Details of delivery method",
         },
     ],
@@ -587,6 +594,51 @@
     "numScoreSets": 0,  # NOTE: This is context-dependent and may need overriding per test
 }
 
+TEST_EXPERIMENT_WITH_UPDATE_KEYWORD_RESPONSE = {
+    "recordType": "Experiment",
+    "title": "Test Experiment Title",
+    "shortDescription": "Test experiment",
+    "abstractText": "Abstract",
+    "methodText": "Methods",
+    "createdBy": {
+        "recordType": "User",
+        "firstName": TEST_USER["first_name"],
+        "lastName": TEST_USER["last_name"],
+        "orcidId": TEST_USER["username"],
+    },
+    "modifiedBy": {
+        "recordType": "User",
+        "firstName": TEST_USER["first_name"],
+        "lastName": TEST_USER["last_name"],
+        "orcidId": TEST_USER["username"],
+    },
+    "creationDate": date.today().isoformat(),
+    "modificationDate": date.today().isoformat(),
+    "scoreSetUrns": [],
+    "contributors": [],
+    "keywords": [
+        {
+            "recordType": "ExperimentControlledKeyword",
+            "keyword": {
+                "key": "Phenotypic Assay Profiling Strategy",
+                "label": "Shotgun sequencing",
+                "special": False,
+                "description": "Description"
+            },
+            "description": "Details of phenotypic assay profiling strategy",
+        },
+    ],
+    "doiIdentifiers": [],
+    "primaryPublicationIdentifiers": [],
+    "secondaryPublicationIdentifiers": [],
+    "rawReadIdentifiers": [],
+    # keys to be set after receiving response
+    "urn": None,
+    "experimentSetUrn": None,
+    "officialCollections": [],
+    "numScoreSets": 0,  # NOTE: This is context-dependent and may need overriding per test
+}
+
 TEST_EXPERIMENT_WITH_KEYWORD_HAS_DUPLICATE_OTHERS_RESPONSE = {
     "recordType": "Experiment",
     "title": "Test Experiment Title",
@@ -622,7 +674,7 @@
         },
         {
             "recordType": "ExperimentControlledKeyword",
-            "keyword": {"key": "Delivery method", "label": "Other", "special": False, "description": "Description"},
+            "keyword": {"key": "Delivery Method", "label": "Other", "special": False, "description": "Description"},
             "description": "Description",
         },
     ],