style: fix pylint warnings to pass CI checks

Author: gio24ita

src/UniparkGUI.py

@@ -25,6 +25,7 @@
 
 
 class UniParkApp(tk.Tk):
+    # pylint: disable=too-many-instance-attributes
     def __init__(self):
         super().__init__()
 
@@ -89,7 +90,6 @@ def get_time_phase(self):
             return "DEFLUSSO POMERIDIANO", 25  # Le auto iniziano a uscire
         elif 19 <= hour < 22:
             return "SVUOTAMENTO SERALE", 10  # Parcheggi quasi deserti
-        else:
             return "FASCIA NOTTURNA", 2  # Nessuna attività
 
     def set_theme_colors(self):
@@ -225,6 +225,7 @@ def setup_main_layout(self):
                 0, 0, anchor="nw", image=self.bg_img, tags="static_map"
             )
         except Exception as e:
+            # pylint: disable=broad-exception-caught
             self.canvas.create_text(
                 400,
                 300,
@@ -620,7 +621,7 @@ def on_close(self):
             self.running = False
             self.destroy()
             # os._exit(0) uccide immediatamente tutti i thread orfani in background
-            os._exit(0)
+            os._exit(0)  # pylint: disable=protected-access
 
 
 if __name__ == "__main__":

tests/test_UniParkGUI.py

@@ -1,5 +1,6 @@
 # pylint: disable=redefined-outer-name
 from unittest.mock import MagicMock, patch
+
 import pytest
 
 # Importiamo la classe GUI
@@ -21,11 +22,11 @@ def mock_app():
         # Qui abbiamo aggiunto maxsize, minsize, Canvas e PhotoImage!
         with patch("tkinter.Tk.geometry"), patch("tkinter.Tk.title"), patch(
             "tkinter.Tk.configure"
-        ), patch("tkinter.Tk.resizable"), patch(
-            "tkinter.Tk.maxsize"          
+        ), patch("tkinter.Tk.resizable"), patch("tkinter.Tk.maxsize"), patch(
+            "tkinter.Tk.minsize"
+        ), patch(
+            "tkinter.Tk.protocol"
         ), patch(
-            "tkinter.Tk.minsize"          
-        ), patch("tkinter.Tk.protocol"), patch(
             "tkinter.Tk.destroy"
         ), patch(
             "tkinter.Tk.mainloop"
@@ -89,7 +90,7 @@ def mock_app():
                 app.running = False
                 app.system = mock_system
                 app.zones = mock_system.zones
-                app.canvas = MagicMock() # Mock the canvas to avoid draw errors
+                app.canvas = MagicMock()  # Mock the canvas to avoid draw errors
 
                 # Manually populate fake widgets
                 app.zone_widgets = {
@@ -99,7 +100,7 @@ def mock_app():
                         "lbl_details": MagicMock(),
                         "lbl_queue": MagicMock(),
                         "btn_park": MagicMock(),
-                        "btn_unpark": MagicMock()
+                        "btn_unpark": MagicMock(),
                     }
                 }
 
@@ -118,7 +119,7 @@ def test_gui_initialization(mock_app):
 def test_gui_user_park(mock_app):
     """Test clicking the PARK button."""
     app, mock_zone = mock_app
-    
+
     # NOTA: Ora si chiama user_action (non più manual_action)
     app.user_action(mock_zone, "park")
 
@@ -130,7 +131,7 @@ def test_gui_user_park(mock_app):
 def test_gui_user_unpark(mock_app):
     """Test clicking the UNPARK button."""
     app, mock_zone = mock_app
-    
+
     # NOTA: Ora si chiama user_action
     app.user_action(mock_zone, "unpark")
 
@@ -157,4 +158,4 @@ def test_gui_update_widgets(mock_app):
     app.update_widgets_once()
 
     # Verifica che la UI sia stata chiamata per aggiornarsi
-    app.zone_widgets["TestZone"]["progress"].config.assert_called()
+    app.zone_widgets["TestZone"]["progress"].config.assert_called()