From a02254f5d5cdd8e67bae1167df13282074652198 Mon Sep 17 00:00:00 2001
From: Veronica Berglyd Olsen <1619840+vkbo@users.noreply.github.com>
Date: Sun, 26 Oct 2025 00:12:37 +0200
Subject: [PATCH] Make minor improvements to the code and fix test on MacOS

---
 novelwriter/extensions/modified.py  |  7 +++----
 tests/test_base/test_base_shared.py | 14 ++++++++++----
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/novelwriter/extensions/modified.py b/novelwriter/extensions/modified.py
index b802a4d9..ee1be5e7 100644
--- a/novelwriter/extensions/modified.py
+++ b/novelwriter/extensions/modified.py
@@ -210,12 +210,11 @@ class NPushButton(QPushButton):
         icon: str | None = None, color: str | None = None
     ) -> None:
         super().__init__(parent=parent)
-        self.setText(text)
-        self.setIconSize(iconSize)
         self._icon = icon
         self._color = color
-        if icon:
-            self.refreshIcon()
+        self.setText(text)
+        self.setIconSize(iconSize)
+        self.refreshIcon()
 
     def refreshIcon(self) -> None:
         """Reload the theme icon."""
diff --git a/tests/test_base/test_base_shared.py b/tests/test_base/test_base_shared.py
index a020a694..0a37b624 100644
--- a/tests/test_base/test_base_shared.py
+++ b/tests/test_base/test_base_shared.py
@@ -20,6 +20,8 @@ along with this program. If not, see <https://www.gnu.org/licenses/>.
 """  # noqa
 from __future__ import annotations
 
+import sys
+
 from unittest.mock import MagicMock
 
 import pytest
@@ -218,7 +220,8 @@ def testBaseSharedData_GuiAlert():
     # Alert: Info
     alert.setAlertType(_GuiAlert.INFO, False)
     assert hasattr(alert, "_btnOk")
-    assert alert.windowTitle() == "Information"
+    if sys.platform != "darwin":  # Not set on MacOS
+        assert alert.windowTitle() == "Information"
     alert._btnOk.click()
     assert alert.finalState is True
     alert._state = False
@@ -226,7 +229,8 @@ def testBaseSharedData_GuiAlert():
     # Alert: Warning
     alert.setAlertType(_GuiAlert.WARN, False)
     assert hasattr(alert, "_btnOk")
-    assert alert.windowTitle() == "Warning"
+    if sys.platform != "darwin":  # Not set on MacOS
+        assert alert.windowTitle() == "Warning"
     alert._btnOk.click()
     assert alert.finalState is True
     alert._state = False
@@ -234,7 +238,8 @@ def testBaseSharedData_GuiAlert():
     # Alert: Error
     alert.setAlertType(_GuiAlert.ERROR, False)
     assert hasattr(alert, "_btnOk")
-    assert alert.windowTitle() == "Error"
+    if sys.platform != "darwin":  # Not set on MacOS
+        assert alert.windowTitle() == "Error"
     alert._btnOk.click()
     assert alert.finalState is True
     alert._state = False
@@ -243,7 +248,8 @@ def testBaseSharedData_GuiAlert():
     alert.setAlertType(_GuiAlert.ASK, True)
     assert hasattr(alert, "_btnYes")
     assert hasattr(alert, "_btnNo")
-    assert alert.windowTitle() == "Question"
+    if sys.platform != "darwin":  # Not set on MacOS
+        assert alert.windowTitle() == "Question"
     alert._btnYes.click()
     assert alert.finalState is True
     alert._btnNo.click()