Add ValidationLevel parameter to WhatIf/Test (#27352)

tsmallig33 · notyashhh · web-flow · commit ccdd5530ae28 · 2025-03-25T00:57:35.000+11:00
Co-authored-by: Yash &lt;55773468+notyashhh@users.noreply.github.com&gt;
diff --git a/src/Resources/ResourceManager/Implementation/CmdletBase/DeploymentWhatIfCmdlet.cs b/src/Resources/ResourceManager/Implementation/CmdletBase/DeploymentWhatIfCmdlet.cs
@@ -20,6 +20,10 @@ namespace Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.Cmdlet
 
     public abstract class DeploymentWhatIfCmdlet: DeploymentCmdletBase, IDynamicParameters
     {
+        [Parameter(Mandatory = false, HelpMessage = "Sets the validation level for validate/what-if. ValidationLevel can be Template(Skips provider validation), Provider(Performs full validation), " +
+                                                   "or ProviderNoRbac(Performs full validation using RBAC read checks instead of RBAC write checks for provider validation).")]
+        public string ValidationLevel { get; set; }
+
         /// <summary>
         /// It's important not to call this function more than once during an invocation, as it can call the Bicep CLI.
         /// This is slow, and can also cause diagnostics to be emitted multiple times.
diff --git a/src/Resources/ResourceManager/Implementation/CmdletBase/TestDeploymentCmdletBase.cs b/src/Resources/ResourceManager/Implementation/CmdletBase/TestDeploymentCmdletBase.cs
@@ -17,6 +17,10 @@ public abstract class TestDeploymentCmdletBase : DeploymentCmdletBase, IDynamicP
         [Parameter(Mandatory = false, HelpMessage = "The query string (for example, a SAS token) to be used with the TemplateUri parameter. Would be used in case of linked templates")]
         public string QueryString { get; set; }
 
+        [Parameter(Mandatory = false, HelpMessage = "Sets the validation level for validate/what-if. ValidationLevel can be Template(Skips provider validation), Provider(Performs full validation), " +
+                                                   "or ProviderNoRbac(Performs full validation using RBAC read checks instead of RBAC write checks for provider validation).")]
+        public string ValidationLevel { get; set; }
+
         public override object GetDynamicParameters()
         {
             if (!string.IsNullOrEmpty(QueryString))
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/GetAzureManagementGroupDeploymentWhatIfResultCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/GetAzureManagementGroupDeploymentWhatIfResultCmdlet.cs
@@ -64,7 +64,8 @@ public class GetAzureManagementGroupDeploymentWhatIfResultCmdlet : DeploymentWha
             templateParametersUri: this.TemplateParameterUri,
             templateParametersObject: GetTemplateParameterObject(),
             resultFormat: this.ResultFormat,
-            excludeChangeTypes: this.ExcludeChangeType);
+            excludeChangeTypes: this.ExcludeChangeType,
+            validationLevel: this.ValidationLevel);
     }
 }
 
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/GetAzureSubscriptionDeploymentWhatIfResultCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/GetAzureSubscriptionDeploymentWhatIfResultCmdlet.cs
@@ -61,6 +61,7 @@ public class GetAzureSubscriptionDeploymentWhatIfResultCmdlet : DeploymentWhatIf
             templateParametersUri: this.TemplateParameterUri,
             templateParametersObject: GetTemplateParameterObject(),
             resultFormat: this.ResultFormat,
-            excludeChangeTypes: this.ExcludeChangeType);
+            excludeChangeTypes: this.ExcludeChangeType,
+            validationLevel: this.ValidationLevel);
     }
 }
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/GetAzureTenantDeploymentWhatIfResultCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/GetAzureTenantDeploymentWhatIfResultCmdlet.cs
@@ -59,7 +59,8 @@ public class GetAzureTenantDeploymentWhatIfResultCmdlet : DeploymentWhatIfCmdlet
             templateParametersUri: this.TemplateParameterUri,
             templateParametersObject: GetTemplateParameterObject(),
             resultFormat: this.ResultFormat,
-            excludeChangeTypes: this.ExcludeChangeType);
+            excludeChangeTypes: this.ExcludeChangeType,
+            validationLevel: this.ValidationLevel);
     }
 }
 
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/NewAzureManagementGroupDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/NewAzureManagementGroupDeploymentCmdlet.cs
@@ -105,7 +105,8 @@ public class NewAzureManagementGroupDeploymentCmdlet : DeploymentCreateCmdlet
             templateParametersUri: this.TemplateParameterUri,
             templateParametersObject: GetTemplateParameterObject(),
             resultFormat: this.WhatIfResultFormat,
