Azure / Azure/azure-dev

Add integration tests for UpGraphAction.Run()

Open
#7,970 1 comment 0 reactions 0 assignees View on GitHub
area/core-cli engineering item
Dominant language
Go
Stars
569
Forks
364
Avg merge
2d 19h
Merged PRs (30d)
136

Description

## Context

Follow-up from PR #7776 (exegraph) review - wbreza Pass 3, finding M2.

## Problem

The 462-line UpGraphAction.Run() orchestrator in internal/cmd/up_graph.go has no dedicated unit/integration tests. The underlying engine core has 54 tests, but the graph-building and lifecycle orchestration logic in up_graph.go is only exercised via functional/E2E tests.

## Acceptance Criteria

- [ ] Add integration tests for UpGraphAction.Run() using existing mock infrastructure (mockexec, mockinput, etc.)
- [ ] Cover key scenarios: single-layer provision + deploy, multi-layer provision, service dependencies (uses:), error propagation from provision/deploy steps
- [ ] Verify deploy progress tracker integration (callbacks fire correctly)

## Estimate

~2-3hr effort using existing mock patterns from deploy_test.go and provision_test.go.

## Related

- PR #7776 (exegraph engine)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.