Validate report test output

Author: mschenk42

command/report.go

@@ -45,6 +45,7 @@ Options:
   -full-message=false        Include full commit message
   -terminal-off=false        Exclude time spent in terminal (Terminal plug-in is required)
   -color=false               Always output color even if no terminal is detected, i.e 'gtm report -color | less -R'
+	-testing=false             This is used for automated testing to force default test path
 
   Commit Limiting:
 
@@ -73,7 +74,7 @@ Options:
 // Run executes report command with args
 func (c ReportCmd) Run(args []string) int {
 	var limit int
-	var color, terminalOff, fullMessage bool
+	var color, terminalOff, fullMessage, testing bool
 	var today, yesterday, thisWeek, lastWeek, thisMonth, lastMonth, thisYear, lastYear, all bool
 	var before, after, message, author, tags, format string
 	cmdFlags := flag.NewFlagSet("report", flag.ContinueOnError)
@@ -96,6 +97,7 @@ func (c ReportCmd) Run(args []string) int {
 	cmdFlags.StringVar(&message, "message", "", "")
 	cmdFlags.StringVar(&tags, "tags", "", "")
 	cmdFlags.BoolVar(&all, "all", false, "")
+	cmdFlags.BoolVar(&testing, "testing", false, "")
 	cmdFlags.Usage = func() { c.Ui.Output(c.Help()) }
 	if err := cmdFlags.Parse(args); err != nil {
 		return 1
@@ -123,7 +125,7 @@ func (c ReportCmd) Run(args []string) int {
 	projCommits := []report.ProjectCommits{}
 
 	switch {
-	case !isMinGW && !isatty.IsTerminal(os.Stdin.Fd()):
+	case !testing && !isMinGW && !isatty.IsTerminal(os.Stdin.Fd()):
 		scanner := bufio.NewScanner(os.Stdin)
 		for scanner.Scan() {
 			if !sha1Regex.MatchString(scanner.Text()) {
@@ -140,7 +142,7 @@ func (c ReportCmd) Run(args []string) int {
 
 		projCommits = append(projCommits, report.ProjectCommits{Path: curProjPath, Commits: commits})
 
-	case len(cmdFlags.Args()) > 0:
+	case !testing && len(cmdFlags.Args()) > 0:
 		for _, a := range cmdFlags.Args() {
 			if !sha1Regex.MatchString(a) {
 				c.Ui.Error(fmt.Sprintf("%s %s", invalidSHA1, a))

command/report_test.go

@@ -33,12 +33,17 @@ func TestReportDefaultOptions(t *testing.T) {
 	ui := new(cli.MockUi)
 	c := ReportCmd{Ui: ui}
 
-	args := []string{""}
+	args := []string{"-testing=true"}
 	rc := c.Run(args)
 
 	if rc != 0 {
 		t.Errorf("gtm report(%+v), want 0 got %d, %s", args, rc, ui.ErrorWriter.String())
 	}
+
+	want := "2m 40s  89% [m] event/event.go"
+	if !strings.Contains(ui.OutputWriter.String(), want) {
+		t.Errorf("gtm report(%+v), want %s got %s, %s", args, want, ui.OutputWriter.String(), ui.ErrorWriter.String())
+	}
 }
 
 func TestReportSummary(t *testing.T) {
@@ -63,12 +68,17 @@ func TestReportSummary(t *testing.T) {
 	ui := new(cli.MockUi)
 	c := ReportCmd{Ui: ui}
 
-	args := []string{"-format", "summary"}
+	args := []string{"-format", "summary", "-testing=true"}
 	rc := c.Run(args)
 
 	if rc != 0 {
 		t.Errorf("gtm report(%+v), want 0 got %d, %s", args, rc, ui.ErrorWriter.String())
 	}
+
+	want := "3m  0s This is a commit"
+	if !strings.Contains(ui.OutputWriter.String(), want) {
+		t.Errorf("gtm report(%+v), want %s got %s, %s", args, want, ui.OutputWriter.String(), ui.ErrorWriter.String())
+	}
 }
 
 func TestReportAll(t *testing.T) {
@@ -93,6 +103,7 @@ func TestReportAll(t *testing.T) {
 	ui := new(cli.MockUi)
 	c := ReportCmd{Ui: ui}
 
+	// TODO: in order to test output of multi-project reporting, we need the ability to mock the project index
 	args := []string{"-all"}
 	rc := c.Run(args)
 
@@ -123,12 +134,17 @@ func TestReportTimelineHours(t *testing.T) {
 	ui := new(cli.MockUi)
 	c := ReportCmd{Ui: ui}
 
-	args := []string{"-format", "timeline-hours"}
+	args := []string{"-format", "timeline-hours", "-testing=true"}
 	rc := c.Run(args)
 
 	if rc != 0 {
 		t.Errorf("gtm report(%+v), want 0 got %d, %s", args, rc, ui.ErrorWriter.String())
 	}
+
+	want := "Sun Mar 20"
+	if !strings.Contains(ui.OutputWriter.String(), want) {
+		t.Errorf("gtm report(%+v), want %s got %s, %s", args, want, ui.OutputWriter.String(), ui.ErrorWriter.String())
+	}
 }
 
 func TestReportTimelineCommits(t *testing.T) {
@@ -153,12 +169,17 @@ func TestReportTimelineCommits(t *testing.T) {
 	ui := new(cli.MockUi)
 	c := ReportCmd{Ui: ui}
 
-	args := []string{"-format", "timeline-commits"}
+	args := []string{"-format", "timeline-commits", "-testing=true"}
 	rc := c.Run(args)
 
 	if rc != 0 {
 		t.Errorf("gtm report(%+v), want 0 got %d, %s", args, rc, ui.ErrorWriter.String())
 	}
+
+	want := "Wed Mar 06"
+	if !strings.Contains(ui.OutputWriter.String(), want) {
+		t.Errorf("gtm report(%+v), want %s got %s, %s", args, want, ui.OutputWriter.String(), ui.ErrorWriter.String())
+	}
 }
 
 func TestReportFiles(t *testing.T) {
@@ -183,12 +204,17 @@ func TestReportFiles(t *testing.T) {
 	ui := new(cli.MockUi)
 	c := ReportCmd{Ui: ui}
 
-	args := []string{"-format", "files"}
+	args := []string{"-format", "files", "-testing=true"}
 	rc := c.Run(args)
 
 	if rc != 0 {
 		t.Errorf("gtm report(%+v), want 0 got %d, %s", args, rc, ui.ErrorWriter.String())
 	}
+
+	want := "3m  0s"
+	if !strings.Contains(ui.OutputWriter.String(), want) {
+		t.Errorf("gtm report(%+v), want %s got %s, %s", args, want, ui.OutputWriter.String(), ui.ErrorWriter.String())
+	}
 }
 
 func TestReportInvalidOption(t *testing.T) {