fix: Fixed MSI UDT test to only check for the required output if the imported type is also used by a parameter #4498
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
We currently have the case that the elastic-san module failed because one of its child-modules (
volume-Group
) has amanagedIdentities
parameter which is imported into the parentelastic-san
module for the type of thevolume-group
. Now, this adds themanagedIdentityAllType
to thedefintions
section of the parent module.The tests found this declaration and assumed the type is used and because it supported the
systemAssignedIdentity
property, it required thesystemAssignedMIPrincipalId
, which in this case is incorrect. The child needs it, not the parent.I updated the test so that it not only checks if the UDT is declared, but if it's also used by a parameter in the same module.
Tested
Pipeline Reference
Type of Change
version.json
:version.json
.version.json
.