/
hound_test.go
127 lines (109 loc) · 2.48 KB
/
hound_test.go
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
package main
import (
"github.com/ezekg/git-hound/Godeps/_workspace/src/sourcegraph.com/sourcegraph/go-diff/diff"
"testing"
)
func TestDiffs(t *testing.T) {
hound := &Hound{}
config := []byte(`
warn:
- '(?i)user(name)?\W*[:=,]\W*.+$'
fail:
- '(?i)pass(word)?\W*[:=,]\W*.+$'
`)
if err := hound.Parse(config); err != nil {
t.Fatalf("Should parse - %s", err)
}
// Should fail
{
fileName, hunk := getDiff(`diff --git a/test1.go b/test1.go
index 000000..000000 000000
--- a/test1.go
+++ b/test1.go
@@ -1,2 +3,4 @@
+Password: something-secret`)
warnc := make(chan string)
failc := make(chan error)
donec := make(chan bool)
go hound.Sniff(fileName, hunk, warnc, failc, donec)
select {
case <-failc:
break
case <-warnc:
t.Fatalf("Should not warn")
case <-donec:
t.Fatalf("Should receive message")
}
}
// Should pass but output warning
{
fileName, hunk := getDiff(`diff --git a/test2.go b/test2.go
index 000000..000000 000000
--- a/test2.go
+++ b/test2.go
@@ -1,2 +3,4 @@
+Username: something-secret`)
warnc := make(chan string)
failc := make(chan error)
donec := make(chan bool)
go hound.Sniff(fileName, hunk, warnc, failc, donec)
select {
case <-failc:
t.Fatalf("Should not fail")
case <-warnc:
break
case <-donec:
t.Fatalf("Should receive message")
}
}
// Should pass
{
fileName, hunk := getDiff(`diff --git a/test3.go b/test3.go
index 000000..000000 000000
--- a/test3.go
+++ b/test3.go
@@ -1,2 +3,4 @@
+Something that is okay to commit`)
warnc := make(chan string)
failc := make(chan error)
donec := make(chan bool)
go hound.Sniff(fileName, hunk, warnc, failc, donec)
select {
case <-failc:
t.Fatal("Should not fail")
case <-warnc:
t.Fatal("Should not warn")
case <-donec:
break
}
}
// Should only pay attention to added lines and pass
{
fileName, hunk := getDiff(`diff --git a/test4.go b/test4.go
index 000000..000000 000000
--- a/test4.go
+++ b/test4.go
@@ -1,2 +3,4 @@
-Password: something-secret`)
warnc := make(chan string)
failc := make(chan error)
donec := make(chan bool)
go hound.Sniff(fileName, hunk, warnc, failc, donec)
select {
case <-failc:
t.Fatal("Should not fail")
case <-warnc:
t.Fatal("Should not warn")
case <-donec:
break
}
}
}
func getDiff(diffContents string) (string, *diff.Hunk) {
fileDiff, _ := diff.ParseFileDiff([]byte(diffContents))
fileName := fileDiff.NewName
for _, hunk := range fileDiff.GetHunks() {
return fileName, hunk
}
return fileName, nil
}