Added tests for deleting the last user
the build failed
Details
the build failed
Details
This commit is contained in:
parent
152a399567
commit
537aebc873
|
@ -1,7 +1,5 @@
|
||||||
package models
|
package models
|
||||||
|
|
||||||
import "fmt"
|
|
||||||
|
|
||||||
// DeleteUserByID deletes a user by its ID
|
// DeleteUserByID deletes a user by its ID
|
||||||
func DeleteUserByID(id int64) error {
|
func DeleteUserByID(id int64) error {
|
||||||
// Check if the id is 0
|
// Check if the id is 0
|
||||||
|
|
|
@ -19,6 +19,9 @@ func TestCreateUser(t *testing.T) {
|
||||||
}
|
}
|
||||||
|
|
||||||
// Delete every preexisting user to have a fresh start
|
// Delete every preexisting user to have a fresh start
|
||||||
|
_, err := x.Where("1 = 1").Delete(&User{})
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
allusers, err := ListUsers("")
|
allusers, err := ListUsers("")
|
||||||
assert.NoError(t, err)
|
assert.NoError(t, err)
|
||||||
for _, user := range allusers {
|
for _, user := range allusers {
|
||||||
|
@ -31,6 +34,10 @@ func TestCreateUser(t *testing.T) {
|
||||||
createdUser, err := CreateUser(dummyuser)
|
createdUser, err := CreateUser(dummyuser)
|
||||||
assert.NoError(t, err)
|
assert.NoError(t, err)
|
||||||
|
|
||||||
|
// Create a second new user
|
||||||
|
createdUser2, err := CreateUser(User{Username:dummyuser.Username + "2", Email:dummyuser.Email + "m", Password:dummyuser.Password})
|
||||||
|
assert.NoError(t, err)
|
||||||
|
|
||||||
// Check if it fails to create the same user again
|
// Check if it fails to create the same user again
|
||||||
_, err = CreateUser(dummyuser)
|
_, err = CreateUser(dummyuser)
|
||||||
assert.Error(t, err)
|
assert.Error(t, err)
|
||||||
|
@ -129,6 +136,11 @@ func TestCreateUser(t *testing.T) {
|
||||||
assert.Error(t, err)
|
assert.Error(t, err)
|
||||||
assert.True(t, IsErrIDCannotBeZero(err))
|
assert.True(t, IsErrIDCannotBeZero(err))
|
||||||
|
|
||||||
|
// Try delete the last user (Should fail)
|
||||||
|
err = DeleteUserByID(createdUser2.ID)
|
||||||
|
assert.Error(t, err)
|
||||||
|
assert.True(t, IsErrCannotDeleteLastUser(err))
|
||||||
|
|
||||||
// Log some user action
|
// Log some user action
|
||||||
err = logAction("test something", User{ID: 1}, 1)
|
err = logAction("test something", User{ID: 1}, 1)
|
||||||
assert.NoError(t, err)
|
assert.NoError(t, err)
|
||||||
|
|
Loading…
Reference in New Issue