Revert missing tests

Author: ppizarror

pygame_menu/_scrollarea.py

@@ -252,7 +252,7 @@ def __init__(
         self._border_color = border_color
         self._border_width = border_width
         self._bg_surface = None
-        self._decorator = Decorator(self)
+        self._decorator = Decorator(self)  # type: ignore
         self._scrollbar_positions = tuple(unique_scrolls)  # Ensure unique
         self._translate = (0, 0)
         self._world = world

pygame_menu/examples/other/dynamic_widget_update.py

@@ -104,7 +104,7 @@ def __init__(self) -> None:
 
         # Update the widgets based on selected value from selector get_value
         # returns selected item tuple and index, so [0][1] means the second object
-        # from ('The first', 1) tuple
+        # from ("The first", 1) tuple
         self._update_from_selection(int(self.selector_widget.get_value()[0][1]))
 
     def animate_quit_button(

pygame_menu/themes.py

@@ -709,24 +709,24 @@ def validate(self) -> Theme:
         self.widget_url_color = self._format_color_opacity(self.widget_url_color)
 
         # List to tuple
-        self.scrollarea_outer_margin = self._vec_to_tuple(
+        self.scrollarea_outer_margin = self._vec_to_tuple(  # type: ignore
             self.scrollarea_outer_margin, 2, NumberInstance
         )
-        self.title_offset = self._vec_to_tuple(self.title_offset, 2, NumberInstance)
-        self.widget_background_inflate = self._vec_to_tuple(
+        self.title_offset = self._vec_to_tuple(self.title_offset, 2, NumberInstance)  # type: ignore
+        self.widget_background_inflate = self._vec_to_tuple(  # type: ignore
             self.widget_background_inflate, 2, int
         )
-        self.widget_border_inflate = self._vec_to_tuple(
+        self.widget_border_inflate = self._vec_to_tuple(  # type: ignore
             self.widget_border_inflate, 2, int
         )
-        self.widget_box_arrow_margin = self._vec_to_tuple(
+        self.widget_box_arrow_margin = self._vec_to_tuple(  # type: ignore
             self.widget_box_arrow_margin, 3, int
         )
-        self.widget_box_inflate = self._vec_to_tuple(self.widget_box_inflate, 2, int)
-        self.widget_box_margin = self._vec_to_tuple(
+        self.widget_box_inflate = self._vec_to_tuple(self.widget_box_inflate, 2, int)  # type: ignore
+        self.widget_box_margin = self._vec_to_tuple(  # type: ignore
             self.widget_box_margin, 2, NumberInstance
         )
-        self.widget_margin = self._vec_to_tuple(self.widget_margin, 2, NumberInstance)
+        self.widget_margin = self._vec_to_tuple(self.widget_margin, 2, NumberInstance)  # type: ignore
         if isinstance(self.widget_padding, VectorInstance):
             self.widget_padding = self._vec_to_tuple(self.widget_padding)
             assert 2 <= len(self.widget_padding) <= 4, (
@@ -739,7 +739,7 @@ def validate(self) -> Theme:
                 assert p >= 0, "all padding elements must be equal or greater than zero"
         else:
             assert self.widget_padding >= 0, "padding cannot be a negative number"
-        self.widget_offset = self._vec_to_tuple(self.widget_offset, 2, NumberInstance)
+        self.widget_offset = self._vec_to_tuple(self.widget_offset, 2, NumberInstance)  # type: ignore
 
         # Check sizes
         assert self.border_width >= 0, "border width must be equal or greater than zero"

test/_utils.py

@@ -385,7 +385,6 @@ def mouse_click(
             evtype, {"button": button, "pos": (x, y), "rel": rel, "test": testmode}
         )
         if update_mouse:
-            # print('set mouse position', (x, y))
             pygame.mouse.set_pos((x, y))
         if inlist:
             event_obj = [event_obj]

test/conftest.py

@@ -12,6 +12,7 @@
 
 @pytest.fixture(autouse=True)
 def disable_gc():
+    """Disable garbage collection."""
     gc.disable()
     yield
     gc.enable()

test/test_base.py

@@ -117,7 +117,7 @@ def test_repr_class_id():
     """Class ID repr flag returns Class<id> format."""
     obj = Base("id2")
     obj._class_id__repr__ = True
-    assert repr(obj) == 'Base<"id2">'
+    assert repr(obj) == "Base<\"id2\">"
 
 
 def test_repr_conflict_raises():
@@ -165,4 +165,4 @@ def test_verbose_type_error():
 def test_class_id_format():
     """get_class_id returns Class<id> format."""
     obj = Base("id")
-    assert obj.get_class_id() == 'Base<"id">'
+    assert obj.get_class_id() == "Base<\"id\">"

test/test_baseimage.py

@@ -186,6 +186,7 @@ def test_modes(mode):
 def test_modes_invalid():
     """Test invalid drawing mode."""
     with pytest.raises(AssertionError):
+        # Attempt to create an invalid drawing mode
         pygame_menu.BaseImage(
             pygame_menu.baseimage.IMAGE_EXAMPLE_GRAY_LINES, drawing_mode=-1
         )
@@ -362,11 +363,8 @@ def test_transform():
 
     # Image channels
     image.pick_channels(("r", "g", "b"))
-
     assert image.get_at((10, 10)) == (56, 56, 56, 255)
-
     image.set_at((10, 10), (0, 0, 0))
-    # assert image.get_at((10, 10)) == (0, 0, 0, 255)
 
 
 @pytest.mark.parametrize(

test/test_controls.py

@@ -31,6 +31,7 @@ def toggle_button(menu):
     state = {"value": False}
 
     def toggle():
+        """Toggle state."""
         state["value"] = not state["value"]
 
     btn = menu.add.button("toggle", toggle)
@@ -150,6 +151,7 @@ def test_key_apply_affects_future_widgets(menu):
     state = {"value": False}
 
     def toggle():
+        """Toggle state."""
         state["value"] = not state["value"]
 
     btn = menu.add.button("x", toggle)
@@ -164,6 +166,7 @@ def test_custom_controller_apply(toggle_button):
     count = {"n": 0}
 
     def custom_apply(event, _):
+        """Custom apply method overrides default behavior."""
         count["n"] += 1
         return event.key == pygame.K_a
 

test/test_decorator.py

@@ -31,8 +31,25 @@ def test_time_draw():
     for _ in range(10000):
         deco.add_pixel(1, 2, (0, 0, 0))
 
+    # (100) no cache, 0.214
+    # (100) with cache, 0.646
+    # (250) no cache, 0.467
+    # (250) with cache, 0.594
+    # (300) no cache, 0.581
+    # (300) with cache, 0.606
+    # (400) no cache, 0.82
+    # (400) with cache, 0.638
+    # (500) no cache, 1.087
+    # (500) with cache, 0.601
+    # (750) no cache, 1.484
+    # (750) with cache, 0.664
+    # (1.000) no cache, 2.228
+    # (1.000) with cache, 0.615
+    # (10.000) no cache, 20.430
+    # (10.000) with cache, 0.599
+    print("Total decorations", deco._total_decor(), "Cache", deco.cache)
     total_tests = 10
-    total = 0
+    total = 0  # Total time
     for i in range(total_tests):
         t = timeit.timeit(lambda: widg.draw(surface), number=1000)
         total += t
@@ -97,7 +114,6 @@ def test_copy():
     """Test decorator copy."""
     widg = NoneWidget()
     deco = widg.get_decorator()
-
     with pytest.raises(Exception):
         copy.copy(deco)
     with pytest.raises(Exception):
@@ -141,6 +157,7 @@ def test_enable_disable():
     test = [False]
 
     def fun(surf, obj):
+        """Test callable decoration."""
         test[0] = True
         assert isinstance(surf, pygame.Surface)
         assert isinstance(obj, Button)
@@ -277,6 +294,7 @@ def test_general():
     test = [False]
 
     def fun(_, __):
+        """Test callable decoration."""
         test[0] = True
 
     deco.add_callable(fun)
@@ -287,6 +305,7 @@ def fun(_, __):
     test[0] = False
 
     def fun_noargs():
+        """No args function."""
         test[0] = True
 
     deco.add_callable(fun_noargs, pass_args=False)

test/test_examples.py

@@ -53,9 +53,11 @@ def test_example_multi_input():
     multi_input.update_menu_sound(("sound", None), True)
     multi_input.update_menu_sound(("sound", None), False)
 
+    # Test methods within submenus
     settings = multi_input.main_menu.get_submenus()[0]
     settings.get_widget("store").apply()
 
+    # Check range slider has event
     rslider = settings.get_widget("range_slider")
     assert rslider._onchange is not None
     assert rslider.get_value() == 50
@@ -238,7 +240,6 @@ def test_example_other_scrollbar():
     """Test scrollbar example."""
     pygame.event.post(PygameEventUtils.keydown(pygame.K_v, inlist=False))
     pygame.event.post(PygameEventUtils.keydown(pygame.K_h, inlist=False))
-
     scrollbar.main(test=True)
     scrollbar.h_changed(1)
     scrollbar.v_changed(1)