Merge branch 'UDENG-5696-initial-client-core-cli' into UDENG-5705-Ubu…

…ntu-Insights-Consent-Manager

cmd/exposed-server/daemon/daemon.go

@@ -1 +1,2 @@
+// Package daemon is responsible for running the exposed-server in the background.
 package daemon

cmd/exposed-server/main.go

@@ -1 +1,2 @@
+// Package main for the exposed-server.
 package main

cmd/ingest-server/daemon/daemon.go

@@ -1 +1,2 @@
+// Package daemon is responsible for running the ingest-server in the background.
 package daemon

cmd/ingest-server/main.go

@@ -1 +1,2 @@
+// Package main for the ingest-server.
 package main

cmd/insights/commands/collect.go

@@ -24,7 +24,7 @@ var defaultCollectConfig = collectConfig{
 	extraMetrics: "",
 }
 
-func installCollectCmd(app *App) error {
+func installCollectCmd(app *App) {
 	app.collectConfig = defaultCollectConfig
 
 	collectCmd := &cobra.Command{
@@ -45,11 +45,11 @@ func installCollectCmd(app *App) error {
 
 				fileInfo, err := os.Stat(args[1])
 				if err != nil {
-					return fmt.Errorf("the second argument, SOURCE-METRICS-PATH, should be a valid JSON file. Error: %s", err.Error())
+					return fmt.Errorf("the second argument, source-metrics-path, should be a valid JSON file. Error: %s", err.Error())
 				}
 
 				if fileInfo.IsDir() {
-					return fmt.Errorf("the second argument, SOURCE-METRICS-PATH, should be a valid JSON file, not a directory.")
+					return fmt.Errorf("the second argument, source-metrics-path, should be a valid JSON file, not a directory")
 				}
 			}
 
@@ -73,5 +73,4 @@ func installCollectCmd(app *App) error {
 	collectCmd.Flags().BoolVarP(&app.collectConfig.dryRun, "dry-run", "d", false, "perform a dry-run where a report is collected, but not written to disk")
 
 	app.rootCmd.AddCommand(collectCmd)
-	return nil
 }

cmd/insights/commands/root.go

@@ -1,3 +1,4 @@
+// Package commands contains the commands for the Ubuntu Insights CLI.
 package commands
 
 import (
@@ -7,8 +8,7 @@ import (
 	"github.com/ubuntu/ubuntu-insights/internal/constants"
 )
 
-const cmdName = "ubuntu-insights"
-
+// App represents the application.
 type App struct {
 	rootCmd *cobra.Command
 
@@ -30,11 +30,11 @@ var defaultRootConfig = rootConfig{
 	InsightsDir: constants.GetDefaultCachePath(),
 }
 
-// Registers commands and returns a new app
+// New registers commands and returns a new App.
 func New() (*App, error) {
 	a := App{}
 	a.rootCmd = &cobra.Command{
-		Use:           "ubuntu-insights [COMMAND]",
+		Use:           constants.CmdName + " [COMMAND]",
 		Short:         "",
 		Long:          "",
 		SilenceErrors: true,
@@ -48,7 +48,7 @@ func New() (*App, error) {
 	}
 
 	err := installRootCmd(&a)
-	err = installCollectCmd(&a)
+	installCollectCmd(&a)
 	installUploadCmd(&a)
 	installConsentCmd(&a)
 

cmd/insights/commands/root_test.go

@@ -2,9 +2,8 @@ package commands
 
 import (
 	"context"
-	"testing"
-
 	"log/slog"
+	"testing"
 
 	"github.com/stretchr/testify/assert"
 	"github.com/ubuntu/ubuntu-insights/internal/constants"

cmd/insights/main.go

@@ -1,9 +1,9 @@
+// Main package for the insights command line tool.
 package main
 
 import (
-	"os"
-
 	"log/slog"
+	"os"
 
 	"github.com/ubuntu/ubuntu-insights/cmd/insights/commands"
 	"github.com/ubuntu/ubuntu-insights/internal/constants"

cmd/insights/main_test.go

@@ -45,6 +45,7 @@ func TestRun(t *testing.T) {
 
 	for name, tc := range tests {
 		t.Run(name, func(t *testing.T) {
+			t.Parallel()
 			a := testApp{
 				done:            make(chan struct{}),
 				runError:        tc.runError,

internal/constants/constants.go

@@ -1,16 +1,20 @@
+// Package constants is responsible for defining the constants used in the application.
+// It also provides utility functions to get the default configuration and cache paths.
 package constants
 
 import (
-	"os"
-
 	"log/slog"
+	"os"
 )
 
 const (
-	// DefaultAppFolder is the name of the default root folder
+	// CmdName is the name of the command line tool.
+	CmdName = "ubuntu-insights"
+
+	// DefaultAppFolder is the name of the default root folder.
 	DefaultAppFolder = "ubuntu-insights"
 
-	// DefaultLogLevel is the default log level selected without any verbosity flags
+	// DefaultLogLevel is the default log level selected without any verbosity flags.
 	DefaultLogLevel = slog.LevelInfo
 
 	// DefaultConfigFileName is the default base name of the consent state files
@@ -26,7 +30,7 @@ type options struct {
 
 type option func(*options)
 
-// GetDefaultConfigPath is the default path to the configuration file
+// GetDefaultConfigPath is the default path to the configuration file.
 func GetDefaultConfigPath(opts ...option) string {
 	o := options{baseDir: os.UserCacheDir}
 	for _, opt := range opts {
@@ -36,7 +40,7 @@ func GetDefaultConfigPath(opts ...option) string {
 	return getBaseDir(o.baseDir) + string(os.PathSeparator) + DefaultAppFolder
 }
 
-// GetDefaultCachePath is the default path to the cache directory
+// GetDefaultCachePath is the default path to the cache directory.
 func GetDefaultCachePath(opts ...option) string {
 	o := options{baseDir: os.UserConfigDir}
 	for _, opt := range opts {
@@ -46,7 +50,7 @@ func GetDefaultCachePath(opts ...option) string {
 	return getBaseDir(o.baseDir) + string(os.PathSeparator) + DefaultAppFolder
 }
 
-// getBaseDir is a helper function to handle the case where the baseDir function returns an error, and instead return an empty string
+// getBaseDir is a helper function to handle the case where the baseDir function returns an error, and instead return an empty string.
 func getBaseDir(baseDirFunc func() (string, error)) string {
 	dir, err := baseDirFunc()
 	if err != nil {

internal/constants/constants_test.go

@@ -9,7 +9,8 @@ import (
 	"github.com/ubuntu/ubuntu-insights/internal/constants"
 )
 
-func Test_GetUserConfigDir(t *testing.T) {
+//nolint:dupl //Tests for GetDefaultConfigPath is very similar to GetDefaultCachePath.
+func Test_GetDefaultConfigPath(t *testing.T) {
 	t.Parallel()
 
 	tests := map[string]struct {
@@ -25,13 +26,13 @@ func Test_GetUserConfigDir(t *testing.T) {
 		"os.UserConfigDir error": {
 			want: string(os.PathSeparator) + constants.DefaultAppFolder,
 			mock: func() (string, error) {
-				return "", fmt.Errorf("error")
+				return "", fmt.Errorf("os.UserCacheDir error")
 			},
 		},
 		"os.UserConfigDir error 2": {
 			want: string(os.PathSeparator) + constants.DefaultAppFolder,
 			mock: func() (string, error) {
-				return "abc", fmt.Errorf("error")
+				return "abc", fmt.Errorf("os.UserCacheDir error")
 			},
 		},
 	}
@@ -45,7 +46,8 @@ func Test_GetUserConfigDir(t *testing.T) {
 	}
 }
 
-func Test_userCacheDir(t *testing.T) {
+//nolint:dupl //Tests for GetDefaultConfigPath is very similar to GetDefaultCachePath.
+func Test_GetDefaultCachePath(t *testing.T) {
 	t.Parallel()
 
 	tests := map[string]struct {
@@ -61,13 +63,13 @@ func Test_userCacheDir(t *testing.T) {
 		"os.UserCacheDir error": {
 			want: string(os.PathSeparator) + constants.DefaultAppFolder,
 			mock: func() (string, error) {
-				return "", fmt.Errorf("error")
+				return "", fmt.Errorf("os.UserCacheDir error")
 			},
 		},
-		"os.UserCacheDir error 2": {
+		"os.UserCacheDir error with return": {
 			want: string(os.PathSeparator) + constants.DefaultAppFolder,
 			mock: func() (string, error) {
-				return "abc", fmt.Errorf("error")
+				return "return", fmt.Errorf("os.UserCacheDir error")
 			},
 		},
 	}

internal/constants/export_test.go

@@ -2,7 +2,7 @@ package constants
 
 type Option = option
 
-func WithBaseDir(baseDir func() (string, error)) option {
+func WithBaseDir(baseDir func() (string, error)) func(o *options) {
 	return func(o *options) {
 		o.baseDir = baseDir
 	}

internal/testutils/cmd.go

@@ -1,3 +1,4 @@
+// Package testutils provides helper functions for testing
 package testutils
 
 import (
@@ -8,6 +9,7 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
+// CmdTestCase is a test case for testing cobra CMD flags.
 type CmdTestCase struct {
 	Name           string
 	Short          string
@@ -17,7 +19,9 @@ type CmdTestCase struct {
 	BaseCmd        *cobra.Command
 }
 
+// FlagTestHelper is a helper function to test cobra CMD flags.
 func FlagTestHelper(t *testing.T, testCase CmdTestCase) {
+	t.Helper()
 	var flag *pflag.Flag
 
 	if testCase.PersistentFlag {