edit error message; edit test

Lingling Peng · Lingling Peng · commit 0e85642e8ee2 · 2025-12-10T11:14:25.000-05:00
diff --git a/synapseclient/models/form.py b/synapseclient/models/form.py
@@ -160,7 +160,7 @@ async def create_my_form_data():
 
         if not self.group_id or not self.name or not self.data_file_handle_id:
             raise ValueError(
-                "'group_id', 'name', and 'data_file_handle_id' must be provided to create a FormData."
+                "Missing required fields: 'group_id', 'name', and 'data_file_handle_id' are required to create a FormData."
             )
 
         form_change_request = FormChangeRequest(
diff --git a/tests/integration/synapseclient/models/async/test_form_async.py b/tests/integration/synapseclient/models/async/test_form_async.py
@@ -106,7 +106,7 @@ async def test_create_raise_error_on_missing_fields(self, syn: Synapse) -> None:
             await form_data.create_async(synapse_client=syn)
         assert (
             str(e.value)
-            == "'group_id', 'name', and 'data_file_handle_id' must be provided to create a FormData."
+            == "Missing required fields: 'group_id', 'name', and 'data_file_handle_id' are required to create a FormData."
         )
 
     async def test_list_form_data_reviewer_false(
diff --git a/tests/integration/synapseclient/models/synchronous/test_form.py b/tests/integration/synapseclient/models/synchronous/test_form.py
@@ -104,7 +104,7 @@ def test_create_raise_error_on_missing_fields(self, syn: Synapse) -> None:
             form_data.create(synapse_client=syn)
         assert (
             str(e.value)
-            == "'group_id', 'name', and 'data_file_handle_id' must be provided to create a FormData."
+            == "Missing required fields: 'group_id', 'name', and 'data_file_handle_id' are required to create a FormData."
         )
 
     def test_list_form_data_reviewer_false(

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ async def create_my_form_data():`
`160`	`160`
`161`	`161`	`if not self.group_id or not self.name or not self.data_file_handle_id:`
`162`	`162`	`raise ValueError(`
`163`		`- "'group_id', 'name', and 'data_file_handle_id' must be provided to create a FormData."`
	`163`	`+ "Missing required fields: 'group_id', 'name', and 'data_file_handle_id' are required to create a FormData."`
`164`	`164`	`)`
`165`	`165`
`166`	`166`	`form_change_request = FormChangeRequest(`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ async def test_create_raise_error_on_missing_fields(self, syn: Synapse) -> None:`
`106`	`106`	`await form_data.create_async(synapse_client=syn)`
`107`	`107`	`assert (`
`108`	`108`	`str(e.value)`
`109`		`- == "'group_id', 'name', and 'data_file_handle_id' must be provided to create a FormData."`
	`109`	`+ == "Missing required fields: 'group_id', 'name', and 'data_file_handle_id' are required to create a FormData."`
`110`	`110`	`)`
`111`	`111`
`112`	`112`	`async def test_list_form_data_reviewer_false(`
Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ def test_create_raise_error_on_missing_fields(self, syn: Synapse) -> None:`
`104`	`104`	`form_data.create(synapse_client=syn)`
`105`	`105`	`assert (`
`106`	`106`	`str(e.value)`
`107`		`- == "'group_id', 'name', and 'data_file_handle_id' must be provided to create a FormData."`
	`107`	`+ == "Missing required fields: 'group_id', 'name', and 'data_file_handle_id' are required to create a FormData."`
`108`	`108`	`)`
`109`	`109`
`110`	`110`	`def test_list_form_data_reviewer_false(`