Skip to content

Commit d315b71

Browse files
sebastianfeldmannsebastianfeldmann
committed
Fix condition tests
1 parent 19e20cf commit d315b71

File tree

5 files changed

+16
-16
lines changed

5 files changed

+16
-16
lines changed

‎hooks/conditions/filechanged/all_test.go‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ func TestAllFilesAreChanged(t *testing.T) {
1515
repo.SetFiles([]string{"foo", "bar", "baz"})
1616

1717
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "bar"}})
18-
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options)
18+
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options, []*configuration.Condition{})
1919

2020
action := NewAll(inOut, conf, repo)
2121
if !action.IsTrue(condition) {
@@ -32,7 +32,7 @@ func TestNotAllChangedFiles(t *testing.T) {
3232
repo.SetFiles([]string{"foo", "bar", "baz"})
3333

3434
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
35-
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options)
35+
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options, []*configuration.Condition{})
3636

3737
action := NewAll(inOut, conf, repo)
3838
if action.IsTrue(condition) {
@@ -49,7 +49,7 @@ func TestAllChangedFilesDetectionFailed(t *testing.T) {
4949
repo.SetFilesError(true)
5050

5151
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
52-
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options)
52+
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options, []*configuration.Condition{})
5353

5454
action := NewAll(inOut, conf, repo)
5555
if action.IsTrue(condition) {
@@ -66,7 +66,7 @@ func TestAllChangedFilesFailed(t *testing.T) {
6666
repo.SetFilesError(true)
6767

6868
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
69-
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options)
69+
condition := configuration.NewCondition("CaptainHook::ChangedFiles.All", options, []*configuration.Condition{})
7070

7171
action := NewAll(inOut, conf, repo)
7272
if action.IsTrue(condition) {

‎hooks/conditions/filechanged/any_test.go‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ func TestAnyFilesAreStaged(t *testing.T) {
1515
repo.SetFiles([]string{"foo", "bar", "baz"})
1616

1717
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
18-
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options)
18+
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options, []*configuration.Condition{})
1919

2020
action := NewAny(inOut, conf, repo)
2121
if !action.IsTrue(condition) {
@@ -32,7 +32,7 @@ func TestAnyNoFilesAreStaged(t *testing.T) {
3232
repo.SetFiles([]string{"foo", "bar", "baz"})
3333

3434
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"bing", "bang"}})
35-
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options)
35+
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options, []*configuration.Condition{})
3636

3737
action := NewAny(inOut, conf, repo)
3838
if action.IsTrue(condition) {
@@ -49,7 +49,7 @@ func TestAnyStagedFilesFailed(t *testing.T) {
4949
repo.SetFilesError(true)
5050

5151
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
52-
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options)
52+
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options, []*configuration.Condition{})
5353

5454
action := NewAny(inOut, conf, repo)
5555
if action.IsTrue(condition) {
@@ -65,7 +65,7 @@ func TestAnyStagedFilesDetectFailed(t *testing.T) {
6565
repo := test.CreateFakeRepo()
6666

6767
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
68-
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options)
68+
condition := configuration.NewCondition("CaptainHook::StagedFiles.Any", options, []*configuration.Condition{})
6969

7070
action := NewAny(inOut, conf, repo)
7171
if action.IsTrue(condition) {

‎hooks/conditions/filestaged/all_test.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ func TestAllFilesAreStaged(t *testing.T) {
1313
repo.SetFiles([]string{"foo", "bar", "baz"})
1414

1515
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "bar"}})
16-
condition := configuration.NewCondition("CaptainHook::StagedFiles.All", options)
16+
condition := configuration.NewCondition("CaptainHook::StagedFiles.All", options, []*configuration.Condition{})
1717

