Skip to content

Commit ad4058e

Browse files
committed
fix(cli): logic issues
1 parent eed9536 commit ad4058e

1 file changed

Lines changed: 8 additions & 5 deletions

File tree

cli/seed_users.go

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,22 +59,25 @@ func (c *Context) SeedUsers(ctx context.Context, userSeedRecords []UserSeedRecor
5959
group := groups[record.GetGroup()]
6060

6161
// check if user already exists
62-
user, err := c.entClient.User.Query().Where(user.EmailEQ(record.Email)).First(ctx)
63-
if err == nil {
62+
count, err := c.entClient.User.Query().Where(user.EmailEQ(record.Email)).Count(ctx)
63+
if err != nil {
64+
return fmt.Errorf("count users %q: %w", record.Email, err)
65+
}
66+
if count > 0 {
6467
log.Printf("⚠️ User %q already exists, skipping creation", record.Email)
6568
continue
6669
}
6770

68-
user, err = c.entClient.User.Create().
71+
newUser, err := c.entClient.User.Create().
6972
SetEmail(record.GetEmail()).
7073
SetName(record.GetEmail()). // Use email as name for seeded users
7174
SetGroup(group).
7275
Save(ctx)
7376
if err != nil {
74-
return err
77+
return fmt.Errorf("create user %q: %w", record.Email, err)
7578
}
7679

77-
log.Printf("✅ User %q (Group %q, %d) is created", user.Email, group.Name, group.ID)
80+
log.Printf("✅ User %q (Group %q, %d) is created", newUser.Email, group.Name, group.ID)
7881
}
7982

8083
return nil

0 commit comments

Comments
 (0)