mmintel/mb-grid-solutions.com
1
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases 1 Wiki Activity

fix: ensure smtp metadata logger doesn't crash in tests
Some checks failed
🚀 Build & Deploy / 🔍 Prepare (push) Successful in 4s
Details
🚀 Build & Deploy / 🧪 QA (push) Successful in 2m13s
Details
🚀 Build & Deploy / 🏗️ Build (push) Successful in 14m2s
Details
🚀 Build & Deploy / 🚀 Deploy (push) Successful in 11s
Details
🚀 Build & Deploy / 🧪 Post-Deploy Verification (push) Successful in 2m51s
Details
🚀 Build & Deploy / 🔔 Notify (push) Successful in 2s
Details
Nightly QA / call-qa-workflow (push) Failing after 40s
Details

Browse Source
This commit is contained in:
Marc Mintel
2026-04-15 00:05:45 +02:00
parent 7109b3c32e
commit 8427d348dd
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/api/contact/route.ts
View File

@@ -123,7 +123,7 @@ export async function POST(req: Request) {
subject: `Kontaktanfrage von ${name}`, subject: `Kontaktanfrage von ${name}`,
html: notificationHtml, html: notificationHtml,
}); });
logger.info("Notification email sent successfully", { messageId: info.messageId }); logger.info("Notification email sent successfully", { messageId: info?.messageId });
} catch (notifyError) { } catch (notifyError) {
logger.error("Failed to send notification email", { error: notifyError }); logger.error("Failed to send notification email", { error: notifyError });
throw notifyError; // Re-throw to be caught by the outer SMTP catch throw notifyError; // Re-throw to be caught by the outer SMTP catch
@@ -145,7 +145,7 @@ export async function POST(req: Request) {
subject: `Ihre Kontaktanfrage bei ${clientName}`, subject: `Ihre Kontaktanfrage bei ${clientName}`,
html: confirmationHtml, html: confirmationHtml,
}); });
logger.info("Confirmation email sent successfully", { messageId: info.messageId }); logger.info("Confirmation email sent successfully", { messageId: info?.messageId });
} catch (confirmError) { } catch (confirmError) {
logger.warn( logger.warn(
"Failed to send confirmation email, but notification was sent", "Failed to send confirmation email, but notification was sent",

2
tests/api-contact.test.ts
View File

@@ -8,7 +8,7 @@ const { mockCreate, mockSendEmail } = vi.hoisted(() => ({
// Mock Nodemailer // Mock Nodemailer
const { mockSendMail } = vi.hoisted(() => ({ const { mockSendMail } = vi.hoisted(() => ({
mockSendMail: vi.fn(), mockSendMail: vi.fn().mockResolvedValue({ messageId: "mock-message-id" }),
})); }));
vi.mock("nodemailer", () => ({ vi.mock("nodemailer", () => ({