1818
action := NewAll(inOut, conf, repo)
1919
if !action.IsTrue(condition) {
@@ -28,7 +28,7 @@ func TestNotAllFilesAreStaged(t *testing.T) {
2828
repo.SetFiles([]string{"foo", "bar", "baz"})
2929

3030
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
31-
condition := configuration.NewCondition("CaptainHook::StagedFiles.All", options)
31+
condition := configuration.NewCondition("CaptainHook::StagedFiles.All", options, []*configuration.Condition{})
3232

3333
action := NewAll(inOut, conf, repo)
3434
if action.IsTrue(condition) {
@@ -43,7 +43,7 @@ func TestAllStagedFilesFailed(t *testing.T) {
4343
repo.SetFilesError(true)
4444

4545
options := configuration.NewOptions(map[string]interface{}{"files": []interface{}{"foo", "fiz"}})
46-
condition := configuration.NewCondition("CaptainHook::StagedFiles.All", options)
46+
condition := configuration.NewCondition("CaptainHook::StagedFiles.All", options, []*configuration.Condition{})
4747

4848
action := NewAll(inOut, conf, repo)
4949
if action.IsTrue(condition) {

‎hooks/conditions/inconfig/customvalueisfalsy_test.go‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ func TestCustomValueIsFalsy(t *testing.T) {
1212
repo := test.CreateFakeRepo()
1313

1414
options := configuration.NewOptions(map[string]interface{}{"value": "fiz"})
15-
condition := configuration.NewCondition("CaptainHook::InConfig.CustomValueIsFalsy", options)
15+
condition := configuration.NewCondition("CaptainHook::InConfig.CustomValueIsFalsy", options, []*configuration.Condition{})
1616

1717
action := NewCustomValueIsFalsy(inOut, conf, repo)
1818
if !action.IsTrue(condition) {
@@ -26,7 +26,7 @@ func TestCustomValueIsNotFalsy(t *testing.T) {
2626
repo := test.CreateFakeRepo()
2727

2828
options := configuration.NewOptions(map[string]interface{}{"value": "foo"})
29-
condition := configuration.NewCondition("CaptainHook::InConfig.CustomValueIsFalsy", options)
29+
condition := configuration.NewCondition("CaptainHook::InConfig.CustomValueIsFalsy", options, []*configuration.Condition{})
3030

3131
action := NewCustomValueIsFalsy(inOut, conf, repo)
3232
if action.IsTrue(condition) {

‎hooks/conditions/status/onbranch_test.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ func TestBranchConfigMissing(t *testing.T) {
1212
repo := test.CreateFakeRepo()
1313

1414
options := configuration.NewOptions(map[string]interface{}{})
15-
condition := configuration.NewCondition("CaptainHook::Status.OnBranch", options)
15+
condition := configuration.NewCondition("CaptainHook::Status.OnBranch", options, []*configuration.Condition{})
1616

1717
action := NewOnBranch(inOut, conf, repo)
1818
if action.IsTrue(condition) {
@@ -26,7 +26,7 @@ func TestBranchMainAllowed(t *testing.T) {
2626
repo := test.CreateFakeRepo()
2727

2828
options := configuration.NewOptions(map[string]interface{}{"name": "main"})
29-
condition := configuration.NewCondition("CaptainHook::Status.OnBranch", options)
29+
condition := configuration.NewCondition("CaptainHook::Status.OnBranch", options, []*configuration.Condition{})
3030

3131
action := NewOnBranch(inOut, conf, repo)
3232
if !action.IsTrue(condition) {
@@ -40,7 +40,7 @@ func TestBranchMainNotAllowed(t *testing.T) {
4040
repo := test.CreateFakeRepo()
4141

4242
options := configuration.NewOptions(map[string]interface{}{"name": "foo"})
43-
condition := configuration.NewCondition("CaptainHook::Status.OnBranch", options)
43+
condition := configuration.NewCondition("CaptainHook::Status.OnBranch", options, []*configuration.Condition{})
4444

4545
action := NewOnBranch(inOut, conf, repo)
4646
if action.IsTrue(condition) {

0 commit comments

Comments
 (0)