Fix misnamed class attribute

Author: samuelhwilliams

requirements/dev.txt

@@ -12,6 +12,8 @@ babel==2.14.0
     # via
     #   -r docs.txt
     #   sphinx
+beautifulsoup4==4.12.3
+    # via -r tests.txt
 blinker==1.8.1
     # via
     #   -r tests.txt
@@ -101,6 +103,8 @@ jinja2==3.1.4
     #   -r typing.txt
     #   flask
     #   sphinx
+lxml==5.2.2
+    # via -r tests.txt
 markupsafe==2.1.5
     # via
     #   -r docs.txt
@@ -170,6 +174,10 @@ snowballstemmer==2.2.0
     # via
     #   -r docs.txt
     #   sphinx
+soupsieve==2.5
+    # via
+    #   -r tests.txt
+    #   beautifulsoup4
 sphinx==7.1.2
     # via
     #   -r docs.txt
@@ -216,10 +224,16 @@ tomli==2.0.1
     #   tox
 tox==4.15.0
     # via -r dev.in
+types-beautifulsoup4==4.12.0.20240511
+    # via -r typing.txt
 types-docutils==0.21.0.20240423
     # via
     #   -r typing.txt
     #   types-pygments
+types-html5lib==1.1.11.20240228
+    # via
+    #   -r typing.txt
+    #   types-beautifulsoup4
 types-pygments==2.18.0.20240506
     # via -r typing.txt
 types-setuptools==69.5.0.20240423

requirements/tests.in

@@ -1,3 +1,5 @@
 pytest
 flask-sqlalchemy
 pygments
+beautifulsoup4
+lxml

requirements/tests.txt

@@ -4,6 +4,8 @@
 #
 #    pip-compile tests.in
 #
+beautifulsoup4==4.12.3
+    # via -r tests.in
 blinker==1.8.1
     # via flask
 click==8.1.7
@@ -24,6 +26,8 @@ itsdangerous==2.2.0
     # via flask
 jinja2==3.1.4
     # via flask
+lxml==5.2.2
+    # via -r tests.in
 markupsafe==2.1.5
     # via
     #   jinja2
@@ -36,6 +40,8 @@ pygments==2.18.0
     # via -r tests.in
 pytest==8.2.1
     # via -r tests.in
+soupsieve==2.5
+    # via beautifulsoup4
 sqlalchemy==2.0.29
     # via flask-sqlalchemy
 tomli==2.0.1

requirements/typing.in

@@ -3,3 +3,4 @@ pyright
 pytest
 types-pygments
 flask-sqlalchemy
+types-beautifulsoup4

requirements/typing.txt

@@ -48,8 +48,12 @@ tomli==2.0.1
     # via
     #   mypy
     #   pytest
+types-beautifulsoup4==4.12.0.20240511
+    # via -r typing.in
 types-docutils==0.21.0.20240423
     # via types-pygments
+types-html5lib==1.1.11.20240228
+    # via types-beautifulsoup4
 types-pygments==2.18.0.20240506
     # via -r typing.in
 types-setuptools==69.5.0.20240423

src/flask_debugtoolbar/panels/__init__.py

@@ -18,7 +18,7 @@ class DebugPanel:
     has_content = False
 
     # If the client is able to activate/de-activate the panel
-    user_enable = False
+    user_activate = False
 
     # We'll maintain a local context instance so we can expose our template
     # context variables to panels which need them:
@@ -62,8 +62,9 @@ def render(self, template_name: str, context: dict[str, t.Any]) -> str:
         template = self.jinja_env.get_template(template_name)
         return template.render(**context)
 
-    def dom_id(self) -> str:
-        return f"flDebug{self.name.replace(' ', '')}Panel"
+    @classmethod
+    def dom_id(cls) -> str:
+        return f"flDebug{cls.name.replace(' ', '')}Panel"
 
     def nav_title(self) -> str:
         """Title showing in toolbar"""

tests/test_toolbar.py

@@ -1,7 +1,9 @@
 from __future__ import annotations
 
+from bs4 import BeautifulSoup
 from flask import Flask
 from flask.testing import FlaskClient
+from werkzeug.utils import import_string
 
 
 def load_app(name: str) -> FlaskClient:
@@ -15,3 +17,26 @@ def test_basic_app() -> None:
     index = app.get("/")
     assert index.status_code == 200
     assert b'<div id="flDebug"' in index.data
+
+
+def test_debug_switch_included_for_user_activated_panels() -> None:
+    checked_panels = set()
+
+    app = load_app("basic_app")
+    index = app.get("/")
+
+    soup = BeautifulSoup(index.text, "html.parser")
+
+    for panel in app.application.config["DEBUG_TB_PANELS"]:
+        panel_cls = import_string(panel)
+        panel_id = panel_cls.dom_id()
+        panel_element = soup.select_one(f"#{panel_id}")
+
+        assert panel_element
+        assert (
+            bool(panel_element.select_one(".flDebugSwitch")) is panel_cls.user_activate
+        ), f"Panel {panel_id} is incorrectly showing (or not showing) a debug switch"
+
+        checked_panels.add(panel_id)
+
+    assert len(checked_panels) == len(app.application.config["DEBUG_TB_PANELS"])