aa-log: add missing unit test.

cmd/aa-log/main.go

@@ -8,6 +8,7 @@ import (
 	"bufio"
 	"fmt"
 	"os"
+	"path/filepath"
 	"regexp"
 	"strings"
 )
@@ -156,24 +157,32 @@ func (aaLogs AppArmorLogs) String() string {
 	return res
 }
 
-func main() {
+func aaLog(args []string, path string) error {
 	profile := ""
-	if len(os.Args) >= 2 {
+	if len(args) >= 2 {
-		profile = os.Args[1]
+		profile = args[1]
 	}
 
-	file, err := os.Open(LogFile)
+	file, err := os.Open(filepath.Clean(path))
 	if err != nil {
-		fmt.Println(err)
+		return err
-		os.Exit(1)
 	}
+	/* #nosec G307 */
 	defer func() {
 		if err := file.Close(); err != nil {
-			fmt.Println("Error closing file:", err)
+			fmt.Println(err)
-			os.Exit(1)
 		}
 	}()
 
 	aaLogs := NewApparmorLogs(file, profile)
 	fmt.Print(aaLogs.String())
+	return err
+}
+
+func main() {
+	err := aaLog(os.Args, LogFile)
+	if err != nil {
+		fmt.Println(err)
+		os.Exit(1)
+	}
 }

cmd/aa-log/main_test.go

@@ -127,3 +127,32 @@ func TestAppArmorLogs_String(t *testing.T) {
 		})
 	}
 }
+
+func Test_app(t *testing.T) {
+	tests := []struct {
+		name    string
+		args    []string
+		path    string
+		wantErr bool
+	}{
+		{
+			name:    "OK",
+			args:    []string{"aa-log", ""},
+			path:    "../../tests/audit.log",
+			wantErr: false,
+		},
+		{
+			name:    "No logfile",
+			args:    []string{"aa-log", ""},
+			path:    "../../tests/log",
+			wantErr: true,
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if err := aaLog(tt.args, tt.path); (err != nil) != tt.wantErr {
+				t.Errorf("aaLog() error = %v, wantErr %v", err, tt.wantErr)
+			}
+		})
+	}
+}