Add test for Game.run()

Author: lfcd85

kadai3-1/lfcd85/typinggame/export_test.go

@@ -0,0 +1,3 @@
+package typinggame
+
+var ExportGameRun = (*Game).run

kadai3-1/lfcd85/typinggame/typinggame.go

@@ -18,12 +18,11 @@ type Game struct {
 }
 
 func Execute(g Game) error {
-	g.run(os.Stdin, os.Stdout)
+	g.run(inputChannel(os.Stdin), os.Stdout)
 	return nil
 }
 
-func (g *Game) run(r io.Reader, w io.Writer) {
-	ch := input(r)
+func (g *Game) run(ch <-chan string, w io.Writer) {
 	bc := context.Background()
 	ctx, cancel := context.WithTimeout(bc, g.TimeLimit)
 	defer cancel()
@@ -40,10 +39,10 @@ LOOP:
 		case input := <-ch:
 			if input == word {
 				score++
-				fmt.Fprintln(w, "ok! current score:", score)
+				fmt.Fprintln(w, input, "... OK! current score:", score)
 				word = g.Words[rand.Intn(len(g.Words))]
 			} else {
-				fmt.Fprintln(w, "ng")
+				fmt.Fprintln(w, input, "... NG: try again.")
 			}
 		case <-ctx.Done():
 			fmt.Fprintln(w)
@@ -53,7 +52,7 @@ LOOP:
 	}
 }
 
-func input(r io.Reader) <-chan string {
+func inputChannel(r io.Reader) <-chan string {
 	ch := make(chan string)
 
 	go func() {

kadai3-1/lfcd85/typinggame/typinggame_test.go

@@ -1,6 +1,8 @@
 package typinggame_test
 
 import (
+	"bytes"
+	"regexp"
 	"testing"
 	"time"
 
@@ -9,11 +11,56 @@ import (
 
 func TestExecute(t *testing.T) {
 	g := typinggame.Game{
-		typinggame.Words{"hoge", "fuga", "piyo"},
+		typinggame.Words{"hoge"},
 		1 * time.Second,
 	}
 
 	if err := typinggame.Execute(g); err != nil {
 		t.Errorf("failed to execute new game: %v", err)
 	}
 }
+
+func TestGame_run(t *testing.T) {
+	ch := make(chan string)
+	go func() {
+		time.Sleep(100 * time.Millisecond)
+		ch <- "hoga"
+		time.Sleep(100 * time.Millisecond)
+		ch <- "hoge"
+	}()
+
+	g := typinggame.Game{
+		typinggame.Words{"hoge"},
+		1 * time.Second,
+	}
+
+	var output bytes.Buffer
+	typinggame.ExportGameRun(&g, ch, &output)
+
+	cases := []struct {
+		output   string
+		expected bool
+	}{
+		{"hoga ... NG", true},
+		{"hoga ... OK", false},
+		{"hoge ... OK", true},
+		{"hoge ... NG", false},
+	}
+
+	for _, c := range cases {
+		c := c
+		t.Run(c.output, func(t *testing.T) {
+			t.Parallel()
+
+			actual := regexp.MustCompile(c.output).MatchString(output.String())
+			if actual != c.expected {
+				switch c.expected {
+				case true:
+					t.Errorf("%v should be outputted but actually was not", c.output)
+				case false:
+					t.Errorf("%v should not be outputted but actyally was", c.output)
+				}
+			}
+		})
+	}
+}