Improvement: Do not dump raw object during decoding (#169)

psyclaudeZ · web-flow · commit f03ace623b23 · 2025-09-04T18:28:24.000Z
Do not dump raw object during decoding
diff --git a/conjure_python_client/_serde/decoder.py b/conjure_python_client/_serde/decoder.py
@@ -89,8 +89,8 @@ def check_null_field(
             deserialized[python_arg_name] = {}
         else:
             raise Exception(
-                "field {} not found in object {}".format(
-                    field_definition.identifier, obj
+                "field {} not found in the object".format(
+                    field_definition.identifier
                 )
             )
 
diff --git a/test/serde/test_decode_object.py b/test/serde/test_decode_object.py
@@ -59,7 +59,7 @@ def test_object_with_missing_field_should_throw_helpful_exception():
         ConjureDecoder().read_from_string(
             """{"path": "bar", "redundant": 1}""", CreateDatasetRequest
         )
-    message_regex = re.compile("field fileSystemId not found in object {.+}")
+    message_regex = re.compile("field fileSystemId not found in the object")
     assert message_regex.match(str(excinfo.value)), excinfo.value
 
 

Original file line number	Diff line number	Diff line change
`@@ -89,8 +89,8 @@ def check_null_field(`
`89`	`89`	`deserialized[python_arg_name] = {}`
`90`	`90`	`else:`
`91`	`91`	`raise Exception(`
`92`		`- "field {} not found in object {}".format(`
`93`		`- field_definition.identifier, obj`
	`92`	`+ "field {} not found in the object".format(`
	`93`	`+ field_definition.identifier`
`94`	`94`	`)`
`95`	`95`	`)`
`96`	`96`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ def test_object_with_missing_field_should_throw_helpful_exception():`
`59`	`59`	`ConjureDecoder().read_from_string(`
`60`	`60`	`"""{"path": "bar", "redundant": 1}""", CreateDatasetRequest`
`61`	`61`	`)`
`62`		`- message_regex = re.compile("field fileSystemId not found in object {.+}")`
	`62`	`+ message_regex = re.compile("field fileSystemId not found in the object")`
`63`	`63`	`assert message_regex.match(str(excinfo.value)), excinfo.value`
`64`	`64`
`65`	`65`