fail on delete of non-existing user

app/rest/api/admin.go

@@ -91,7 +91,7 @@ func (a *admin) deleteMeRequestCtrl(w http.ResponseWriter, r *http.Request) {
 	log.Printf("[INFO] delete all user comments by request for %s, site %s", claims.User.ID, claims.SiteID)
 
 	if err := a.dataService.DeleteUser(claims.SiteID, claims.User.ID); err != nil {
-		rest.SendErrorJSON(w, r, http.StatusInternalServerError, err, "can't delete user")
+		rest.SendErrorJSON(w, r, http.StatusBadRequest, err, "can't delete user")
 		return
 	}
 	a.cache.Flush(claims.SiteID, claims.User.ID)

app/rest/api/admin_test.go

@@ -456,10 +456,23 @@ func TestAdmin_DeleteMeRequestFailed(t *testing.T) {
 
 	token, err := srv.Authenticator.JWTService.Token(&claims)
 	assert.Nil(t, err)
-	req, err = http.NewRequest(http.MethodDelete, fmt.Sprintf("%s/api/v1/admin/deleteme?token=%s", ts.URL, token), nil)
+	req, err = http.NewRequest(http.MethodGet, fmt.Sprintf("%s/api/v1/admin/deleteme?token=%s", ts.URL, token), nil)
 	assert.Nil(t, err)
 	req.SetBasicAuth("dev", "bad-password")
 	resp, err = client.Do(req)
 	assert.Nil(t, err)
 	assert.Equal(t, 401, resp.StatusCode)
+
+	// try bad user
+	badClaims := claims
+	badClaims.User.ID = "no-such-id"
+	token, err = srv.Authenticator.JWTService.Token(&badClaims)
+	assert.Nil(t, err)
+	req, err = http.NewRequest(http.MethodGet, fmt.Sprintf("%s/api/v1/admin/deleteme?token=%s", ts.URL, token), nil)
+	assert.Nil(t, err)
+	req.SetBasicAuth("dev", "password")
+	resp, err = client.Do(req)
+	assert.Nil(t, err)
+	assert.Equal(t, 400, resp.StatusCode, resp.Status)
+
 }

app/store/engine/bolt_admin.go

@@ -138,13 +138,21 @@ func (b *BoltDB) DeleteUser(siteID string, userID string) error {
 		usersBkt := tx.Bucket([]byte(userBucketName))
 		if usersBkt != nil {
 			if e := usersBkt.DeleteBucket([]byte(userID)); e != nil {
-				return errors.Wrapf(err, "failed to delete user bucker for %s", userID)
+				return errors.Wrapf(err, "failed to delete user bucket for %s", userID)
 			}
 		}
 		return nil
 	})
 
-	return nil
+	if err != nil {
+		return errors.Wrap(err, "can't delete user meta")
+	}
+
+	if len(comments) == 0 {
+		return errors.Errorf("unknown user %s", userID)
+	}
+
+	return err
 }
 
 // SetBlock blocks/unblocks user for given site