Skip to content

Commit da7a7e3

Browse files
jschwinger233CMGS
authored andcommitted
pass unittests
1 parent 5944f69 commit da7a7e3

File tree

2 files changed

+20
-18
lines changed

2 files changed

+20
-18
lines changed

types/options.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ func (o *DeployOptions) Validate() error {
3939
return errors.WithStack(ErrEmptyAppName)
4040
}
4141
if o.Podname == "" {
42-
return errors.WithStack(ErrEmptyAppName)
42+
return errors.WithStack(ErrEmptyPodName)
4343
}
4444
if o.Image == "" {
4545
return errors.WithStack(ErrEmptyImage)

types/options_test.go

+19-17
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ package types
33
import (
44
"testing"
55

6+
"github.com/pkg/errors"
7+
68
"github.com/stretchr/testify/assert"
79
)
810

@@ -18,7 +20,7 @@ func TestSetNodeOptions(t *testing.T) {
1820
DeltaVolume: VolumeMap{"/data": 1, "/data2": 2},
1921
DeltaStorage: -10,
2022
}
21-
assert.Equal(t, ErrEmptyNodeName, o.Validate())
23+
assert.Equal(t, ErrEmptyNodeName, errors.Unwrap(o.Validate()))
2224

2325
o.Nodename = "nodename"
2426
assert.NoError(t, o.Validate())
@@ -43,22 +45,22 @@ func TestDeployOptions(t *testing.T) {
4345
assert := assert.New(t)
4446

4547
o := &DeployOptions{Entrypoint: &Entrypoint{}}
46-
assert.Equal(ErrEmptyAppName, o.Validate())
48+
assert.Equal(ErrEmptyAppName, errors.Unwrap(o.Validate()))
4749

4850
o.Name = "testname"
49-
assert.Equal(ErrEmptyPodName, o.Validate())
51+
assert.Equal(ErrEmptyPodName, errors.Unwrap(o.Validate()))
5052

5153
o.Podname = "podname"
52-
assert.Equal(ErrEmptyImage, o.Validate())
54+
assert.Equal(ErrEmptyImage, errors.Unwrap(o.Validate()))
5355

5456
o.Image = "image"
55-
assert.Equal(ErrEmptyCount, o.Validate())
57+
assert.Equal(ErrEmptyCount, errors.Unwrap(o.Validate()))
5658

5759
o.Count = 1
58-
assert.Equal(ErrEmptyEntrypointName, o.Validate())
60+
assert.Equal(ErrEmptyEntrypointName, errors.Unwrap(o.Validate()))
5961

6062
o.Entrypoint.Name = "bad_entry_point"
61-
assert.Equal(ErrUnderlineInEntrypointName, o.Validate())
63+
assert.Equal(ErrUnderlineInEntrypointName, errors.Unwrap(o.Validate()))
6264

6365
o.Entrypoint.Name = "good-entry-point"
6466
assert.NoError(o.Validate())
@@ -68,7 +70,7 @@ func TestCopyOptions(t *testing.T) {
6870
assert := assert.New(t)
6971

7072
o := &CopyOptions{}
71-
assert.Equal(ErrNoFilesToCopy, o.Validate())
73+
assert.Equal(ErrNoFilesToCopy, errors.Unwrap(o.Validate()))
7274

7375
o.Targets = map[string][]string{
7476
"workload_id": {
@@ -83,10 +85,10 @@ func TestSendOptions(t *testing.T) {
8385
assert := assert.New(t)
8486

8587
o := &SendOptions{}
86-
assert.Equal(ErrNoWorkloadIDs, o.Validate())
88+
assert.Equal(ErrNoWorkloadIDs, errors.Unwrap(o.Validate()))
8789

8890
o.IDs = []string{"workload_id1", "workload_id2"}
89-
assert.Equal(ErrNoFilesToSend, o.Validate())
91+
assert.Equal(ErrNoFilesToSend, errors.Unwrap(o.Validate()))
9092

9193
o.Data = map[string][]byte{
9294
"filepath1": []byte("filecontent1"),
@@ -99,13 +101,13 @@ func TestReplaceOptions(t *testing.T) {
99101
assert := assert.New(t)
100102

101103
o := &ReplaceOptions{DeployOptions: DeployOptions{Entrypoint: &Entrypoint{}}}
102-
assert.Equal(ErrEmptyAppName, o.Validate())
104+
assert.Equal(ErrEmptyAppName, errors.Unwrap(o.Validate()))
103105

104106
o.DeployOptions.Name = "testname"
105-
assert.Equal(ErrEmptyEntrypointName, o.Validate())
107+
assert.Equal(ErrEmptyEntrypointName, errors.Unwrap(o.Validate()))
106108

107109
o.DeployOptions.Entrypoint.Name = "bad_entry_point"
108-
assert.Equal(ErrUnderlineInEntrypointName, o.Validate())
110+
assert.Equal(ErrUnderlineInEntrypointName, errors.Unwrap(o.Validate()))
109111

110112
o.DeployOptions.Entrypoint.Name = "good-entry-point"
111113
assert.NoError(o.Validate())
@@ -122,13 +124,13 @@ func TestValidatingAddNodeOptions(t *testing.T) {
122124
assert := assert.New(t)
123125

124126
o := &AddNodeOptions{}
125-
assert.Equal(ErrEmptyNodeName, o.Validate())
127+
assert.Equal(ErrEmptyNodeName, errors.Unwrap(o.Validate()))
126128

127129
o.Nodename = "nodename"
128-
assert.Equal(ErrEmptyPodName, o.Validate())
130+
assert.Equal(ErrEmptyPodName, errors.Unwrap(o.Validate()))
129131

130132
o.Podname = "podname"
131-
assert.Equal(ErrEmptyNodeEndpoint, o.Validate())
133+
assert.Equal(ErrEmptyNodeEndpoint, errors.Unwrap(o.Validate()))
132134

133135
o.Endpoint = "tcp://endpoint:2376"
134136
assert.NoError(o.Validate())
@@ -138,7 +140,7 @@ func TestImageOptions(t *testing.T) {
138140
assert := assert.New(t)
139141

140142
o := &ImageOptions{Step: -1}
141-
assert.Equal(ErrEmptyPodName, o.Validate())
143+
assert.Equal(ErrEmptyPodName, errors.Unwrap(o.Validate()))
142144

143145
o.Podname = "podname"
144146
assert.NoError(o.Validate())

0 commit comments

Comments
 (0)