Broke out Config flags to internal.

Author: bengarrett

lib/example_test.go

@@ -25,18 +25,17 @@ func ExampleRead() {
 }
 
 func ExampleConfig_Status() {
-	c := Config{
-		test: true,
-	}
+	c := Config{}
+	c.SetTest()
 	if err := c.WalkDir("../test"); err != nil {
 		log.Panicln(err)
 	}
 	fmt.Print(c.Status())
 
 	c = Config{
 		Dupes: true,
-		test:  true,
 	}
+	c.SetTest()
 	if err := c.WalkDir("../test"); err != nil {
 		log.Panicln(err)
 	}

lib/zipcmt.go

@@ -22,7 +22,6 @@ import (
 )
 
 type Config struct {
-	Timer     time.Time
 	Dirs      []string
 	Save      string
 	LogName   string
@@ -35,13 +34,30 @@ type Config struct {
 	Raw       bool
 	Print     bool
 	Quiet     bool
-	test      bool
-	zips      int
-	cmmts     int
-	names     int
-	saved     int
-	exports   export
-	hashes    hash
+	internal
+}
+
+type internal struct {
+	test    bool
+	zips    int
+	cmmts   int
+	names   int
+	saved   int
+	exports export
+	hashes  hash
+	timer   time.Time
+}
+
+func (i *internal) SetTest() {
+	i.test = true
+}
+
+func (i *internal) SetTimer() {
+	i.timer = time.Now()
+}
+
+func (i *internal) Timer() time.Duration {
+	return time.Since(i.timer)
 }
 
 type (
@@ -263,7 +279,7 @@ func (c Config) Status() string {
 			s := fmt.Sprintf("Saved %d comments from %d finds", c.saved, c.cmmts)
 			c.WriteLog(s)
 		}
-		s := fmt.Sprintf("Scan finished, time taken: %s", time.Since(c.Timer))
+		s := fmt.Sprintf("Scan finished, time taken: %s", c.Timer())
 		c.WriteLog(s)
 	}
 	if c.Quiet {
@@ -293,7 +309,7 @@ func (c Config) Status() string {
 		color.Primary.Sprintf("%d %s%s", c.cmmts, unq, cm)
 	if !c.test {
 		s += color.Secondary.Sprint(", taking ") +
-			color.Primary.Sprintf("%s", time.Since(c.Timer)) + "\n"
+			color.Primary.Sprintf("%s", c.Timer()) + "\n"
 	}
 	return s
 }

lib/zipcmt_test.go

@@ -43,9 +43,9 @@ func TestConfig_Clean(t *testing.T) {
 				Raw:       tt.fields.Raw,
 				Print:     tt.fields.Print,
 				Quiet:     tt.fields.Quiet,
-				zips:      tt.fields.zips,
-				cmmts:     tt.fields.cmmts,
 			}
+			c.zips = tt.fields.zips
+			c.cmmts = tt.fields.cmmts
 			if err := c.clean(); (err != nil) != tt.wantErr {
 				t.Errorf("Config.cean() error = %v, wantErr %v", err, tt.wantErr)
 			}
@@ -62,8 +62,6 @@ func Test_Read(t *testing.T) {
 		Raw       bool
 		Print     bool
 		Quiet     bool
-		zips      int
-		cmmts     int
 	}
 	tests := []struct {
 		name     string
@@ -129,9 +127,9 @@ func TestConfig_Scans(t *testing.T) {
 				Raw:       tt.fields.Raw,
 				Print:     tt.fields.Print,
 				Quiet:     tt.fields.Quiet,
-				zips:      tt.fields.zips,
-				cmmts:     tt.fields.cmmts,
 			}
+			c.zips = tt.fields.zips
+			c.cmmts = tt.fields.cmmts
 			if err := c.WalkDir(tt.root); (err != nil) != tt.wantErr {
 				t.Errorf("Config.Scans() error = %v, wantErr %v", err, tt.wantErr)
 			}
@@ -171,9 +169,9 @@ func TestConfig_separator(t *testing.T) {
 				Raw:       tt.fields.Raw,
 				Print:     tt.fields.Print,
 				Quiet:     tt.fields.Quiet,
-				zips:      tt.fields.zips,
-				cmmts:     tt.fields.cmmts,
 			}
+			c.zips = tt.fields.zips
+			c.cmmts = tt.fields.cmmts
 			if got := strings.TrimSpace(c.separator(tt.fname)); got != tt.want {
 				t.Errorf("Config.separator() = %v, want %v", got, tt.want)
 			}
@@ -213,10 +211,10 @@ func TestConfig_Status(t *testing.T) {
 				Raw:       tt.fields.Raw,
 				Print:     tt.fields.Print,
 				Quiet:     tt.fields.Quiet,
-				test:      true,
-				zips:      tt.fields.zips,
-				cmmts:     tt.fields.cmmts,
 			}
+			c.zips = tt.fields.zips
+			c.cmmts = tt.fields.cmmts
+			c.SetTest()
 			if got := strings.TrimSpace(c.Status()); got != tt.want {
 				t.Errorf("Config.Status() = \ngot:  %v,\nwant: %v", got, tt.want)
 			}

main.go

@@ -11,7 +11,6 @@ import (
 	"runtime"
 	"strings"
 	"text/tabwriter"
-	"time"
 
 	zipcmt "github.com/bengarrett/zipcmt/lib"
 	"github.com/gookit/color"
@@ -32,7 +31,7 @@ func main() {
 	const ellipsis = "\u2026"
 	var c zipcmt.Config
 	var noprint bool
-	c.Timer = time.Now()
+	c.SetTimer()
 	flag.BoolVar(&noprint, "noprint", false, "do not print comments to the terminal to improve the performance of the scan")
 	flag.BoolVar(&c.NoWalk, "norecursive", false, "do not recursively walk through any subdirectories while scanning for zip archives")
 	flag.BoolVar(&c.Export, "export", false, fmt.Sprintf("save the comments as text files stored alongside the zip files (%s)",