test: add ioutil abstraction coverage

Author: everdrone

cmd/get.go

@@ -16,7 +16,7 @@ var GetCmd = &cobra.Command{
 	Short: "Scrape and download assets from a URL, a file or a both",
 	Args:  cobra.MinimumNArgs(1),
 	RunE: func(cmd *cobra.Command, args []string) error {
-		log.Logger = instance.DefaultLogger(cmd.OutOrStderr())
+		log.Logger = log.Output(instance.DefaultLogger(cmd.OutOrStderr()))
 
 		g := instance.New(cmd)
 		g.ParseFlags()

internal/instance/log.go

@@ -8,11 +8,10 @@ import (
 
 	"github.com/fatih/color"
 	"github.com/rs/zerolog"
-	"github.com/rs/zerolog/log"
 )
 
-func DefaultLogger(w io.Writer) zerolog.Logger {
-	return log.Output(zerolog.ConsoleWriter{
+func DefaultLogger(w io.Writer) zerolog.ConsoleWriter {
+	return zerolog.ConsoleWriter{
 		Out:             w,
 		TimeFormat:      time.RFC3339Nano,
 		FormatTimestamp: func(i interface{}) string { return "" },
@@ -62,13 +61,5 @@ func DefaultLogger(w io.Writer) zerolog.Logger {
 		FormatMessage: func(i interface{}) string {
 			return fmt.Sprintf("%s", i)
 		},
-	})
+	}
 }
-
-/*
-╷ Error: Something bad happened
-│   Description of the error, here we can be more detailed
-╵   something.hcl:2-3
-╷ Error: Something bad happened
-╵   Description of the error, here we can be more detailed
-*/

internal/instance/log_test.go

@@ -0,0 +1,57 @@
+package instance
+
+import (
+	"bytes"
+	"fmt"
+	"testing"
+
+	"github.com/rs/zerolog/log"
+)
+
+func TestDefaultLogger(t *testing.T) {
+	tests := []struct {
+		name string
+		f    func()
+		want string
+	}{
+		{
+			"trace",
+			func() { log.Trace().Bool("test", true).Msg("trace") },
+			" TRC  trace test=true\n",
+		},
+		{
+			"debug",
+			func() { log.Debug().Bool("test", true).Msg("debug") },
+			" DBG  debug test=true\n",
+		},
+		{
+			"info",
+			func() { log.Info().Bool("test", true).Msg("info") },
+			" INF  info test=true\n",
+		},
+		{
+			"warning",
+			func() { log.Warn().Bool("test", true).Msg("warn") },
+			" WRN  warn test=true\n",
+		},
+		{
+			"error",
+			func() { log.Err(fmt.Errorf("text")).Bool("test", true).Msg("err") },
+			" ERR  err error=text test=true\n",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			buf := bytes.Buffer{}
+
+			log.Logger = log.Output(DefaultLogger(&buf))
+
+			tt.f()
+
+			if got := buf.String(); got != tt.want {
+				t.Errorf("got: %v, want: %v", got, tt.want)
+			}
+		})
+	}
+}

internal/utils/filesystem_test.go

@@ -116,3 +116,37 @@ func TestAbs(t *testing.T) {
 
 	resetWd()
 }
+
+func TestIoUtil(t *testing.T) {
+	initialWd, _ := os.Getwd()
+	defer func() {
+		_ = os.Chdir(initialWd)
+	}()
+
+	root := tu.GetOSRoot()
+	Fs = new(tu.MockFs)
+	Wd = root
+
+	// test writefile
+	if err := Io.WriteFile(Fs, "test.txt", []byte("test"), 0644); err != nil {
+		t.Fail()
+	}
+
+	// test readfile
+	if b, err := Io.ReadFile(Fs, "test.txt"); err != nil {
+		t.Fail()
+	} else {
+		if string(b) != "test" {
+			t.Fail()
+		}
+	}
+
+	// test exists
+	if exists, err := Io.Exists(Fs, "test.txt"); err != nil {
+		t.Fail()
+	} else {
+		if !exists {
+			t.Fail()
+		}
+	}
+}