-            excludeChangeTypes: this.WhatIfExcludeChangeType);
+            excludeChangeTypes: this.WhatIfExcludeChangeType,
+            validationLevel: this.ValidationLevel);
 
         protected override bool ShouldSkipConfirmationIfNoChange() => this.ProceedIfNoChange;
     }
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/NewAzureSubscriptionDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/NewAzureSubscriptionDeploymentCmdlet.cs
@@ -100,7 +100,8 @@ public class NewAzureSubscriptionDeploymentCmdlet : DeploymentCreateCmdlet
             templateParametersUri: this.TemplateParameterUri,
             templateParametersObject: GetTemplateParameterObject(),
             resultFormat: this.WhatIfResultFormat,
-            excludeChangeTypes: this.WhatIfExcludeChangeType);
+            excludeChangeTypes: this.WhatIfExcludeChangeType,
+            validationLevel: this.ValidationLevel);
 
         protected override bool ShouldSkipConfirmationIfNoChange() => this.ProceedIfNoChange;
     }
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/NewAzureTenantDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/NewAzureTenantDeploymentCmdlet.cs
@@ -99,7 +99,8 @@ public class NewAzureTenantDeploymentCmdlet: DeploymentCreateCmdlet
             templateParametersUri : this.TemplateParameterUri,
             templateParametersObject : GetTemplateParameterObject(),
             resultFormat : this.WhatIfResultFormat,
-            excludeChangeTypes: this.WhatIfExcludeChangeType
+            excludeChangeTypes: this.WhatIfExcludeChangeType,
+            validationLevel: this.ValidationLevel
         );
 
         protected override bool ShouldSkipConfirmationIfNoChange() => this.ProceedIfNoChange;
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/TestAzureManagementGroupDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/TestAzureManagementGroupDeploymentCmdlet.cs
@@ -55,7 +55,8 @@ protected override void OnProcessRecord()
                 TemplateFile = this.TemplateUri ?? this.TryResolvePath(this.TemplateFile),
                 TemplateObject = this.TemplateObject,
                 TemplateParameterObject = this.GetTemplateParameterObject(),
-                ParameterUri = this.TemplateParameterUri
+                ParameterUri = this.TemplateParameterUri,
+                ValidationLevel = this.ValidationLevel
             };
 
             var validationInfo = NewResourceManagerSdkClient.ValidateDeployment(parameters);
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/TestAzureSubscriptionDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/TestAzureSubscriptionDeploymentCmdlet.cs
@@ -51,7 +51,8 @@ protected override void OnProcessRecord()
                 TemplateObject = TemplateObject,
                 QueryString = QueryString,
                 TemplateParameterObject = GetTemplateParameterObject(),
-                ParameterUri = TemplateParameterUri
+                ParameterUri = TemplateParameterUri,
+                ValidationLevel = ValidationLevel
             };
 
             var validationInfo = NewResourceManagerSdkClient.ValidateDeployment(parameters);
diff --git a/src/Resources/ResourceManager/Implementation/Deployments/TestAzureTenantDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/Deployments/TestAzureTenantDeploymentCmdlet.cs
@@ -51,7 +51,8 @@ protected override void OnProcessRecord()
                 TemplateObject = this.TemplateObject,
                 QueryString = QueryString,
                 TemplateParameterObject = this.GetTemplateParameterObject(),
-                ParameterUri = this.TemplateParameterUri
+                ParameterUri = this.TemplateParameterUri,
+                ValidationLevel = this.ValidationLevel
             };
 
             var validationInfo = NewResourceManagerSdkClient.ValidateDeployment(parameters);
diff --git a/src/Resources/ResourceManager/Implementation/ResourceGroupDeployments/GetAzureResourceGroupDeploymentWhatIfResultCmdlet.cs b/src/Resources/ResourceManager/Implementation/ResourceGroupDeployments/GetAzureResourceGroupDeploymentWhatIfResultCmdlet.cs
@@ -61,6 +61,7 @@ public class GetAzureResourceGroupDeploymentWhatIfResultCmdlet : DeploymentWhatI
             templateParametersUri: this.TemplateParameterUri,
             templateParametersObject: GetTemplateParameterObject(),
             resultFormat: this.ResultFormat,
