fix: suppress wait messages and progress bars in quiet mode

[Fnuworsu]

Description

Fixes #37.

When running azldev with the -q (or --quiet) flag, the user expects minimal output. However, the spinner ("Waiting for mock...") and progress bars would still write to stderr because their rendering logic in event.go did not respect the quiet configuration.

This PR flows the parsed quiet flag down from app.go to the appEventListener and subsequently to each event, explicitly ignoring calls to SetLongRunning and SetProgress directly writing to os.Stderr if quiet mode is enabled.

Testing

Unit Tests
Added TestEvent_QuietModeSkipsLongRunningAndProgressRendering to verify os.Stderr output suppression.

By capturing os.Stderr and manually invoking .SetLongRunning() and .SetProgress(), the unit test guarantees that these indicators emit zero visual artifact string data when initialized with quiet = true. Tests strictly validate empty output buffers, null spinner references, and uninitialized component objects.

Signed-off-by: Fnuworsu nuworsufelix49@gmail.com

[Copilot]

Pull request overview

Propagates the -q/--quiet flag into the azldev event/rendering layer so long-running spinners and progress bars don’t emit output during quiet runs.

Changes:

• Thread quiet from App → appEventListener → event, and short-circuit SetLongRunning/SetProgress when quiet is enabled
• Update event listener initialization call sites to pass the quiet setting
• Add a new testproj/ fixture directory (TOML config + local .gitignore)

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
testproj/azldev.toml	Adds a sample project config (appears duplicative of existing scenario testdata).
testproj/.gitignore	Ignores build/output directories for the new testproj/ fixture.
internal/app/azldev/eventlistener.go	Adds quiet to the event listener and propagates it into created events.
internal/app/azldev/event.go	Skips spinner/progress rendering when quiet is enabled.
internal/app/azldev/core/testutils/testenv.go	Updates test env setup for the new NewEventListener(..., quiet) signature.
internal/app/azldev/app.go	Passes a.quiet into event listener setup (including after reinitializing logging).

[Fnuworsu]

@microsoft-github-policy-service agree

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

[reubeno]

@Fnuworsu Thanks for working on this! I've queued the PR checks to run and will make sure someone reviews the code changes.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

[Fnuworsu]

@Fnuworsu Thanks for working on this! I've queued the PR checks to run and will make sure someone reviews the code changes.

@reubeno Can you queue the PR checks again? Thank you!

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

[Fnuworsu]

@reubeno the code format workflow failed the last time. I went through it and made necessary changes. Can you restart the PR checks?

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

internal/app/azldev/eventlistener.go:64

• This test sets event.quiet directly, so it doesn’t validate the PR’s main behavior change: plumbing the App’s -q flag into appEventListener.StartEvent() and into each created event. Consider adding a test that constructs a listener with quiet=true, calls StartEvent(), and asserts no stderr control output and that SetLongRunning/SetProgress are effectively suppressed through that path.

	}
}

// Event implements the [opctx.EventListener] interface.
//

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.