Fix tests

Author: João Palet

internal/cmd/beta/sqlserverflex/user/create/create.go

@@ -25,10 +25,6 @@ const (
 	roleFlag       = "role"
 )
 
-var (
-	rolesDefault = []string{"read"}
-)
-
 type inputModel struct {
 	*globalflags.GlobalFlagModel
 

internal/cmd/beta/sqlserverflex/user/create/create_test.go

@@ -88,9 +88,26 @@ func TestParseInput(t *testing.T) {
 			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
 				delete(flagValues, usernameFlag)
 			}),
+			isValid: false,
+		},
+		{
+			description: "no database specified",
+			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, databaseFlag)
+			}),
 			isValid: true,
 			expectedModel: fixtureInputModel(func(model *inputModel) {
-				model.Username = nil
+				model.Database = nil
+			}),
+		},
+		{
+			description: "no roles specified",
+			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
+				delete(flagValues, roleFlag)
+			}),
+			isValid: true,
+			expectedModel: fixtureInputModel(func(model *inputModel) {
+				model.Roles = nil
 			}),
 		},
 		{
@@ -133,30 +150,6 @@ func TestParseInput(t *testing.T) {
 			}),
 			isValid: false,
 		},
-		{
-			description: "database missing",
-			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
-				delete(flagValues, databaseFlag)
-			}),
-			isValid: false,
-		},
-		{
-			description: "roles missing",
-			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
-				delete(flagValues, roleFlag)
-			}),
-			isValid: true,
-			expectedModel: fixtureInputModel(func(model *inputModel) {
-				model.Roles = &rolesDefault
-			}),
-		},
-		{
-			description: "invalid role",
-			flagValues: fixtureFlagValues(func(flagValues map[string]string) {
-				flagValues[roleFlag] = "invalid-role"
-			}),
-			isValid: false,
-		},
 	}
 
 	for _, tt := range tests {