Skip to content

Conversation

@yenfryherrerafeliz
Copy link
Contributor

Issue #3239

Description of changes:

When a structure member has the locationName xmlName trait then we should use that name instead of the original name.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

When a structure member has the locationName `xmlName trait` then we should use that name instead of the original name.
@yenfryherrerafeliz
Copy link
Contributor Author

We need to find a way to skip the locationName just when it is defined at structure level and keep it when defined at member level.

Add a property to know whether the `locationName` was defined at the structure level or at the member level in the model definition. This is needed to decide when we need to use the locationName at the serialization process.
@yenfryherrerafeliz
Copy link
Contributor Author

vendor/bin/behat --format=progress --tags=integ
...................................................................... 70
...................................................................... 140
...................................................................... 210
...................................................................... 280
...................................................................... 350
...................................................................... 420
...............

127 scenarios (127 passed)
435 steps (435 passed)
29m20.48s (58.03Mb)
vendor/bin/behat --format=progress --suite=smoke --tags='~@noassumerole'
...................................................................... 70
...................................................................... 140
...................................................................... 210
............

111 scenarios (111 passed)
222 steps (222 passed)
0m39.67s (94.67Mb)

@yenfryherrerafeliz yenfryherrerafeliz merged commit bdb8e6e into aws:master Jan 7, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant