From 0e1fa84332daee57d510bd5f046cfcbafbf9728f Mon Sep 17 00:00:00 2001
From: einhornimmond <info@einhornimmond.de>
Date: Sat, 1 Feb 2025 11:46:37 +0100
Subject: [PATCH] use spy instead of mock which calls original function

---
 backend/src/emails/sendEmailVariants.test.ts | 30 ++++++++------------
 1 file changed, 12 insertions(+), 18 deletions(-)

diff --git a/backend/src/emails/sendEmailVariants.test.ts b/backend/src/emails/sendEmailVariants.test.ts
index 76705deb9..ee36c3e9a 100644
--- a/backend/src/emails/sendEmailVariants.test.ts
+++ b/backend/src/emails/sendEmailVariants.test.ts
@@ -11,7 +11,7 @@ import { logger, i18n as localization } from '@test/testSetup'
 
 import { CONFIG } from '@/config'
 
-import { sendEmailTranslated } from './sendEmailTranslated'
+import * as sendEmailTranslatedApi  from './sendEmailTranslated'
 import {
   sendAddedContributionMessageEmail,
   sendAccountActivationEmail,
@@ -50,13 +50,7 @@ afterAll(async () => {
   await con.close()
 })
 
-jest.mock('./sendEmailTranslated', () => {
-  const originalModule = jest.requireActual('./sendEmailTranslated')
-  return {
-    __esModule: true,
-    sendEmailTranslated: jest.fn((a) => originalModule.sendEmailTranslated(a)),
-  }
-})
+const sendEmailTranslatedSpy = jest.spyOn(sendEmailTranslatedApi, 'sendEmailTranslated')
 
 describe('sendEmailVariants', () => {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -77,7 +71,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -131,7 +125,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -182,7 +176,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -235,7 +229,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -292,7 +286,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -348,7 +342,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -403,7 +397,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -457,7 +451,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -513,7 +507,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },
@@ -572,7 +566,7 @@ describe('sendEmailVariants', () => {
 
     describe('calls "sendEmailTranslated"', () => {
       it('with expected parameters', () => {
-        expect(sendEmailTranslated).toBeCalledWith({
+        expect(sendEmailTranslatedSpy).toBeCalledWith({
           receiver: {
             to: 'Peter Lustig <peter@lustig.de>',
           },