Add unit test coverage on the dashboard component

.gitignore

@@ -57,6 +57,12 @@ opt/manifests/
 
 cover.out
 
+# Coverage HTML reports
+*_coverage.html
+*_coverage.out
+coverage.html
+coverage.out
+overall_coverage.out
 config/manager/kustomization.yaml
 
 # Ignore any local.mk files that would be consumed by the Makefile
@@ -67,3 +73,6 @@ local.mk
 
 # Ignore catalog related files
 catalog/
+
+# Ignore backup files
+*.backup

Makefile

@@ -445,7 +445,18 @@ unit-test: envtest ginkgo # directly use ginkgo since the framework is not compa
         		--coverprofile=cover.out \
         		--succinct \
         		$(TEST_SRC)
-CLEANFILES += cover.out
+CLEANFILES += cover.out coverage.html
+
+.PHONY: test-coverage
+test-coverage: ## Generate HTML coverage report
+	$(MAKE) unit-test
+	@if [ ! -f cover.out ] || [ ! -s cover.out ]; then \
+		echo "Warning: cover.out is missing or empty. Skipping coverage report generation."; \
+		echo "Make sure unit tests are configured to generate coverage data."; \
+		exit 0; \
+	fi
+	go tool cover -html=cover.out -o coverage.html
+	@echo "Coverage report generated: coverage.html"
 
 $(PROMETHEUS_TEST_DIR)/%.rules.yaml: $(PROMETHEUS_TEST_DIR)/%.unit-tests.yaml $(PROMETHEUS_CONFIG_YAML) $(YQ)
 	$(YQ) eval ".data.\"$(@F:.rules.yaml=.rules)\"" $(PROMETHEUS_CONFIG_YAML) > $@

internal/controller/components/dashboard/dashboard.go

@@ -22,6 +22,11 @@ import (
 	"github.com/opendatahub-io/opendatahub-operator/v2/pkg/metadata/annotations"
 )
 
+const (
+	// AnacondaSecretName is the name of the anaconda access secret.
+	AnacondaSecretName = "anaconda-ce-access" //nolint:gosec // This is a Kubernetes secret name, not a credential
+)
+
 type componentHandler struct{}
 
 func init() { //nolint:gochecknoinits
@@ -33,14 +38,17 @@ func (s *componentHandler) GetName() string {
 }
 
 func (s *componentHandler) Init(platform common.Platform) error {
-	mi := defaultManifestInfo(platform)
+	mi, err := DefaultManifestInfo(platform)
+	if err != nil {
+		return err
+	}
 
-	if err := odhdeploy.ApplyParams(mi.String(), "params.env", imagesMap); err != nil {
+	if err := odhdeploy.ApplyParams(mi.String(), "params.env", ImagesMap); err != nil {
 		return fmt.Errorf("failed to update images on path %s: %w", mi, err)
 	}
 
-	extra := bffManifestsPath()
-	if err := odhdeploy.ApplyParams(extra.String(), "params.env", imagesMap); err != nil {
+	extra := BffManifestsPath()
+	if err := odhdeploy.ApplyParams(extra.String(), "params.env", ImagesMap); err != nil {
 		return fmt.Errorf("failed to update modular-architecture images on path %s: %w", extra, err)
 	}
 
@@ -72,6 +80,10 @@ func (s *componentHandler) IsEnabled(dsc *dscv2.DataScienceCluster) bool {
 func (s *componentHandler) UpdateDSCStatus(ctx context.Context, rr *types.ReconciliationRequest) (metav1.ConditionStatus, error) {
 	cs := metav1.ConditionUnknown
 
+	if rr.Client == nil {
+		return cs, errors.New("client is nil")
+	}
+
 	c := componentApi.Dashboard{}
 	c.Name = componentApi.DashboardInstanceName
 
@@ -89,8 +101,6 @@ func (s *componentHandler) UpdateDSCStatus(ctx context.Context, rr *types.Reconc
 	dsc.Status.Components.Dashboard.ManagementState = ms
 	dsc.Status.Components.Dashboard.DashboardCommonStatus = nil
 
-	rr.Conditions.MarkFalse(ReadyConditionType)
-
 	if s.IsEnabled(dsc) {
 		dsc.Status.Components.Dashboard.DashboardCommonStatus = c.Status.DashboardCommonStatus.DeepCopy()
 

internal/controller/components/dashboard/dashboard_controller.go

@@ -18,6 +18,7 @@ package dashboard
 
 import (
 	"context"
+	"errors"
 	"fmt"
 
 	consolev1 "github.com/openshift/api/console/v1"
@@ -46,7 +47,11 @@ import (
 
 // NewComponentReconciler creates a ComponentReconciler for the Dashboard API.
 func (s *componentHandler) NewComponentReconciler(ctx context.Context, mgr ctrl.Manager) error {
-	componentName := computeComponentName()
+	if mgr == nil {
+		return errors.New("could not create the dashboard controller: manager cannot be nil")
+	}
+
+	componentName := ComputeComponentName()
 
 	_, err := reconciler.ReconcilerFor(mgr, &componentApi.Dashboard{}).
 		// operands - owned
@@ -95,9 +100,10 @@ func (s *componentHandler) NewComponentReconciler(ctx context.Context, mgr ctrl.
 			GenericFunc: func(tge event.TypedGenericEvent[client.Object]) bool { return false },
 			DeleteFunc:  func(tde event.TypedDeleteEvent[client.Object]) bool { return false },
 		}), reconciler.Dynamic(reconciler.CrdExists(gvk.DashboardHardwareProfile))).
-		WithAction(initialize).
+		WithAction(Initialize).
 		WithAction(setKustomizedParams).
-		WithAction(configureDependencies).
+		WithAction(ConfigureDependencies).
+		WithAction(CustomizeResources).
 		WithAction(kustomize.NewAction(
 			// Those are the default labels added by the legacy deploy method
 			// and should be preserved as the original plugin were affecting
@@ -112,15 +118,15 @@ func (s *componentHandler) NewComponentReconciler(ctx context.Context, mgr ctrl.
 		)).
 		WithAction(deploy.NewAction()).
 		WithAction(deployments.NewAction()).
-		WithAction(reconcileHardwareProfiles).
-		WithAction(updateStatus).
+		WithAction(ReconcileHardwareProfiles).
+		WithAction(UpdateStatus).
 		// must be the final action
 		WithAction(gc.NewAction(
 			gc.WithUnremovables(gvk.OdhDashboardConfig),
 		)).
 		// declares the list of additional, controller specific conditions that are
 		// contributing to the controller readiness status
-		WithConditions(conditionTypes...).
+		WithConditions(ConditionTypes...).
 		Build(ctx)
 
 	if err != nil {