Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: Added case handling for
loadFromFile()
functions that introduc…
…e generated variables in ARM JSON files (#1917) ## Description - When using a Bicep function like `loadFileAsBase64()` or `loadFromJson`, the compiled ARM template contains a generated variable with a name like `$fxv#1` or `$fxv#2` - This variable currently fails the Pester test 'Variable names should be camel-cased' - This PR updates the test by - Adding support for the aforementioned deterministic variable name - Updating the test to return the incorrect variables as opposed to just claim that there's at least one incorrect variable ## Pipeline Reference <!-- Insert your Pipeline Status Badge below --> | Pipeline | | -------- | | [](https://github.com/AlexanderSehr/bicep-registry-modules/actions/workflows/avm.res.key-vault.vault.yml) | | [](https://github.com/AlexanderSehr/bicep-registry-modules/actions/workflows/avm.res.storage.storage-account.yml) (Pester passed) | ## Type of Change <!-- Use the check-boxes [x] on the options that are relevant. --> - [x] Update to CI Environment or utlities (Non-module effecting changes) - [ ] Azure Verified Module updates: - [ ] Bugfix containing backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in `version.json`: - [ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description. - [ ] The bug was found by the module author, and no one has opened an issue to report it yet. - [ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in `version.json`. - [ ] Breaking changes and I have bumped the MAJOR version in `version.json`. - [ ] Update to documentation --------- Co-authored-by: Erika Gressi <[email protected]>
- Loading branch information