feat: add more details to 'have_permission' rule message

omissis · Sep 11, 2022 · 6fe84c1 · 6fe84c1
1 parent 5767729
commit 6fe84c1
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/internal/arch/file/expect/have_permissions.go b/internal/arch/file/expect/have_permissions.go
@@ -68,13 +68,13 @@ func (e havePermissionsExpression) getViolation(filePath string) rule.CoreViolat
 		iNodeType = "directory"
 	}
 
-	format := "%s '%s' does not have permissions matching '%s'"
+	format := "%s '%s' does not have permissions matching '%s', '%s' found"
 
 	if e.options.negated {
-		format = "%s '%s' does have permissions matching '%s'"
+		format = "%s '%s' does have permissions matching '%s', '%s' found"
 	}
 
 	return rule.NewCoreViolation(
-		fmt.Sprintf(format, iNodeType, filepath.Base(filePath), e.permissions),
+		fmt.Sprintf(format, iNodeType, filepath.Base(filePath), e.permissions, info.Mode().String()),
 	)
 }