Fix: use scopeForTemplate instead of deployment for active check

The deployment object returned by generateDeploymentObject embeds a
Scope that can be nil in test environments (e.g. mockedScope returns
an empty SubscriptionDeployment). Using scopeForTemplate resolves
the scope from the provider's configuration, avoiding nil panics
in existing tests.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: spboyer

cli/azd/pkg/infra/provisioning/bicep/bicep_provider.go

@@ -810,9 +810,13 @@ func (p *BicepProvider) Deploy(ctx context.Context) (*provisioning.DeployResult,
 		p.console.StopSpinner(ctx, "", input.StepDone)
 	}
 
-	// Check for active deployments at the target scope and wait if any are in progress
-	if err := p.waitForActiveDeployments(ctx, deployment); err != nil {
-		return nil, err
+	// Check for active deployments at the target scope and wait if any are in progress.
+	// Use scopeForTemplate to get the raw scope — deployment.Scope may have a nil
+	// inner scope in test mocks.
+	if activeScope, err := p.scopeForTemplate(planned.Template); err == nil {
+		if err := p.waitForActiveDeployments(ctx, activeScope); err != nil {
+			return nil, err
+		}
 	}
 
 	progressCtx, cancelProgress := context.WithCancel(ctx)