Skip to content

Commit 31b8198

Browse files
committed
cli/command/volume: TestVolumeCreateClusterOpts: minor fixes and refactor
- assert unhandled error - discard stdout/stderr output Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
1 parent 5b8c08d commit 31b8198

1 file changed

Lines changed: 18 additions & 16 deletions

File tree

cli/command/volume/create_test.go

Lines changed: 18 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -233,25 +233,27 @@ func TestVolumeCreateClusterOpts(t *testing.T) {
233233
})
234234

235235
cmd := newCreateCommand(cli)
236+
cmd.SetOut(io.Discard)
237+
cmd.SetErr(io.Discard)
236238
cmd.SetArgs([]string{"name"})
237-
cmd.Flags().Set("driver", "csi")
238-
cmd.Flags().Set("group", "gronp")
239-
cmd.Flags().Set("scope", "multi")
240-
cmd.Flags().Set("sharing", "onewriter")
241-
cmd.Flags().Set("type", "mount")
242-
cmd.Flags().Set("sharing", "onewriter")
243-
cmd.Flags().Set("required-bytes", "1234")
244-
cmd.Flags().Set("limit-bytes", "567890")
239+
assert.Check(t, cmd.Flags().Set("driver", "csi"))
240+
assert.Check(t, cmd.Flags().Set("group", "gronp"))
241+
assert.Check(t, cmd.Flags().Set("scope", "multi"))
242+
assert.Check(t, cmd.Flags().Set("sharing", "onewriter"))
243+
assert.Check(t, cmd.Flags().Set("type", "mount"))
244+
assert.Check(t, cmd.Flags().Set("sharing", "onewriter"))
245+
assert.Check(t, cmd.Flags().Set("required-bytes", "1234"))
246+
assert.Check(t, cmd.Flags().Set("limit-bytes", "567890"))
245247

246-
cmd.Flags().Set("secret", "key1=secret1")
247-
cmd.Flags().Set("secret", "key2=secret2")
248+
assert.Check(t, cmd.Flags().Set("secret", "key1=secret1"))
249+
assert.Check(t, cmd.Flags().Set("secret", "key2=secret2"))
248250

249-
cmd.Flags().Set("topology-required", "region=R1,zone=Z1")
250-
cmd.Flags().Set("topology-required", "region=R1,zone=Z2")
251-
cmd.Flags().Set("topology-required", "region=R1,zone=Z3")
251+
assert.Check(t, cmd.Flags().Set("topology-required", "region=R1,zone=Z1"))
252+
assert.Check(t, cmd.Flags().Set("topology-required", "region=R1,zone=Z2"))
253+
assert.Check(t, cmd.Flags().Set("topology-required", "region=R1,zone=Z3"))
252254

253-
cmd.Flags().Set("topology-preferred", "region=R1,zone=Z2")
254-
cmd.Flags().Set("topology-preferred", "region=R1,zone=Z3")
255+
assert.Check(t, cmd.Flags().Set("topology-preferred", "region=R1,zone=Z2"))
256+
assert.Check(t, cmd.Flags().Set("topology-preferred", "region=R1,zone=Z3"))
255257

256-
cmd.Execute()
258+
assert.NilError(t, cmd.Execute())
257259
}

0 commit comments

Comments
 (0)