Skip to content

Commit 22c2e53

Browse files
authored
fix(instance): image update positional arg (#3468)
1 parent 3b4ca44 commit 22c2e53

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

cmd/scw/testdata/test-all-usage-instance-image-update-usage.golden

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
Update properties of an instance image.
44

55
USAGE:
6-
scw instance image update [arg=value ...]
6+
scw instance image update <image-id ...> [arg=value ...]
77

88
EXAMPLES:
99
Update image name

docs/commands/instance.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,7 @@ Update properties of an instance image.
265265
**Usage:**
266266

267267
```
268-
scw instance image update [arg=value ...]
268+
scw instance image update <image-id ...> [arg=value ...]
269269
```
270270

271271

internal/namespaces/instance/v1/custom_image.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,7 @@ func imageUpdateCommand() *core.Command {
307307
{
308308
Name: "image-id",
309309
Required: true,
310-
Positional: false,
310+
Positional: true,
311311
},
312312
{
313313
Name: "name",

internal/namespaces/instance/v1/custom_image_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ func Test_ImageUpdate(t *testing.T) {
9999
t.Run("Change name", core.Test(&core.TestConfig{
100100
BeforeFunc: createImage("ImageName"),
101101
Commands: GetCommands(),
102-
Cmd: "scw instance image update image-id={{ .ImageName.Image.ID }} name=foo",
102+
Cmd: "scw instance image update {{ .ImageName.Image.ID }} name=foo",
103103
Check: core.TestCheckCombine(
104104
func(t *testing.T, ctx *core.CheckFuncCtx) {
105105
assert.Equal(t, "foo", ctx.Result.(*instance.UpdateImageResponse).Image.Name)
@@ -116,7 +116,7 @@ func Test_ImageUpdate(t *testing.T) {
116116
t.Run("Change public from default false to true", core.Test(&core.TestConfig{
117117
BeforeFunc: createImage("ImagePub"),
118118
Commands: GetCommands(),
119-
Cmd: "scw instance image update image-id={{ .ImagePub.Image.ID }} public=true",
119+
Cmd: "scw instance image update {{ .ImagePub.Image.ID }} public=true",
120120
Check: core.TestCheckCombine(
121121
func(t *testing.T, ctx *core.CheckFuncCtx) {
122122
assert.Equal(t, true, ctx.Result.(*instance.UpdateImageResponse).Image.Public)
@@ -137,7 +137,7 @@ func Test_ImageUpdate(t *testing.T) {
137137
createImage("ImageExtraVol"),
138138
),
139139
Commands: GetCommands(),
140-
Cmd: "scw instance image update image-id={{ .ImageExtraVol.Image.ID }} extra-volumes.1.id={{ .SnapshotVol.ID }}",
140+
Cmd: "scw instance image update {{ .ImageExtraVol.Image.ID }} extra-volumes.1.id={{ .SnapshotVol.ID }}",
141141
Check: core.TestCheckCombine(
142142
func(t *testing.T, ctx *core.CheckFuncCtx) {
143143
assert.Equal(t, "snapVol", ctx.Result.(*instance.UpdateImageResponse).Image.ExtraVolumes["1"].Name)

0 commit comments

Comments
 (0)