Extend unit tests of updates using CIDRSliceFlag (#175)

Author: joaopalet

internal/cmd/mariadb/instance/update/update_test.go

@@ -226,6 +226,17 @@ func TestParseInput(t *testing.T) {
 			flagValues:  fixtureFlagValues(),
 			isValid:     false,
 		},
+		{
+			description: "no acl flag",
+			argValues:   fixtureArgValues(),
+			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, sgwAclFlag)
+			}),
+			isValid: true,
+			expectedModel: fixtureInputModel(func(model *inputModel) {
+				model.SgwAcl = nil
+			}),
+		},
 		{
 			description:  "repeated acl flags",
 			argValues:    fixtureArgValues(),

internal/cmd/mongodbflex/instance/update/update_test.go

@@ -251,6 +251,17 @@ func TestParseInput(t *testing.T) {
 			}),
 			isValid: false,
 		},
+		{
+			description: "no acl flag",
+			argValues:   fixtureArgValues(),
+			flagValues: fixtureStandardFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, aclFlag)
+			}),
+			isValid: true,
+			expectedModel: fixtureStandardInputModel(func(model *inputModel) {
+				model.ACL = nil
+			}),
+		},
 		{
 			description: "repeated acl flags",
 			argValues:   fixtureArgValues(),

internal/cmd/opensearch/instance/update/update_test.go

@@ -226,6 +226,17 @@ func TestParseInput(t *testing.T) {
 			flagValues:  fixtureFlagValues(),
 			isValid:     false,
 		},
+		{
+			description: "no acl flag",
+			argValues:   fixtureArgValues(),
+			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, sgwAclFlag)
+			}),
+			isValid: true,
+			expectedModel: fixtureInputModel(func(model *inputModel) {
+				model.SgwAcl = nil
+			}),
+		},
 		{
 			description:  "repeated acl flags",
 			argValues:    fixtureArgValues(),

internal/cmd/postgresflex/instance/update/update_test.go

@@ -251,6 +251,17 @@ func TestParseInput(t *testing.T) {
 			}),
 			isValid: false,
 		},
+		{
+			description: "no acl flag",
+			argValues:   fixtureArgValues(),
+			flagValues: fixtureStandardFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, aclFlag)
+			}),
+			isValid: true,
+			expectedModel: fixtureStandardInputModel(func(model *inputModel) {
+				model.ACL = nil
+			}),
+		},
 		{
 			description: "repeated acl flags",
 			argValues:   fixtureArgValues(),

internal/cmd/rabbitmq/instance/update/update_test.go

@@ -226,6 +226,17 @@ func TestParseInput(t *testing.T) {
 			flagValues:  fixtureFlagValues(),
 			isValid:     false,
 		},
+		{
+			description: "no acl flag",
+			argValues:   fixtureArgValues(),
+			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, sgwAclFlag)
+			}),
+			isValid: true,
+			expectedModel: fixtureInputModel(func(model *inputModel) {
+				model.SgwAcl = nil
+			}),
+		},
 		{
 			description:  "repeated acl flags",
 			argValues:    fixtureArgValues(),

internal/cmd/redis/instance/update/update_test.go

@@ -226,6 +226,17 @@ func TestParseInput(t *testing.T) {
 			flagValues:  fixtureFlagValues(),
 			isValid:     false,
 		},
+		{
+			description: "no acl flag",
+			argValues:   fixtureArgValues(),
+			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, sgwAclFlag)
+			}),
+			isValid: true,
+			expectedModel: fixtureInputModel(func(model *inputModel) {
+				model.SgwAcl = nil
+			}),
+		},
 		{
 			description:  "repeated acl flags",
 			argValues:    fixtureArgValues(),