-            excludeChangeTypes: this.ExcludeChangeType);
+            excludeChangeTypes: this.ExcludeChangeType,
+            validationLevel: this.ValidationLevel);
     }
 }
diff --git a/src/Resources/ResourceManager/Implementation/ResourceGroupDeployments/NewAzureResourceGroupDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/ResourceGroupDeployments/NewAzureResourceGroupDeploymentCmdlet.cs
@@ -121,7 +121,8 @@ public class NewAzureResourceGroupDeploymentCmdlet : DeploymentCreateCmdlet
             templateParametersUri: this.TemplateParameterUri,
             templateParametersObject: this.GetTemplateParameterObject(),
             resultFormat: this.WhatIfResultFormat,
-            excludeChangeTypes: this.WhatIfExcludeChangeType);
+            excludeChangeTypes: this.WhatIfExcludeChangeType,
+            validationLevel: this.ValidationLevel);
 
         protected override void OnProcessRecord()
         {
diff --git a/src/Resources/ResourceManager/Implementation/ResourceGroupDeployments/TestAzureResourceGroupDeploymentCmdlet.cs b/src/Resources/ResourceManager/Implementation/ResourceGroupDeployments/TestAzureResourceGroupDeploymentCmdlet.cs
@@ -76,7 +76,8 @@ protected override void OnProcessRecord()
                         Type = RollbackToLastDeployment ? OnErrorDeploymentType.LastSuccessful : OnErrorDeploymentType.SpecificDeployment,
                         DeploymentName = RollbackToLastDeployment ? null : RollBackDeploymentName
                     }
-                    : null
+                    : null,
+                ValidationLevel = ValidationLevel
             };
 
             var validationInfo = NewResourceManagerSdkClient.ValidateDeployment(parameters);
diff --git a/src/Resources/ResourceManager/SdkClient/NewResourceManagerSdkClient.cs b/src/Resources/ResourceManager/SdkClient/NewResourceManagerSdkClient.cs
@@ -481,6 +481,11 @@ private Deployment CreateBasicDeployment(PSDeploymentCmdletParameters parameters
             deployment.Tags = parameters?.Tags == null ? null : new Dictionary<string, string>(parameters.Tags);
             deployment.Properties.OnErrorDeployment = parameters.OnErrorDeployment;
 
+            if (!string.IsNullOrEmpty(parameters.ValidationLevel))
+            {
+                deployment.Properties.ValidationLevel = parameters.ValidationLevel;
+            }
+
             return deployment;
         }
 
diff --git a/src/Resources/ResourceManager/SdkModels/Deployments/PSDeploymentCmdletParameters.cs b/src/Resources/ResourceManager/SdkModels/Deployments/PSDeploymentCmdletParameters.cs
@@ -43,5 +43,7 @@ public class PSDeploymentCmdletParameters
         public OnErrorDeployment OnErrorDeployment { get; set; }
 
         public IDictionary<string, IList<string>> AuxTenantHeaders { get; set; }
+
+        public string ValidationLevel { get; set; }
     }
 }
diff --git a/src/Resources/ResourceManager/SdkModels/Deployments/PSDeploymentWhatIfCmdletParameters.cs b/src/Resources/ResourceManager/SdkModels/Deployments/PSDeploymentWhatIfCmdletParameters.cs
@@ -34,7 +34,8 @@ public PSDeploymentWhatIfCmdletParameters(
             Hashtable templateObject = null,
             Hashtable templateParametersObject = null,
             WhatIfResultFormat resultFormat = WhatIfResultFormat.FullResourcePayloads,
-            string[] excludeChangeTypes = null)
+            string[] excludeChangeTypes = null,
+            string validationLevel = null)
         {
             this.DeploymentName = deploymentName ?? this.GenerateDeployName();
             this.ScopeType = scopeType;
@@ -53,6 +54,7 @@ public PSDeploymentWhatIfCmdletParameters(
                 .Select(changeType => changeType.ToLowerInvariant())
                 .Distinct()
                 .Select(changeType => (ChangeType)Enum.Parse(typeof(ChangeType), changeType, true));
+            this.ValidationLevel = validationLevel;
         }
 
         public string DeploymentName
@@ -87,6 +89,8 @@ public string DeploymentName
 
         public IEnumerable<ChangeType> ExcludeChangeTypes { get; }
 
+        public string ValidationLevel { get; set; }
+
         public DeploymentWhatIf ToDeploymentWhatIf()
         {
             var properties = new DeploymentWhatIfProperties
@@ -134,6 +138,11 @@ public DeploymentWhatIf ToDeploymentWhatIf()
                     : null;
             }
 
+            if (!string.IsNullOrEmpty(this.ValidationLevel))
+            {
+                properties.ValidationLevel = this.ValidationLevel;
+            }
+
             return new DeploymentWhatIf(properties, this.Location);
         }
 
