Andrey/uptime-kuma
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[autofix.ci] apply automated fixes

Browse Source
This commit is contained in:
autofix-ci[bot] 2026-01-06 02:06:33 +00:00 committed by GitHub
parent 942fd5021c
commit 67cd608e27
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
server/model/monitor.js
View File

@ -1023,7 +1023,7 @@ class Monitor extends BeanModel {
if (isDatabaseError) {
const errorMessage = e.message || `${e.code || "Unknown error"}`;
log.error("monitor", `Database connection error detected in monitor: ${errorMessage}`);
// Try to send notification using cached notifications
try {
const { Notification } = require("../notification");

2
server/server.js
View File

@ -2029,7 +2029,7 @@ let unexpectedErrorHandler = async (error, promise) => {
if (isDatabaseError) {
const errorMessage = error.message || `${error.code || "Unknown error"}`;
log.error("server", `Database connection error detected: ${errorMessage}`);
// Try to send notification using cached notifications
try {
await Notification.sendDatabaseDownNotification(errorMessage);

18
test/backend-test/test-database-down-notification.js
View File

@ -10,10 +10,10 @@ describe("Database Down Notification", () => {
before(async () => {
// Initialize data directory first (required before connecting)
Database.initDataDir({});
// Ensure database is connected (this copies template DB which has basic tables)
await Database.connect(true); // testMode = true
// Ensure notification table exists with required columns
const hasNotificationTable = await R.hasTable("notification");
if (!hasNotificationTable) {
@ -65,10 +65,10 @@ describe("Database Down Notification", () => {
test("refreshCache() loads notifications into cache", async () => {
await Notification.refreshCache();
assert.ok(Notification.notificationCache.length > 0, "Cache should contain notifications");
assert.ok(Notification.cacheLastRefresh > 0, "Cache refresh time should be set");
// Verify test notification is in cache
const cached = Notification.notificationCache.find(n => n.id === testNotification.id);
assert.ok(cached, "Test notification should be in cache");
@ -97,7 +97,7 @@ describe("Database Down Notification", () => {
try {
await Notification.sendDatabaseDownNotification("Test database error: ECONNREFUSED");
// Should have been called for each notification in cache
assert.ok(sendCallCount > 0, "send() should have been called");
assert.strictEqual(Notification.databaseDownNotificationSent, true, "Flag should be set");
@ -109,7 +109,7 @@ describe("Database Down Notification", () => {
test("sendDatabaseDownNotification() only sends once per database down event", async () => {
Notification.resetDatabaseDownFlag();
// Mock the send method
let sendCallCount = 0;
const originalSend = Notification.send;
@ -122,7 +122,7 @@ describe("Database Down Notification", () => {
// First call
await Notification.sendDatabaseDownNotification("Test error 1");
const firstCallCount = sendCallCount;
// Second call should not send again
await Notification.sendDatabaseDownNotification("Test error 2");
assert.strictEqual(sendCallCount, firstCallCount, "Should not send again");
@ -139,7 +139,7 @@ describe("Database Down Notification", () => {
// Should not throw
await Notification.sendDatabaseDownNotification("Test error");
// Flag should remain false since cache is empty
assert.strictEqual(Notification.databaseDownNotificationSent, false);
});
@ -160,7 +160,7 @@ describe("Database Down Notification", () => {
try {
// Should not throw
await Notification.refreshCache();
// Cache should remain unchanged (not cleared)
assert.ok(Array.isArray(Notification.notificationCache), "Cache should still be an array");
} finally {