From 84fb67174018d3580daed04a0bb669621d6bed00 Mon Sep 17 00:00:00 2001 From: firestar5683 <168790843+firestar5683@users.noreply.github.com> Date: Thu, 22 Oct 2026 23:38:16 -0500 Subject: [PATCH] Type Ford CAN and developer panel test fixtures --- .../ui/tests/test_ford_release_alpha.py | 21 ++++++++++--------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/openpilot/starpilot/ui/tests/test_ford_release_alpha.py b/openpilot/starpilot/ui/tests/test_ford_release_alpha.py index 67a57db6ba..1fb3170598 100644 --- a/openpilot/starpilot/ui/tests/test_ford_release_alpha.py +++ b/openpilot/starpilot/ui/tests/test_ford_release_alpha.py @@ -1,7 +1,6 @@ """Both actual Developer panels expose only qualified Ford release opt-in.""" from tempfile import TemporaryDirectory from types import SimpleNamespace -from typing import Any from unittest.mock import patch import pytest from opendbc.car import gen_empty_fingerprint @@ -18,17 +17,19 @@ def panel_for(module, cp, params): ui = SimpleNamespace(CP=cp.as_reader(), params=params, is_release=True, engaged=False, has_longitudinal_control=False, is_offroad=lambda: True, update_params=lambda: None) layout = large.DeveloperLayout if module is large else compact.DeveloperLayoutMici - panel: Any = layout.__new__(layout) - panel._alpha_long_toggle = Toggle() - panel._alpha_long_toggle.title, panel._alpha_long_toggle.description = 'Alpha Long', 'Existing alpha confirmation' - panel._long_maneuver_toggle = Toggle() - panel._lat_maneuver_toggle = Toggle() + panel = layout.__new__(layout) + alpha = Toggle() + alpha.title, alpha.description = 'Alpha Long', 'Existing alpha confirmation' + attributes: list[tuple[str, object]] = [('_alpha_long_toggle', alpha), ('_long_maneuver_toggle', Toggle()), + ('_lat_maneuver_toggle', Toggle())] if module is large: - panel._params, panel._is_release = params, True - for name in ('_joystick_toggle', '_adb_toggle', '_ssh_toggle', '_ui_debug_toggle', '_cellular_disabled_toggle'): - setattr(panel, name, Toggle()) + attributes.extend((('_params', params), ('_is_release', True))) + attributes.extend((name, Toggle()) for name in + ('_joystick_toggle', '_adb_toggle', '_ssh_toggle', '_ui_debug_toggle', '_cellular_disabled_toggle')) else: - panel._refresh_toggles = (('AlphaLongitudinalEnabled', panel._alpha_long_toggle),) + attributes.append(('_refresh_toggles', (('AlphaLongitudinalEnabled', alpha),))) + for name, value in attributes: + setattr(panel, name, value) return panel, ui