hotfix: add logs when it cannot save aoc key (#247)

* hotfix: add logs when it cannot save aoc key

* test: silence logger in testing

src/slash-commands/server-settings/set-aoc-settings.test.ts

@@ -17,7 +17,7 @@ describe('Set aoc key', () => {
   });
 
   it('should reply with error if it cannot set the key', async () => {
-    mockSetAocSettings.mockRejectedValueOnce(new Error('Synthetic Error'));
+    mockSetAocSettings.mockRejectedValueOnce(new Error('Synthetic Error save aoc settings'));
     mockChatInputInteraction.options.getString.mockImplementation((name) => {
       switch (name) {
         case 'key': {
@@ -37,14 +37,12 @@ describe('Set aoc key', () => {
 
     expect(mockSetAocSettings).toHaveBeenCalledOnce();
     expect(mockChatInputInteraction.reply).toHaveBeenCalledOnce();
-    expect(mockChatInputInteraction.reply).toHaveBeenCalledWith('Cannot set this AOC key. Please try again');
+    expect(mockChatInputInteraction.reply).toHaveBeenCalledWith('Cannot set this AOC key. Please try again. Error: Error: Synthetic Error save aoc settings');
   });
 
   it('Should be able to set AOC key and reply', async () => {
     mockSetAocSettings.mockResolvedValueOnce({
       guildId: faker.string.numeric(),
-      reminderChannel: null,
-      autobumpThreads: [],
       aocLeaderboardId: faker.string.numeric(),
     });
     mockChatInputInteraction.options.getString.mockImplementation((name) => {

src/slash-commands/server-settings/set-aoc-settings.ts

@@ -18,8 +18,8 @@ export const execute: SlashCommandHandler = async (interaction) => {
 
   const op = await Result.safe(setAocSettings(guildId, key, leaderboardId));
   if (op.isErr()) {
-    logger.info(`[set-aoc-key]: ${interaction.member!.user.username} failed to set AOC Key.`);
-    await interaction.reply('Cannot set this AOC key. Please try again');
+    logger.info(`[set-aoc-key]: ${interaction.member!.user.username} failed to set AOC Key. Error: ${op.unwrapErr()}`);
+    await interaction.reply(`Cannot set this AOC key. Please try again. Error: ${op.unwrapErr()}`);
     return;
   }
 

src/slash-commands/server-settings/utils.ts

@@ -31,7 +31,18 @@ export const setAocSettings = async (guildId: string, aocKey: string, aocLeaderb
   const db = getDbClient();
   return db.serverChannelsSettings.upsert({
     where: { guildId },
-    update: { aocKey, aocLeaderboardId },
-    create: { guildId, aocKey },
+    update: {
+      aocKey,
+      aocLeaderboardId,
+    },
+    create: {
+      guildId,
+      aocKey,
+      aocLeaderboardId,
+    },
+    select: {
+      guildId: true,
+      aocLeaderboardId: true,
+    },
   });
 };

src/utils/logger.ts

@@ -13,6 +13,7 @@ const devOptions: pino.LoggerOptions = {
       colorize: true,
     },
   },
+  enabled: !process.env.VITEST,
 };
 
 const prodOptions: pino.LoggerOptions = {