fix: using correct option for takeOwnership flag
Fixed test to correctly assert the new flag Signed-off-by: Adam Blasko <adam.blasko1@gmail.com>
This commit is contained in:
parent
2b9ed46b32
commit
e16ba0399c
|
|
@ -155,7 +155,7 @@ func (st *HelmState) appendTakeOwnershipFlags(flags []string, helm helmexec.Inte
|
|||
return flags
|
||||
}
|
||||
switch {
|
||||
case ops.HideNotes:
|
||||
case ops.TakeOwnership:
|
||||
flags = append(flags, "--take-ownership")
|
||||
}
|
||||
return flags
|
||||
|
|
|
|||
|
|
@ -340,18 +340,18 @@ func TestAppendTakeOwnershipFlags(t *testing.T) {
|
|||
flags: []string{},
|
||||
helm: testutil.NewVersionHelmExec("3.16.0"),
|
||||
opt: &SyncOpts{
|
||||
HideNotes: true,
|
||||
TakeOwnership: true,
|
||||
},
|
||||
expected: []string{},
|
||||
},
|
||||
},
|
||||
{
|
||||
name: "hide-notes from cmd flag",
|
||||
name: "take-ownership from cmd flag",
|
||||
args: args{
|
||||
flags: []string{},
|
||||
helm: testutil.NewVersionHelmExec("3.17.0"),
|
||||
opt: &SyncOpts{
|
||||
HideNotes: true,
|
||||
TakeOwnership: true,
|
||||
},
|
||||
expected: []string{"--take-ownership"},
|
||||
},
|
||||
|
|
|
|||
Loading…
Reference in New Issue