refactor(cmd): improve error message when resource file is not refere…

…nced

Signed-off-by: Xavier Coulon <[email protected]>

pkg/validation/components.go

@@ -1,7 +1,6 @@
 package validation
 
 import (
-	"fmt"
 	"io/fs"
 	"path/filepath"
 
@@ -31,12 +30,12 @@ func CheckComponents(logger Logger, afs afero.Afero, baseDir string, components
 			// look for a Kustomization file in the directory
 			if kp, found := lookupKustomizationFile(logger, afs, path); found {
 				if err := checkKustomizeResources(logger, afs, kp); err != nil {
-					return fmt.Errorf("invalid resources at %s: %w", path, err)
+					return err
 				}
 				if d.Name() != "base" {
 					logger.Debug("checking Kustomization build ", "path", path)
 					if err := checkBuild(logger, fsys, path); err != nil {
-						return fmt.Errorf("invalid resources at %s: %w", path, err)
+						return err
 					}
 				}
 			}

pkg/validation/components_test.go

@@ -330,7 +330,7 @@ data:
 			err = validation.CheckComponents(logger, afs, "/path/to", "components")
 
 			// then
-			require.Error(t, err, "resource is not referenced: /path/to/components/configmap2.yaml")
+			require.EqualError(t, err, "resource is not referenced in /path/to/components/kustomization.yaml: configmap2.yaml")
 			assert.Empty(t, logger.Errors())
 			assert.Empty(t, logger.Warnings())
 		})

pkg/validation/resources.go

@@ -86,7 +86,7 @@ entries:
 				continue entries
 			}
 		}
-		return fmt.Errorf("resource is not referenced: %s", filepath.Join(path, e.Name()))
+		return fmt.Errorf("resource is not referenced in %s: %s", path, e.Name())
 	}
 	return nil
 }