From dbf6128747ac5b4b1c46d0bb41e821c1ee6af571 Mon Sep 17 00:00:00 2001
From: Alice Gaudon <alice@gaudon.pro>
Date: Tue, 26 Jan 2021 17:56:57 +0100
Subject: [PATCH] Fix AddNameToUsersMigration not calling callbacks

---
 src/auth/migrations/AddNameToUsersMigration.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/auth/migrations/AddNameToUsersMigration.ts b/src/auth/migrations/AddNameToUsersMigration.ts
index a9eadc6..cf513ab 100644
--- a/src/auth/migrations/AddNameToUsersMigration.ts
+++ b/src/auth/migrations/AddNameToUsersMigration.ts
@@ -15,10 +15,12 @@ export default class AddNameToUsersMigration extends Migration {
 
         // Give every user a random name
         const users = await User.select().get(this.getCurrentConnection());
+        const callbacks: (() => Promise<void>)[] = [];
         await Promise.all(users.map(user => {
             user.name = nanoid();
-            return user.save(this.getCurrentConnection());
+            return user.save(this.getCurrentConnection(), c => callbacks.push(c));
         }));
+        await Promise.all(callbacks);
 
         await this.query(`ALTER TABLE users
             ADD CONSTRAINT UNIQUE (name)`);