Add more unit tests

exec/printer/default_test.go

@@ -0,0 +1,26 @@
+package printer
+
+import (
+	"github.com/captainhook-go/captainhook/configuration"
+	"github.com/captainhook-go/captainhook/events"
+	"github.com/captainhook-go/captainhook/hooks/app"
+	"github.com/captainhook-go/captainhook/test"
+	"testing"
+)
+
+func TestDefaultHookSuccess(t *testing.T) {
+	io := test.CreateFakeIO()
+	conf := test.CreateFakeConfig()
+	repo := test.CreateFakeRepo()
+	ctx := app.NewContext(io, conf, repo)
+	hook := &configuration.Hook{}
+	p := NewDefaultPrinter(io)
+
+	event := events.NewHookStartedEvent(ctx, hook)
+
+	p.HookStarted(event)
+
+	if len(io.Out) < 1 {
+		t.Errorf("Should have written something")
+	}
+}

git/config/config_test.go

@@ -0,0 +1,21 @@
+package config
+
+import (
+	"github.com/captainhook-go/captainhook/git/types"
+	"testing"
+)
+
+func TestGet(t *testing.T) {
+	g := types.NewCmd("config")
+	g.AddOptions(Get("user.name"))
+
+	if len(g.Options) < 3 {
+		t.Errorf("Option not added correctly")
+	}
+	if g.Options[1] != "--get" {
+		t.Errorf("Wrong option")
+	}
+	if g.Options[2] != "user.name" {
+		t.Errorf("Wrong option")
+	}
+}

git/diff/diff_test.go

@@ -0,0 +1,18 @@
+package diff
+
+import (
+	"github.com/captainhook-go/captainhook/git/types"
+	"testing"
+)
+
+func TestCached(t *testing.T) {
+	g := types.NewCmd("diff")
+	g.AddOptions(Cached)
+
+	if len(g.Options) < 2 {
+		t.Errorf("Option not added correctly")
+	}
+	if g.Options[1] != "--cached" {
+		t.Errorf("Wrong option")
+	}
+}

git/difftree/difftree_test.go

@@ -0,0 +1,11 @@
+package difftree
+
+import (
+	"testing"
+)
+
+func TestNothing(t *testing.T) {
+	if true == false {
+		t.Errorf("This is weired")
+	}
+}

git/log/log_test.go

@@ -0,0 +1,18 @@
+package log
+
+import (
+	"github.com/captainhook-go/captainhook/git/types"
+	"testing"
+)
+
+func TestAbbrevRev(t *testing.T) {
+	g := types.NewCmd("log")
+	g.AddOptions(AbbrevCommit)
+
+	if len(g.Options) < 2 {
+		t.Errorf("Option not added correctly")
+	}
+	if g.Options[1] != "--abbrev-commit" {
+		t.Errorf("Wrong option")
+	}
+}

git/revparse/revparse_test.go

@@ -0,0 +1,18 @@
+package revparse
+
+import (
+	"github.com/captainhook-go/captainhook/git/types"
+	"testing"
+)
+
+func TestAbbrevRev(t *testing.T) {
+	g := types.NewCmd("revparse")
+	g.AddOptions(AbbrevRef)
+
+	if len(g.Options) < 2 {
+		t.Errorf("Option not added correctly")
+	}
+	if g.Options[1] != "--abbrev-ref" {
+		t.Errorf("Wrong option")
+	}
+}

git/types/message_test.go

@@ -0,0 +1,14 @@
+package types
+
+import (
+	"testing"
+)
+
+func TestSubject(t *testing.T) {
+	expected := "Foo bar"
+	m := NewCommitMessage("Foo bar\n\nFiz baz.", "#")
+
+	if m.Subject() != expected {
+		t.Errorf("Wrong subject, expected '" + expected + "' got '" + m.Subject() + "'")
+	}
+}

test/helper.go

@@ -1,10 +1,13 @@
 package test
 
 import (
+	"context"
 	"github.com/captainhook-go/captainhook/configuration"
 	"github.com/captainhook-go/captainhook/git"
+	"github.com/captainhook-go/captainhook/git/types"
 	"github.com/captainhook-go/captainhook/hooks/app"
 	"github.com/captainhook-go/captainhook/io"
+	"strings"
 )
 
 func CreateFakeIO() *IOMock {
@@ -22,3 +25,9 @@ func CreateFakeRepo() *RepoMock {
 func CreateFakeHookContext(inOut io.IO, conf *configuration.Configuration, repo git.Repo) *app.Context {
 	return app.NewContext(inOut, conf, repo)
 }
+
+func CreateFakeExecutor() types.Executor {
+	return func(ctx context.Context, name string, debug bool, args ...string) (string, error) {
+		return name + " " + strings.Join(args, " "), nil
+	}
+}

test/io.go

@@ -5,6 +5,7 @@ import "github.com/captainhook-go/captainhook/io"
 type IOMock struct {
 	stdIn []string
 	args  map[string]string
+	Out   []string
 }
 
 func (inOut *IOMock) SetStdIn(input []string) {
@@ -56,7 +57,7 @@ func (inOut *IOMock) IsVerbose() bool {
 }
 
 func (inOut *IOMock) Write(message string, newline bool, verbosity int) {
-
+	inOut.Out = append(inOut.Out, message)
 }
 
 func (inOut *IOMock) Ask(message string, defaultValue string) string {