From 8d946ce921248a5633664ed8368241cd1b469748 Mon Sep 17 00:00:00 2001 From: Aster Fialla Date: Tue, 24 Feb 2026 16:58:39 -0500 Subject: [PATCH] put console.error in commands --- src/commands.js | 1 + tests/commands.test.js | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/commands.js b/src/commands.js index 853a13b..dd41528 100644 --- a/src/commands.js +++ b/src/commands.js @@ -111,6 +111,7 @@ cmds.importCommand = async function(message, args) { } // If just one error was returned. else { + console.error(error); return await message.reply(error.message); } } diff --git a/tests/commands.test.js b/tests/commands.test.js index 9c38f59..9537944 100644 --- a/tests/commands.test.js +++ b/tests/commands.test.js @@ -24,9 +24,6 @@ jest.mock('../src/helpers/importHelper.js', () => { } } }) -jest.mock('console', () => { - return {error: jest.fn()} -}) import {messageHelper} from "../src/helpers/messageHelper.js"; @@ -197,15 +194,18 @@ describe('commands', () => { expect(message.reply).toHaveBeenCalledWith(expected); }) - test('if pluralKitImport returns one error, reply with error', async () => { + test('if pluralKitImport returns one error, reply with error and log it', async () => { // Arrange importHelper.pluralKitImport = jest.fn().mockImplementation(() => { throw new Error('error'); }); + jest.spyOn(global.console, 'error').mockImplementation(() => {}) // Act await commands.importCommand(message, args); expect(message.reply).toHaveBeenCalledTimes(1); expect(message.reply).toHaveBeenCalledWith('error'); + expect(console.error).toHaveBeenCalledTimes(1); + expect(console.error).toHaveBeenCalledWith(new Error('error')); }) })