diff --git a/src/Resources/Resources.Test/ResourceGroupDeployments/TestAzureResourceGroupDeploymentCommandTests.cs b/src/Resources/Resources.Test/ResourceGroupDeployments/TestAzureResourceGroupDeploymentCommandTests.cs
@@ -31,6 +31,7 @@
 using Microsoft.Azure.Commands.ResourceManager.Cmdlets.Formatters;
 using Newtonsoft.Json.Linq;
 using Microsoft.Azure.Commands.ResourceManager.Cmdlets.Extensions;
+using FluentAssertions;
 
 namespace Microsoft.Azure.Commands.Resources.Test.Resources
 {
@@ -147,5 +148,36 @@ public void ValidatesPSResourceGroupDeploymentWithUserTemplateWithDiagnostics()
             commandRuntimeMock.Verify(f => f.WriteObject(expectedObject, true), Times.Once());
             commandRuntimeMock.Verify(f => f.WriteObject(It.IsAny<List<PSResourceManagerError>>()), Times.Never());
         }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void ValidatesPSResourceGroupDeploymentWithUserTemplateProviderNoRbac()
+        {
+            PSDeploymentCmdletParameters expectedParameters = new PSDeploymentCmdletParameters()
+            {
+                TemplateFile = templateFile,
+                ValidationLevel = ValidationLevel.ProviderNoRbac
+            };
+            PSDeploymentCmdletParameters actualParameters = new PSDeploymentCmdletParameters();
+
+            TemplateValidationInfo expectedResults = new(new DeploymentValidateResult());
+
+            resourcesClientMock.Setup(f => f.ValidateDeployment(
+                It.IsAny<PSDeploymentCmdletParameters>()))
+                .Returns(expectedResults)
+                .Callback((PSDeploymentCmdletParameters p) => { actualParameters = p; });
+
+            cmdlet.ResourceGroupName = resourceGroupName;
+            cmdlet.TemplateFile = expectedParameters.TemplateFile;
+            cmdlet.ValidationLevel = ValidationLevel.ProviderNoRbac;
+
+            cmdlet.ExecuteCmdlet();
+
+            actualParameters.TemplateFile.Should().Equals(expectedParameters.TemplateFile);
+
+            actualParameters.ValidationLevel.Should().Be(ValidationLevel.ProviderNoRbac);
+
+            Assert.NotNull(actualParameters.TemplateParameterObject);
+        }
     }
 }
diff --git a/src/Resources/Resources.Test/ScenarioTests/DeploymentWhatIfTests.cs b/src/Resources/Resources.Test/ScenarioTests/DeploymentWhatIfTests.cs
@@ -152,6 +152,34 @@ public void TenantLevelWhatIf_SetExcludeChangeType_HidesResourceChanges()
         {
             TestRunner.RunTestScript("Test-WhatIfExcludeChangeTypesAtTenantScope");
         }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void TenantLevelWhatIf_ProviderNoRbacAtTenantScope()
+        {
+            TestRunner.RunTestScript("Test-WhatIfProviderNoRbacAtTenantScope");
+        }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void MGLevelWhatIf_ProviderNoRbacAtManagementGroupScope()
+        {
+            TestRunner.RunTestScript("Test-WhatIfProviderNoRbacAtManagementGroupScope");
+        }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void SubLevelWhatIf_ProviderNoRbacAtSubscriptionScope()
+        {
+            TestRunner.RunTestScript("Test-WhatIfProviderNoRbacAtSubscriptionScope");
+        }
+
+        [Fact]
+        [Trait(Category.AcceptanceType, Category.CheckIn)]
+        public void RGLevelWhatIf_ProviderNoRbacAtResourceGroupScope()
+        {
+            TestRunner.RunTestScript("Test-WhatIfProviderNoRbacAtResourceGroupScope");
+        }
     }
 }
 
