diff options
Diffstat (limited to 'cmd')
-rw-r--r-- | cmd/commands/user.go | 40 | ||||
-rw-r--r-- | cmd/commands/user_test.go | 28 |
2 files changed, 0 insertions, 68 deletions
diff --git a/cmd/commands/user.go b/cmd/commands/user.go index 9f5e5ae0d..e3bc8208a 100644 --- a/cmd/commands/user.go +++ b/cmd/commands/user.go @@ -66,15 +66,6 @@ var ResetUserPasswordCmd = &cobra.Command{ RunE: resetUserPasswordCmdF, } -var updateUserEmailCmd = &cobra.Command{ - Use: "email [user] [new email]", - Short: "Change email of the user", - Long: "Change email of the user.", - Example: ` user email test user@example.com - user activate username`, - RunE: updateUserEmailCmdF, -} - var ResetUserMfaCmd = &cobra.Command{ Use: "resetmfa [users]", Short: "Turn off MFA", @@ -238,7 +229,6 @@ Global Flags: UserCreateCmd, UserInviteCmd, ResetUserPasswordCmd, - updateUserEmailCmd, ResetUserMfaCmd, DeleteUserCmd, DeleteAllUsersCmd, @@ -409,36 +399,6 @@ func resetUserPasswordCmdF(command *cobra.Command, args []string) error { return nil } -func updateUserEmailCmdF(command *cobra.Command, args []string) error { - a, err := cmd.InitDBCommandContextCobra(command) - if err != nil { - return err - } - - newEmail := args[1] - - if !model.IsValidEmail(newEmail) { - return errors.New("Invalid email: '" + newEmail + "'") - } - - if len(args) != 2 { - return errors.New("Expected two arguments. See help text for details.") - } - - user := getUserFromUserArg(a, args[0]) - if user == nil { - return errors.New("Unable to find user '" + args[0] + "'") - } - - user.Email = newEmail - _, errUpdate := a.UpdateUser(user, true) - if err != nil { - return errUpdate - } - - return nil -} - func resetUserMfaCmdF(command *cobra.Command, args []string) error { a, err := cmd.InitDBCommandContextCobra(command) if err != nil { diff --git a/cmd/commands/user_test.go b/cmd/commands/user_test.go index a1081c5d3..960ac3878 100644 --- a/cmd/commands/user_test.go +++ b/cmd/commands/user_test.go @@ -9,7 +9,6 @@ import ( "github.com/mattermost/mattermost-server/api" "github.com/mattermost/mattermost-server/cmd" "github.com/mattermost/mattermost-server/model" - "github.com/stretchr/testify/require" ) func TestCreateUserWithTeam(t *testing.T) { @@ -81,30 +80,3 @@ func TestMakeUserActiveAndInactive(t *testing.T) { // activate the inactive user cmd.CheckCommand(t, "user", "activate", th.BasicUser.Email) } - -func TestChangeUserEmail(t *testing.T) { - th := api.Setup().InitBasic() - defer th.TearDown() - - newEmail := model.NewId() + "@mattermost-test.com" - - cmd.CheckCommand(t, "user", "email", th.BasicUser.Username, newEmail) - if result := <-th.App.Srv.Store.User().GetByEmail(th.BasicUser.Email); result.Err == nil { - t.Fatal("should've updated to the new email") - } - if result := <-th.App.Srv.Store.User().GetByEmail(newEmail); result.Err != nil { - t.Fatal() - } else { - user := result.Data.(*model.User) - if user.Email != newEmail { - t.Fatal("should've updated to the new email") - } - } - - // should fail because using an invalid email - require.Error(t, cmd.RunCommand(t, "user", "email", th.BasicUser.Username, "wrong$email.com")) - - // should fail because user not found - require.Error(t, cmd.RunCommand(t, "user", "email", "invalidUser", newEmail)) - -} |