test: add alert module tests for different welcome_message configurations (#3867)

In this way, all options for `welcome_message` are tested.

CHANGELOG.md

@@ -34,6 +34,7 @@ Thanks to: @dathbe.
   - Simplify DOM setup by removing unnecessary Promise/async patterns
   - Avoid potential port conflicts by using port 3001 for translator unit tests
   - Improve test reliability and maintainability
+- [tests] add alert module tests for different welcome_message configurations (#3867)
 
 ### Updated
 

tests/configs/modules/alert/welcome_false.js

@@ -0,0 +1,18 @@
+let config = {
+	address: "0.0.0.0",
+	ipWhitelist: [],
+	modules: [
+		{
+			module: "alert",
+			config: {
+				display_time: 1000000,
+				welcome_message: false
+			}
+		}
+	]
+};
+
+/*************** DO NOT EDIT THE LINE BELOW ***************/
+if (typeof module !== "undefined") {
+	module.exports = config;
+}

tests/configs/modules/alert/welcome_string.js

@@ -0,0 +1,18 @@
+let config = {
+	address: "0.0.0.0",
+	ipWhitelist: [],
+	modules: [
+		{
+			module: "alert",
+			config: {
+				display_time: 1000000,
+				welcome_message: "Custom welcome message!"
+			}
+		}
+	]
+};
+
+/*************** DO NOT EDIT THE LINE BELOW ***************/
+if (typeof module !== "undefined") {
+	module.exports = config;
+}

tests/e2e/modules/alert_spec.js

@@ -1,17 +1,52 @@
 const helpers = require("../helpers/global-setup");
 
 describe("Alert module", () => {
-	beforeAll(async () => {
-		await helpers.startApplication("tests/configs/modules/alert/default.js");
-		await helpers.getDocument();
-	});
 	afterAll(async () => {
 		await helpers.stopApplication();
 	});
 
-	it("should show the welcome message", async () => {
+	describe("with welcome_message set to false", () => {
+		beforeAll(async () => {
+			await helpers.startApplication("tests/configs/modules/alert/welcome_false.js");
+			await helpers.getDocument();
+		});
+
+		it("should not show any welcome message", async () => {
+			// Wait a bit to ensure no message appears
+			await new Promise((resolve) => setTimeout(resolve, 1000));
+
+			// Check that no alert/notification elements are present
+			const alertElements = document.querySelectorAll(".ns-box .ns-box-inner .light.bright.small");
+			expect(alertElements).toHaveLength(0);
+		});
+	});
+
+	describe("with welcome_message set to true", () => {
+		beforeAll(async () => {
+			await helpers.startApplication("tests/configs/modules/alert/welcome_true.js");
+			await helpers.getDocument();
+
+			// Wait for the application to initialize
+			await new Promise((resolve) => setTimeout(resolve, 1000));
+		});
+
+		it("should show the translated welcome message", async () => {
 			const elem = await helpers.waitForElement(".ns-box .ns-box-inner .light.bright.small");
 			expect(elem).not.toBeNull();
 			expect(elem.textContent).toContain("Welcome, start was successful!");
 		});
 	});
+
+	describe("with welcome_message set to custom string", () => {
+		beforeAll(async () => {
+			await helpers.startApplication("tests/configs/modules/alert/welcome_string.js");
+			await helpers.getDocument();
+		});
+
+		it("should show the custom welcome message", async () => {
+			const elem = await helpers.waitForElement(".ns-box .ns-box-inner .light.bright.small");
+			expect(elem).not.toBeNull();
+			expect(elem.textContent).toContain("Custom welcome message!");
+		});
+	});
+});