add restXml tests for flattened list of structures#781
add restXml tests for flattened list of structures#781srchase merged 1 commit intosmithy-lang:mainfrom
Conversation
|
It's not clear if the |
|
#789 was added which addresses the root element name, namely that it should not be ignored. Instead, models should properly set the root element with an Could you remove the root element part of this in PR? |
c71fcd8 to
611729d
Compare
The test there is a nice addition but that doesn't address the root issue which was that the model is wrong. I'm assuming though that what you're saying is that the model(s) will be updated appropriately to include missing I've removed the test for ignoring the name, only the flattened list of structures remains. |
Yes |
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.