mgdelacroix/craban
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Add game member deletion when user is deleted

Browse source
This commit is contained in:
Miguel de la Cruz 2021-09-13 23:21:31 +02:00
parent e0da55c63d
commit 104068e172
2 changed files with 22 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
server/services/store/user.go
View file

@ -110,8 +110,24 @@ func (us *UserStore) List() ([]*model.User, error) {
} }
func (us *UserStore) DeleteByUsername(username string) error { func (us *UserStore) DeleteByUsername(username string) error {
_, err := us.Q().Delete("users"). user, err := us.GetByUsername(username)
if err != nil {
return fmt.Errorf("cannot get user for deletion: %w", err)
}
_, err = us.Q().Delete("users").
Where(sq.Eq{"username": username}). Where(sq.Eq{"username": username}).
Exec() Exec()
return err if err != nil {
return fmt.Errorf("cannot delete user: %w", err)
}
_, err = us.Q().Delete("gamemembers").
Where(sq.Eq{"user_id": user.ID}).
Exec()
if err != nil {
return fmt.Errorf("cannot delete game members when deleting user: %w", err)
}
return nil
} }

11
server/services/store/user_test.go
View file

@ -56,12 +56,9 @@ func TestDeleteUsername(t *testing.T) {
require.NoError(t, err) require.NoError(t, err)
require.NotZero(t, gameMember) require.NotZero(t, gameMember)
// ToDo: implement member deletion err = th.store.User().DeleteByUsername(user.Username)
/* require.NoError(t, err)
err = th.store.User().DeleteByUsername(user.Username)
require.NoError(t, err)
_, err = th.store.Game().GetMember(game.ID, user.ID) _, err = th.store.Game().GetMember(game.ID, user.ID)
require.Error(t, err) require.Error(t, err)
*/
} }