adding sensitive labels for text areas

Author: jpchauvel

src/edit_python_pe/main.py

@@ -60,14 +60,10 @@ def on_mount(self) -> None:
         self.city_input = Input(placeholder="Ciudad")
         self.homepage_input = Input(placeholder="Página personal")
 
-        self.who_area = TextArea(text="¿Quién eres y a qué te dedicas?")
-        self.python_area = TextArea(text="¿Cómo programas en Python?")
-        self.contributions_area = TextArea(
-            text="¿Tienes algún aporte a la comunidad de Python?"
-        )
-        self.availability_area = TextArea(
-            text="¿Estás disponible para hacer mentoring, consultorías, charlas?"
-        )
+        self.who_area = TextArea()
+        self.python_area = TextArea()
+        self.contributions_area = TextArea()
+        self.availability_area = TextArea()
 
         self.social_container = Vertical()
         self.alias_container = Vertical()
@@ -95,9 +91,27 @@ def on_mount(self) -> None:
 
         self.form_container.mount(self.city_input)
         self.form_container.mount(self.homepage_input)
+        self.form_container.mount(
+            Static("¿Quién eres y a qué te dedicas?", classes="subheader")
+        )
         self.form_container.mount(self.who_area)
+        self.form_container.mount(
+            Static("¿Cómo programas en Python?", classes="subheader")
+        )
         self.form_container.mount(self.python_area)
+        self.form_container.mount(
+            Static(
+                "¿Tienes algún aporte a la comunidad de Python?",
+                classes="subheader",
+            )
+        )
         self.form_container.mount(self.contributions_area)
+        self.form_container.mount(
+            Static(
+                "¿Estás disponible para hacer mentoring, consultorías, charlas?",
+                classes="subheader",
+            )
+        )
         self.form_container.mount(self.availability_area)
 
         self.form_button_bar = Horizontal(
@@ -131,14 +145,10 @@ def clear_form(self) -> None:
         self.email_input.value = ""
         self.city_input.value = ""
         self.homepage_input.value = ""
-        self.who_area.text = "¿Quién eres y a qué te dedicas?"
-        self.python_area.text = "¿Cómo programas en Python?"
-        self.contributions_area.text = (
-            "¿Tienes algún aporte a la comunidad de Python?"
-        )
-        self.availability_area.text = (
-            "¿Estás disponible para hacer mentoring, consultorías, charlas?"
-        )
+        self.who_area.text = ""
+        self.python_area.text = ""
+        self.contributions_area.text = ""
+        self.availability_area.text = ""
 
         for soc in self.social_entries:
             soc.remove()

tests/test_member_app.py

@@ -8,7 +8,7 @@
 sys.path.insert(
     0, os.path.abspath(os.path.join(os.path.dirname(__file__), "../src"))
 )
-from edit_python_pe.main import MemberApp
+from edit_python_pe.main import MemberApp, fork_repo, get_repo, main
 
 
 class TestMemberApp(unittest.TestCase):
@@ -27,7 +27,7 @@ def setUp(self):
         self.app.social_index = 0
         self.app.alias_index = 0
 
-        # Mock UI elements and REPO_PATH
+        # Mock UI elements
         # Use simple stub classes for input widgets and text areas
         class StubInput:
             def __init__(self):
@@ -45,7 +45,6 @@ def __init__(self):
         self.app.python_area = StubTextArea()
         self.app.contributions_area = StubTextArea()
         self.app.availability_area = StubTextArea()
-        self.app.REPO_PATH = "test_repo"
 
         # Patch remove method for entries to avoid Textual lifecycle errors
         # Use stub classes for entries with .remove() method
@@ -116,19 +115,15 @@ class DummyEvent:
         self.assertEqual(self.app.email_input.value, "")
         self.assertEqual(self.app.city_input.value, "")
         self.assertEqual(self.app.homepage_input.value, "")
-        self.assertEqual(
-            self.app.who_area.text, "¿Quién eres y a qué te dedicas?"
-        )
-        self.assertEqual(
-            self.app.python_area.text, "¿Cómo programas en Python?"
-        )
+        self.assertEqual(self.app.who_area.text, "")
+        self.assertEqual(self.app.python_area.text, "")
         self.assertEqual(
             self.app.contributions_area.text,
-            "¿Tienes algún aporte a la comunidad de Python?",
+            "",
         )
         self.assertEqual(
             self.app.availability_area.text,
-            "¿Estás disponible para hacer mentoring, consultorías, charlas?",
+            "",
         )
         self.assertEqual(len(self.app.social_entries), 0)
         self.assertEqual(len(self.app.alias_entries), 0)
@@ -158,7 +153,6 @@ def test_save_member_edit_no_pr(self):
         from unittest.mock import MagicMock, patch
 
         app = self.app
-        app.REPO_PATH = "/tmp/testrepo"
         app.current_file = "existing_member.md"
         app.token = "fake-token"
         app.forked_repo = MagicMock()
@@ -213,7 +207,6 @@ def test_save_member_edit(self):
         from unittest.mock import MagicMock, patch
 
         app = self.app
-        app.REPO_PATH = "/tmp/testrepo"
         app.current_file = "existing_member.md"
         app.token = "fake-token"
         app.forked_repo = MagicMock()
@@ -275,7 +268,6 @@ def test_save_member_new(self):
         from unittest.mock import MagicMock, patch
 
         app = self.app
-        app.REPO_PATH = "/tmp/testrepo"
         app.current_file = None
         app.token = "fake-token"
         app.forked_repo = MagicMock()
@@ -326,7 +318,6 @@ def test_save_member_error_handling(self):
         from unittest.mock import MagicMock, patch
 
         app = self.app
-        app.REPO_PATH = "/tmp/testrepo"
         app.current_file = None
         app.token = "fake-token"
         app.forked_repo = MagicMock()
@@ -462,12 +453,6 @@ def stub_add_alias_entry():
         self.assertGreaterEqual(len(self.app.social_entries), 1)
 
 
-# Test for get_repo function
-import builtins
-
-from edit_python_pe.main import get_repo
-
-
 class TestGetRepo(unittest.TestCase):
     @patch("edit_python_pe.main.getpass.getpass", return_value="valid-token")
     @patch("edit_python_pe.main.Github")
@@ -501,9 +486,6 @@ def test_get_repo_github_exception(self, mock_github, mock_getpass):
             get_repo()
 
 
-from edit_python_pe.main import fork_repo
-
-
 class TestForkRepo(unittest.TestCase):
     @patch("edit_python_pe.main.user_data_dir", return_value="/tmp/testrepo")
     @patch("edit_python_pe.main.os.path.exists", return_value=False)
@@ -533,9 +515,6 @@ def test_fork_repo_no_clone_if_exists(
         mock_forked_repo = MagicMock()
 
 
-from edit_python_pe.main import main
-
-
 class TestMainFunction(unittest.TestCase):
     @patch("edit_python_pe.main.get_repo")
     @patch("edit_python_pe.main.fork_repo")