Handle null when serializing enum to string #6342
Conversation
|
Flakey integ test failure |
|
Can we add a changelog entry? Changelog entry is not required only for house keeping changes such as test code or README update |
|
Can we add a test case in the protocol test , may be src/main/resources/software/amazon/awssdk/protocol/suites/cases/json-core-input.json ? {
"description": "List of enums with null values are marshalled correctly without NPE",
"given": {
"input": {
"ListOfEnums": [
"EnumValue1",
null,
"EnumValue2"
]
}
},
"when": {
"action": "marshall",
"operation": "AllTypes"
},
"then": {
"serializedAs": {
"body": {
"jsonEquals": "{\"ListOfEnums\": [\"EnumValue1\", null, \"EnumValue2\"]}"
}
}
}
}same for maps so we know what we are sending on the wire |
|
|
This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one. |



Motivation and Context
#3861
Modifications
Updated
MemberCopierSpec.javato generate a null check before calling `entry.toString() when serializing enums to strings.Testing