libs/m5ui/bar.py: Add warning prompts.

Signed-off-by: lbuque <[email protected]>

Author: lbuque

docs/en/m5ui/bar.rst

@@ -55,24 +55,6 @@ M5Bar
 .. autoclass:: m5ui.bar.M5Bar
     :members:
 
-    .. py:method:: set_value(value, anim_enable)
-
-        Set the current value of the bar.
-
-        :param int value: The value to set.
-        :param bool anim_enable: Whether to enable animation when changing the value.
-        :return: None
-
-        UiFlow2 Code Block:
-
-            |set_value.png|
-
-        MicroPython Code Block:
-
-            .. code-block:: python
-
-                bar.set_value(75, True)
-
     .. py:method:: get_value()
 
         Get the current value of the bar.
@@ -90,24 +72,6 @@ M5Bar
 
                 current_value = bar.get_value()
 
-    .. py:method:: set_range(min_value, max_value)
-
-        Set the value range of the bar.
-
-        :param int min_value: The minimum value.
-        :param int max_value: The maximum value.
-        :return: None
-
-        UiFlow2 Code Block:
-
-            |set_range.png|
-
-        MicroPython Code Block:
-
-            .. code-block:: python
-
-                bar.set_range(0, 200)
-
     .. py:method:: get_min_value()
 
         Get the minimum value of the bar range.

m5stack/libs/m5ui/bar.py

@@ -4,6 +4,7 @@
 
 from .base import M5Base
 import lvgl as lv
+import warnings
 
 
 class M5Bar(lv.bar):
@@ -53,16 +54,65 @@ def __init__(
 
         self.set_pos(x, y)
         self.set_size(w, h)
-        self.set_range(min_value, max_value)
+        super().set_range(min_value, max_value)
         self.set_bg_color(
             bg_c, 51, lv.PART.MAIN | lv.STATE.DEFAULT
         )  # default opacity is 51 (20% opacity)
         self.set_bg_color(color, lv.OPA.COVER, lv.PART.INDICATOR | lv.STATE.DEFAULT)
-        self.set_value(value, True)
+        super().set_value(value, True)
 
         if is_show_value:
             self.add_event_cb(self._draw_cb, lv.EVENT.DRAW_MAIN_END, None)
 
+    def set_value(self, value: int, anim: bool = False) -> None:
+        """Set the current value of the bar.
+
+        :param int value: The value to set.
+        :param bool anim_enable: Whether to enable animation when changing the value.
+        :return: None
+
+        UiFlow2 Code Block:
+
+            |set_value.png|
+
+        MicroPython Code Block:
+
+            .. code-block:: python
+
+                bar.set_value(75, True)
+        """
+        if not isinstance(value, int):
+            raise ValueError("Value must be an integer.")
+        if value < self.get_min_value():
+            warnings.warn(f"Value is less than min_value, setting to {self.get_min_value()}.")
+            value = self.get_min_value()
+        if value > self.get_max_value():
+            warnings.warn(f"Value is greater than max_value, setting to {self.get_max_value()}.")
+            value = self.get_max_value()
+        super().set_value(value, anim)
+
+    def set_range(self, min_value: int, max_value: int) -> None:
+        """Set the value range of the bar.
+
+        :param int min_value: The minimum value.
+        :param int max_value: The maximum value.
+        :return: None
+
+        UiFlow2 Code Block:
+
+            |set_range.png|
+
+        MicroPython Code Block:
+
+            .. code-block:: python
+
+                bar.set_range(0, 200)
+        """
+        if not isinstance(min_value, int) or not isinstance(max_value, int):
+            raise ValueError("min_value and max_value must be integers.")
+        super().set_range(min_value, max_value)
+        self.set_value(self.get_value(), False)
+
     def _draw_cb(self, event_struct):
         label_dsc = lv.draw_label_dsc_t()
         label_dsc.init()

tests/m5ui/test_bar.py

@@ -0,0 +1,37 @@
+# SPDX-FileCopyrightText: 2025 M5Stack Technology CO LTD
+#
+# SPDX-License-Identifier: MIT
+
+import lvgl as lv
+import sys
+import time
+
+sys.path.append("../../m5stack/libs")
+import m5ui
+import unittest
+
+
+class Test(unittest.TestCase):
+    def __init__(self) -> None:
+        super().__init__()
+        m5ui.init()
+        page0 = m5ui.M5Page()
+        self.bar0 = m5ui.M5Bar(value=25, parent=page0)
+        page0.screen_load()
+
+    def test_set_range(self):
+        self.bar0.set_range(40, 50)
+        self.assertEqual(self.bar0.get_value(), 40)
+
+    def test_set_value(self):
+        self.bar0.set_value(60)
+        self.assertEqual(self.bar0.get_value(), 50)
+        self.bar0.set_value(30)
+        self.assertEqual(self.bar0.get_value(), 40)
+
+    def tearDown(self):
+        time.sleep(3)
+
+
+if __name__ == "__main__":
+    unittest.main()