diff --git a/src/Resources/Resources.Test/ScenarioTests/DeploymentWhatIfTests.ps1 b/src/Resources/Resources.Test/ScenarioTests/DeploymentWhatIfTests.ps1
diff --git a/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/MGLevelWhatIf_ProviderNoRbacAtManagementGroupScope.json b/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/MGLevelWhatIf_ProviderNoRbacAtManagementGroupScope.json
diff --git a/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/RGLevelWhatIf_ProviderNoRbacAtResourceGroupScope.json b/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/RGLevelWhatIf_ProviderNoRbacAtResourceGroupScope.json
diff --git a/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/SubLevelWhatIf_ProviderNoRbacAtSubscriptionScope.json b/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/SubLevelWhatIf_ProviderNoRbacAtSubscriptionScope.json
diff --git a/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/TenantLevelWhatIf_ProviderNoRbacAtTenantScope.json b/src/Resources/Resources.Test/SessionRecords/Microsoft.Azure.Commands.Resources.Test.ScenarioTests.DeploymentWhatIfTests/TenantLevelWhatIf_ProviderNoRbacAtTenantScope.json
diff --git a/src/Resources/Resources/ChangeLog.md b/src/Resources/Resources/ChangeLog.md

Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,8 @@ public class GetAzureManagementGroupDeploymentWhatIfResultCmdlet : DeploymentWha`
`64`	`64`	`templateParametersUri: this.TemplateParameterUri,`
`65`	`65`	`templateParametersObject: GetTemplateParameterObject(),`
`66`	`66`	`resultFormat: this.ResultFormat,`
`67`		`- excludeChangeTypes: this.ExcludeChangeType);`
	`67`	`+ excludeChangeTypes: this.ExcludeChangeType,`
	`68`	`+ validationLevel: this.ValidationLevel);`
`68`	`69`	`}`
`69`	`70`	`}`
`70`	`71`
Original file line number	Diff line number	Diff line change
`@@ -61,6 +61,7 @@ public class GetAzureSubscriptionDeploymentWhatIfResultCmdlet : DeploymentWhatIf`
`61`	`61`	`templateParametersUri: this.TemplateParameterUri,`
`62`	`62`	`templateParametersObject: GetTemplateParameterObject(),`
`63`	`63`	`resultFormat: this.ResultFormat,`
`64`		`- excludeChangeTypes: this.ExcludeChangeType);`
	`64`	`+ excludeChangeTypes: this.ExcludeChangeType,`
	`65`	`+ validationLevel: this.ValidationLevel);`
`65`	`66`	`}`
`66`	`67`	`}`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,8 @@ public class GetAzureTenantDeploymentWhatIfResultCmdlet : DeploymentWhatIfCmdlet`
`59`	`59`	`templateParametersUri: this.TemplateParameterUri,`
`60`	`60`	`templateParametersObject: GetTemplateParameterObject(),`
`61`	`61`	`resultFormat: this.ResultFormat,`
`62`		`- excludeChangeTypes: this.ExcludeChangeType);`
	`62`	`+ excludeChangeTypes: this.ExcludeChangeType,`
	`63`	`+ validationLevel: this.ValidationLevel);`
`63`	`64`	`}`
`64`	`65`	`}`
`65`	`66`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,8 @@ public class NewAzureManagementGroupDeploymentCmdlet : DeploymentCreateCmdlet`
`105`	`105`	`templateParametersUri: this.TemplateParameterUri,`
`106`	`106`	`templateParametersObject: GetTemplateParameterObject(),`
`107`	`107`	`resultFormat: this.WhatIfResultFormat,`
`108`		`- excludeChangeTypes: this.WhatIfExcludeChangeType);`
	`108`	`+ excludeChangeTypes: this.WhatIfExcludeChangeType,`
	`109`	`+ validationLevel: this.ValidationLevel);`
`109`	`110`
`110`	`111`	`protected override bool ShouldSkipConfirmationIfNoChange() => this.ProceedIfNoChange;`
`111`	`112`	`}`
Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,8 @@ public class NewAzureSubscriptionDeploymentCmdlet : DeploymentCreateCmdlet`
`100`	`100`	`templateParametersUri: this.TemplateParameterUri,`
`101`	`101`	`templateParametersObject: GetTemplateParameterObject(),`
`102`	`102`	`resultFormat: this.WhatIfResultFormat,`
`103`		`- excludeChangeTypes: this.WhatIfExcludeChangeType);`
	`103`	`+ excludeChangeTypes: this.WhatIfExcludeChangeType,`
	`104`	`+ validationLevel: this.ValidationLevel);`
`104`	`105`
`105`	`106`	`protected override bool ShouldSkipConfirmationIfNoChange() => this.ProceedIfNoChange;`
`106`	`107`	`}`