fix(appconfig): sourcedConfiguration doesn't use retrievalRole (#30733)

### Issue # (if applicable) Closes #30609 ### Reason for this change To refactor the retrievalRole creation logic. ### Description of changes ### Description of how you validated changes Unit tests: 1. configuration with retrievalRole undefined from bucket source should create a new role 2. configuration with retrievalRole defined should NOT create a new role and should use the passed role for the retrievalRoleArn 3. configuration with retrievalRole undefined from CodePipeline source should NOT create a new role Integ test: 1. update the existing integ test to assert a use case where the retrievalRole is provided. ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
aws · Jul 25, 2024 · 9d79c51 · 9d79c51
1 parent 742c52e
commit 9d79c51
Show file tree

Hide file tree

Showing 9 changed files with 1,393 additions and 340 deletions.
diff --git a/...61df08d8dbf7cd37e60bf216fb982a00/index.py → ...cc3d68f525fe08a0fe0f014069ae539c/index.py b/...61df08d8dbf7cd37e60bf216fb982a00/index.py → ...cc3d68f525fe08a0fe0f014069ae539c/index.py
diff --git a/....snapshot/asset.44e9c4d7a5d3fd2d677e1a7e416b2b56f6b0104bd5eff9cac5557b4c65a9dc61/index.js b/....snapshot/asset.44e9c4d7a5d3fd2d677e1a7e416b2b56f6b0104bd5eff9cac5557b4c65a9dc61/index.js
diff --git a/...ws-appconfig/test/integ.configuration.js.snapshot/aws-appconfig-configuration.assets.json b/...ws-appconfig/test/integ.configuration.js.snapshot/aws-appconfig-configuration.assets.json