Updates to AcrPullVerifyStep

Author: MicroFish91

src/commands/registryCredentials/RegistryCredentialsAddConfigurationListStep.ts

@@ -11,8 +11,9 @@ import { AcrEnableAdminUserConfirmStep } from "./dockerLogin/AcrEnableAdminUserC
 import { AcrEnableAdminUserStep } from "./dockerLogin/AcrEnableAdminUserStep";
 import { DockerLoginRegistryCredentialsAddConfigurationStep } from "./dockerLogin/DockerLoginRegistryCredentialsAddConfigurationStep";
 import { AcrPullEnableStep } from "./identity/AcrPullEnableStep";
+import { AcrPullVerifyStep } from "./identity/AcrPullVerifyStep";
 import { ManagedEnvironmentIdentityEnableStep } from "./identity/ManagedEnvironmentIdentityEnableStep";
-import { ManagedIdentityRegistryCredentialAddConfigurationStep } from "./identity/ManagedIdentityRegistryCredentialAddConfigurationStep";
+import { ManagedIdentityRegistryCredentialsAddConfigurationStep } from "./identity/ManagedIdentityRegistryCredentialsAddConfigurationStep";
 import { RegistryCredentialsAndSecretsConfigurationStep } from "./RegistryCredentialsAndSecretsConfigurationStep";
 import { type RegistryCredentialsContext } from "./RegistryCredentialsContext";
 
@@ -62,8 +63,9 @@ export class RegistryCredentialsAddConfigurationListStep extends AzureWizardProm
             case RegistryCredentialType.SystemAssigned:
                 executeSteps.push(
                     new ManagedEnvironmentIdentityEnableStep(),
+                    new AcrPullVerifyStep(),
                     new AcrPullEnableStep(),
-                    new ManagedIdentityRegistryCredentialAddConfigurationStep(registryDomain),
+                    new ManagedIdentityRegistryCredentialsAddConfigurationStep(registryDomain),
                 );
                 break;
             case RegistryCredentialType.DockerLogin:

src/commands/registryCredentials/identity/AcrPullVerifyStep.ts

@@ -42,25 +42,25 @@ export class AcrPullVerifyStep extends AzureWizardExecuteStep<ManagedIdentityReg
             return {
                 item: new GenericTreeItem(undefined, {
                     contextValue: createUniversallyUniqueContextValue(['containerRegistryAcrPullVerifyStepSuccessItem', activitySuccessContext]),
-                    label: localize('verifyAcrPull', 'Verify "{0}" access for container environment identity', 'acrPull'),
+                    label: localize('verifyAcrPull', 'Verify "{0}" access on container registry "{1}"', 'acrPull', context.registry?.name),
                     iconPath: activitySuccessIcon
                 }),
-                message: localize('verifyAcrPullSuccess', 'Successfully verified "{0}" access for container environment identity.', 'acrPull'),
+                message: localize('verifyAcrPullSuccess', 'Successfully verified "{0}" access on container registry "{1}".', 'acrPull', context.registry?.name),
             };
         } else {
             // 'AcrPullEnableStep' will cover showing this output
             return {};
         }
     }
 
-    public createFailOutput(): ExecuteActivityOutput {
+    public createFailOutput(context: ManagedIdentityRegistryCredentialsContext): ExecuteActivityOutput {
         return {
             item: new GenericParentTreeItem(undefined, {
                 contextValue: createUniversallyUniqueContextValue(['containerRegistryAcrPullVerifyStepFailItem', activityFailContext]),
-                label: localize('verifyAcrPull', 'Verify "{0}" access for container environment identity"', 'acrPull'),
+                label: localize('verifyAcrPull', 'Verify "{0}" access on container registry "{1}"', 'acrPull', context.registry?.name),
                 iconPath: activityFailIcon
             }),
-            message: localize('verifyAcrPullFail', 'Failed to verify "{0}" access for container environment identity.', 'acrPull'),
+            message: localize('verifyAcrPullFail', 'Failed to verify "{0}" access on container registry "{1}".', 'acrPull', context.registry?.name),
         };
     }
 }

…egistryCredentialAddConfigurationStep.ts …gistryCredentialsAddConfigurationStep.tssrc/commands/registryCredentials/identity/ManagedIdentityRegistryCredentialAddConfigurationStep.ts renamed to src/commands/registryCredentials/identity/ManagedIdentityRegistryCredentialsAddConfigurationStep.ts src/commands/registryCredentials/identity/ManagedIdentityRegistryCredentialAddConfigurationStep.ts renamed to src/commands/registryCredentials/identity/ManagedIdentityRegistryCredentialsAddConfigurationStep.ts

@@ -8,7 +8,7 @@ import { acrDomain, type SupportedRegistries } from "../../../constants";
 import { localize } from "../../../utils/localize";
 import { type ManagedIdentityRegistryCredentialsContext } from "./ManagedIdentityRegistryCredentialsContext";
 
-export class ManagedIdentityRegistryCredentialAddConfigurationStep extends AzureWizardExecuteStep<ManagedIdentityRegistryCredentialsContext> {
+export class ManagedIdentityRegistryCredentialsAddConfigurationStep extends AzureWizardExecuteStep<ManagedIdentityRegistryCredentialsContext> {
     public priority: number = 470;
 
     constructor(private readonly supportedRegistryDomain: SupportedRegistries | undefined) {