eds: revert LowLimit/HighLimit parse error to silent pass, remove warning test

Author: friederschueler

canopen/objectdictionary/eds.py

@@ -306,7 +306,7 @@ def build_variable(
             else:
                 var.min = int(min_string, 0)
         except ValueError:
-            logger.warning("Failed to parse LowLimit for %s: %r", name, min_string)
+            pass
     if eds.has_option(section, "HighLimit"):
         try:
             max_string = eds.get(section, "HighLimit")
@@ -315,7 +315,7 @@ def build_variable(
             else:
                 var.max = int(max_string, 0)
         except ValueError:
-            logger.warning("Failed to parse HighLimit for %s: %r", name, max_string)
+            pass
     if eds.has_option(section, "DefaultValue"):
         try:
             var.default_raw = eds.get(section, "DefaultValue")

test/test_eds.py

@@ -154,23 +154,6 @@ def test_record_with_limits(self):
             self.assertEqual(var.min, -1, f"min mismatch at 0x{index:04X}")
             self.assertEqual(var.max, 0, f"max mismatch at 0x{index:04X}")
 
-    def test_invalid_limit_logs_warning(self):
-        import io
-        import logging
-
-        with open(SAMPLE_EDS) as f:
-            content = f.read()
-        invalid_eds = content.replace(
-            "LowLimit=0x02\nPDOMapping=0\n\n[3030]",
-            "LowLimit=INVALID\nPDOMapping=0\n\n[3030]",
-        )
-        with io.StringIO(invalid_eds) as buf:
-            buf.name = "mock.eds"
-            with self.assertLogs("canopen.objectdictionary.eds", level=logging.WARNING) as cm:
-                od = canopen.import_od(buf)
-        self.assertIsNone(od[0x3021].min)
-        self.assertTrue(any("LowLimit" in msg for msg in cm.output))
-
     def test_signed_int_from_hex(self):
         for data_type, test_cases in self.test_data.items():
             for test_case in test_cases: