cleanup

Author: Mike Prosser

src/nipanel/_panel_client.py

@@ -80,7 +80,7 @@ def close_panel(self, panel_id: str, reset: bool) -> None:
         self._invoke_with_retry(self._get_stub().ClosePanel, close_panel_request)
 
     def enumerate_panels(self) -> dict[str, tuple[bool, list[str]]]:
-        """Enumerate all open panels.
+        """Enumerate all available panels.
 
         Returns:
             A dictionary mapping panel IDs to a tuple containing a boolean indicating if the panel

tests/unit/test_panel_client.py

@@ -22,7 +22,7 @@ def test___open_panels___both_panels_open_and_in_memory(fake_panel_channel: grpc
     }
 
 
-def test___open_panels___close_panel_1_with_reset___panel_1_not_in_memory(
+def test___open_panels___close_panel_1_with_reset___enumerate_has_panel_2(
     fake_panel_channel: grpc.Channel,
 ) -> None:
     client = create_panel_client(fake_panel_channel)
@@ -36,7 +36,7 @@ def test___open_panels___close_panel_1_with_reset___panel_1_not_in_memory(
     }
 
 
-def test___open_panels___close_panel_1_without_reset___both_panels_in_memory(
+def test___open_panels___close_panel_1_without_reset___enumerate_has_both_panels(
     fake_panel_channel: grpc.Channel,
 ) -> None:
     client = create_panel_client(fake_panel_channel)

tests/unit/test_streamlit_panel.py

@@ -111,7 +111,7 @@ def test___first_open_panel_fails___open_panel___gets_value(
     assert panel.get_value(value_id) == string_value
 
 
-def test___unopened_panel___set_value___client_has_value(
+def test___unopened_panel___set_value___sets_value(
     fake_panel_channel: grpc.Channel,
 ) -> None:
     """Test that set_value() succeeds before the user opens the panel."""
@@ -210,7 +210,6 @@ def test___open_panel___panel_is_open_and_in_memory(
 def test___with_panel___opens_and_closes_panel(
     fake_panel_channel: grpc.Channel,
 ) -> None:
-    """Test that using the panel in a with statement opens and closes it."""
     panel = StreamlitPanel("my_panel", "path/to/script", grpc_channel=fake_panel_channel)
 
     with panel:
@@ -224,7 +223,6 @@ def test___with_panel___opens_and_closes_panel(
 def test___with_panel___set_value___gets_same_value(
     fake_panel_channel: grpc.Channel,
 ) -> None:
-    """Test that using the panel in a with statement allows setting and getting values."""
     with StreamlitPanel("my_panel", "path/to/script", grpc_channel=fake_panel_channel) as panel:
         value_id = "test_id"
         string_value = "test_value"