fix wrong operation model for user (#18614) (#18615)

Signed-off-by: Nkejiang <ke.jiang@zilliz.com>
Signed-off-by: Nkejiang <ke.jiang@zilliz.com>
Co-authored-by: Nkejiang <ke.jiang@zilliz.com>

internal/metastore/db/dao/user.go

@@ -53,7 +53,7 @@ func (s *userDb) Insert(in *dbmodel.User) error {
 }
 
 func (s *userDb) MarkDeletedByUsername(tenantID string, username string) error {
-	err := s.db.Model(&dbmodel.Index{}).Where("tenant_id = ? AND username = ?", tenantID, username).Update("is_deleted", true).Error
+	err := s.db.Model(&dbmodel.User{}).Where("tenant_id = ? AND username = ?", tenantID, username).Update("is_deleted", true).Error
 	if err != nil {
 		log.Error("update credential_users is_deleted=true failed", zap.String("tenant", tenantID), zap.String("username", username), zap.Error(err))
 		return err

internal/metastore/db/dao/user_test.go

@@ -144,7 +144,7 @@ func TestUser_MarkDeletedByUsername(t *testing.T) {
 
 	// expectation
 	mock.ExpectBegin()
-	mock.ExpectExec("UPDATE `indexes` SET `is_deleted`=?,`updated_at`=? WHERE tenant_id = ? AND username = ?").
+	mock.ExpectExec("UPDATE `credential_users` SET `is_deleted`=?,`updated_at`=? WHERE tenant_id = ? AND username = ?").
 		WithArgs(true, AnyTime{}, tenantID, username).
 		WillReturnResult(sqlmock.NewResult(1, 1))
 	mock.ExpectCommit()
@@ -159,7 +159,7 @@ func TestUser_MarkDeletedByUsername_Error(t *testing.T) {
 
 	// expectation
 	mock.ExpectBegin()
-	mock.ExpectExec("UPDATE `indexes` SET `is_deleted`=?,`updated_at`=? WHERE tenant_id = ? AND username = ?").
+	mock.ExpectExec("UPDATE `credential_users` SET `is_deleted`=?,`updated_at`=? WHERE tenant_id = ? AND username = ?").
 		WithArgs(true, AnyTime{}, tenantID, username).
 		WillReturnError(errors.New("test error"))
 	mock.ExpectRollback()