Skip to content

Commit 7fcabc9

Browse files
lucassabreulucassabreu
committed
(wip)
1 parent 37a2dc9 commit 7fcabc9

File tree

4 files changed

+70
-69
lines changed

4 files changed

+70
-69
lines changed

‎pkg/cmd/task/add/add_test.go

+27-30
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,12 @@ func TestCmdAdd(t *testing.T) {
2828
}
2929
}
3030

31+
dFactory := func(t *testing.T) cmdutil.Factory {
32+
f := mocks.NewMockFactory(t)
33+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
34+
return f
35+
}
36+
3137
tts := []struct {
3238
name string
3339
args []string
@@ -37,44 +43,34 @@ func TestCmdAdd(t *testing.T) {
3743
err string
3844
}{
3945
{
40-
name: "only one format",
41-
args: []string{"--format={}", "-q", "-j", "-n=OK", "-p=OK"},
42-
err: "flags can't be used together.*format.*json.*quiet",
43-
factory: func(t *testing.T) cmdutil.Factory {
44-
return mocks.NewMockFactory(t)
45-
},
46+
name: "only one format",
47+
args: []string{"--format={}", "-q", "-j", "-n=OK", "-p=OK"},
48+
err: "flags can't be used together.*format.*json.*quiet",
49+
factory: dFactory,
4650
},
4751
{
48-
name: "billable or not",
49-
args: []string{"--billable", "--not-billable", "-n=OK", "-p=OK"},
50-
err: "flags can't be used together.*billable.*not-billable",
51-
factory: func(t *testing.T) cmdutil.Factory {
52-
return mocks.NewMockFactory(t)
53-
},
52+
name: "billable or not",
53+
args: []string{"--billable", "--not-billable", "-n=OK", "-p=OK"},
54+
err: "flags can't be used together.*billable.*not-billable",
55+
factory: dFactory,
5456
},
5557
{
56-
name: "assignee or no assignee",
57-
args: []string{"--assignee=l", "--no-assignee", "-n=OK", "-p=OK"},
58-
err: "flags can't be used together.*assignee.*no-assignee",
59-
factory: func(t *testing.T) cmdutil.Factory {
60-
return mocks.NewMockFactory(t)
61-
},
58+
name: "assignee or no assignee",
59+
args: []string{"--assignee=l", "--no-assignee", "-n=OK", "-p=OK"},
60+
err: "flags can't be used together.*assignee.*no-assignee",
61+
factory: dFactory,
6262
},
6363
{
64-
name: "name required",
65-
args: []string{"-p=OK"},
66-
err: `"name" not set`,
67-
factory: func(t *testing.T) cmdutil.Factory {
68-
return mocks.NewMockFactory(t)
69-
},
64+
name: "name required",
65+
args: []string{"-p=OK"},
66+
err: `"name" not set`,
67+
factory: dFactory,
7068
},
7169
{
72-
name: "project required",
73-
args: []string{"-n=OK"},
74-
err: `"project" not set`,
75-
factory: func(t *testing.T) cmdutil.Factory {
76-
return mocks.NewMockFactory(t)
77-
},
70+
name: "project required",
71+
args: []string{"-n=OK"},
72+
err: `"project" not set`,
73+
factory: dFactory,
7874
},
7975
{
8076
name: "client error",
@@ -97,6 +93,7 @@ func TestCmdAdd(t *testing.T) {
9793
args: []string{"-n=a", "-p=b"},
9894
factory: func(t *testing.T) cmdutil.Factory {
9995
f := mocks.NewMockFactory(t)
96+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
10097
f.On("GetWorkspaceID").
10198
Return("", errors.New("workspace error"))
10299
return f

‎pkg/cmd/task/delete/delete_test.go

+8-2
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,19 @@ func TestCmdDelete(t *testing.T) {
3131
args: []string{},
3232
err: "requires arg task",
3333
params: func(t *testing.T) (cmdutil.Factory, report) {
34-
return mocks.NewMockFactory(t), nil
34+
f := mocks.NewMockFactory(t)
35+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
36+
return f, nil
3537
},
3638
},
3739
{
3840
name: "project is required",
3941
err: "flag.*project.*not set",
4042
args: []string{"task-id"},
4143
params: func(t *testing.T) (cmdutil.Factory, report) {
42-
return mocks.NewMockFactory(t), nil
44+
f := mocks.NewMockFactory(t)
45+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
46+
return f, nil
4347
},
4448
},
4549
{
@@ -48,6 +52,7 @@ func TestCmdDelete(t *testing.T) {
4852
args: []string{"task-id", "-p", "p-1"},
4953
params: func(t *testing.T) (cmdutil.Factory, report) {
5054
f := mocks.NewMockFactory(t)
55+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
5156
f.On("GetWorkspaceID").Return("", errors.New("w error"))
5257
return f, nil
5358
},
@@ -58,6 +63,7 @@ func TestCmdDelete(t *testing.T) {
5863
args: []string{"task-id", "-p", "p-1"},
5964
params: func(t *testing.T) (cmdutil.Factory, report) {
6065
f := mocks.NewMockFactory(t)
66+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
6167
f.On("GetWorkspaceID").Return("w", nil)
6268
f.On("Client").Return(nil, errors.New("c error"))
6369
return f, nil

‎pkg/cmd/task/edit/edit_test.go

+30-36
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,11 @@ func TestCmdEdit(t *testing.T) {
2828
return nil
2929
}
3030
}
31+
dFactory := func(t *testing.T) cmdutil.Factory {
32+
f := mocks.NewMockFactory(t)
33+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
34+
return f
35+
}
3136

3237
tts := []struct {
3338
name string
@@ -38,52 +43,40 @@ func TestCmdEdit(t *testing.T) {
3843
err string
3944
}{
4045
{
41-
name: "task id required",
42-
args: []string{"-p=cli"},
43-
err: `requires arg task`,
44-
factory: func(t *testing.T) cmdutil.Factory {
45-
return mocks.NewMockFactory(t)
46-
},
46+
name: "task id required",
47+
args: []string{"-p=cli"},
48+
err: `requires arg task`,
49+
factory: dFactory,
4750
},
4851
{
49-
name: "project required",
50-
args: []string{"task"},
51-
err: `"project" not set`,
52-
factory: func(t *testing.T) cmdutil.Factory {
53-
return mocks.NewMockFactory(t)
54-
},
52+
name: "project required",
53+
args: []string{"task"},
54+
err: `"project" not set`,
55+
factory: dFactory,
5556
},
5657
{
57-
name: "task id not empty",
58-
args: []string{" ", "-p=cli"},
59-
err: `task id should not be empty`,
60-
factory: func(t *testing.T) cmdutil.Factory {
61-
return mocks.NewMockFactory(t)
62-
},
58+
name: "task id not empty",
59+
args: []string{" ", "-p=cli"},
60+
err: `task id should not be empty`,
61+
factory: dFactory,
6362
},
6463
{
65-
name: "only one format",
66-
args: []string{"--format={}", "-q", "-j", "-p=OK", "edit"},
67-
err: "flags can't be used together.*format.*json.*quiet",
68-
factory: func(t *testing.T) cmdutil.Factory {
69-
return mocks.NewMockFactory(t)
70-
},
64+
name: "only one format",
65+
args: []string{"--format={}", "-q", "-j", "-p=OK", "edit"},
66+
err: "flags can't be used together.*format.*json.*quiet",
67+
factory: dFactory,
7168
},
7269
{
73-
name: "billable or not",
74-
args: []string{"--billable", "--not-billable", "edit", "-p=OK"},
75-
err: "flags can't be used together.*billable.*not-billable",
76-
factory: func(t *testing.T) cmdutil.Factory {
77-
return mocks.NewMockFactory(t)
78-
},
70+
name: "billable or not",
71+
args: []string{"--billable", "--not-billable", "edit", "-p=OK"},
72+
err: "flags can't be used together.*billable.*not-billable",
73+
factory: dFactory,
7974
},
8075
{
81-
name: "assignee or no assignee",
82-
args: []string{"--assignee=l", "--no-assignee", "edit", "-p=OK"},
83-
err: "flags can't be used together.*assignee.*no-assignee",
84-
factory: func(t *testing.T) cmdutil.Factory {
85-
return mocks.NewMockFactory(t)
86-
},
76+
name: "assignee or no assignee",
77+
args: []string{"--assignee=l", "--no-assignee", "edit", "-p=OK"},
78+
err: "flags can't be used together.*assignee.*no-assignee",
79+
factory: dFactory,
8780
},
8881
{
8982
name: "client error",
@@ -106,6 +99,7 @@ func TestCmdEdit(t *testing.T) {
10699
args: []string{"edit", "-p=b"},
107100
factory: func(t *testing.T) cmdutil.Factory {
108101
f := mocks.NewMockFactory(t)
102+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
109103
f.On("GetWorkspaceID").
110104
Return("", errors.New("workspace error"))
111105
return f

‎pkg/cmd/task/list/list_test.go

+5-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,9 @@ func TestCmdList(t *testing.T) {
2727
name: "missing project",
2828
err: "required flag.*project",
2929
params: func(t *testing.T) (cmdutil.Factory, report) {
30-
return mocks.NewMockFactory(t), nil
30+
f := mocks.NewMockFactory(t)
31+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
32+
return f, nil
3133
},
3234
},
3335
{
@@ -36,6 +38,7 @@ func TestCmdList(t *testing.T) {
3638
args: []string{"-p=p-1"},
3739
params: func(t *testing.T) (cmdutil.Factory, report) {
3840
f := mocks.NewMockFactory(t)
41+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
3942
f.On("GetWorkspaceID").Return("", errors.New("error"))
4043
return f, nil
4144
},
@@ -46,6 +49,7 @@ func TestCmdList(t *testing.T) {
4649
args: []string{"-p=p-1"},
4750
params: func(t *testing.T) (cmdutil.Factory, report) {
4851
f := mocks.NewMockFactory(t)
52+
f.EXPECT().Config().Return(&mocks.SimpleConfig{})
4953
f.On("GetWorkspaceID").Return("w", nil)
5054
f.On("Client").Return(nil, errors.New("error"))
5155
return f, nil

0 commit comments

Comments
 (0)