Allow to set verbose flag

bespinian · Jun 9, 2023 · 4b689f6 · 4b689f6
1 parent 6c7787f
commit 4b689f6
Show file tree

Hide file tree

Showing 5 changed files with 49 additions and 6 deletions.
diff --git a/internal/livelint/check_can_access_app.go b/internal/livelint/check_can_access_app.go
@@ -59,7 +59,6 @@ func (n *Livelint) canPortForward(pod apiv1.Pod, port int32, checkFunc func(uint
 	handleConnectionError := func(err error) {
 		connectionSuccessful = false
 	}
-	// nolint:reassign
 	runtime.ErrorHandlers = append(runtime.ErrorHandlers, handleConnectionError)
 
 	// prepare the port forwarding

diff --git a/internal/livelint/checks_model.go b/internal/livelint/checks_model.go
@@ -97,7 +97,6 @@ func (m Model) View() string {
 	return doc.String()
 }
 
-// nolint:ireturn
 func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	switch msg := msg.(type) {
 	case tea.KeyMsg:
@@ -224,7 +223,7 @@ type check struct {
 	outcome      summaryMsg
 }
 
-func initalizeStatus(context string) statusMsg {
+func initializeStatus(context string) statusMsg {
 	return statusMsg{context: context, checks: []check{}}
 }
 

diff --git a/internal/livelint/interaction.go b/internal/livelint/interaction.go
@@ -13,6 +13,7 @@ type UserInteraction interface {
 	DisplayCheckCompletion(completionMsg string, kind SummaryType)
 	DisplayError(errorMsg error)
 	AskYesNo(question string) bool
+	SetVerbose()
 	StartSpinner()
 	StopSpinner()
 }
@@ -37,7 +38,7 @@ func (ui *BubbleteaUI) AskYesNo(question string) bool {
 }
 
 func (ui *BubbleteaUI) DisplayContext(contextMsg string) {
-	ui.statusMsg = initalizeStatus(contextMsg)
+	ui.statusMsg = initializeStatus(contextMsg)
 	ui.Send(ui.statusMsg)
 }
 
@@ -60,6 +61,10 @@ func (ui *BubbleteaUI) DisplayError(errorMsg error) {
 	ui.Send(errMsg{err: errorMsg})
 }
 
+func (ui *BubbleteaUI) SetVerbose() {
+	ui.Send(verboseMsg{verbose: true})
+}
+
 func (ui *BubbleteaUI) StartSpinner() {
 	ui.Send(showSpinnerMsg{showing: true})
 }

diff --git a/internal/livelint/mock_ui_test.go b/internal/livelint/mock_ui_test.go
diff --git a/internal/livelint/run_checks.go b/internal/livelint/run_checks.go
@@ -5,16 +5,19 @@ import (
 )
 
 // RunChecks checks for potential issues with a deployment.
-func (n *Livelint) RunChecks(namespace, deploymentName string, isVerbose bool) error {
+func (n *Livelint) RunChecks(namespace, deploymentName string, verbose bool) error {
 	if namespace == "" {
 		return fmt.Errorf("error running checks: %w", errNamespaceUndefined)
 	}
 	if deploymentName == "" {
 		return errDeploymentNameUndefined
 	}
 
-	// nolint:govet
 	n.ui.DisplayContext(fmt.Sprintf("Checking Deployment %q in Namespace %q", deploymentName, namespace))
+	if verbose {
+		n.ui.SetVerbose()
+	}
+
 	n.ui.DisplayCheckStart("Checking Pods")
 
 	allPods, err := n.getDeploymentPods(namespace, deploymentName)