Improved test coverage in 5 places for production code

CHANGES.rst

@@ -7,6 +7,7 @@ Changelog
 Minor changes:
 
 - add ``__all__`` variable to each modules in ``icalendar`` package
+- Improve test coverage.
 
 Breaking changes:
 
@@ -20,6 +21,7 @@ Bug fixes:
 
 - Fix link to stable release of tox in documentation.
 - Fix a bad bytes replace in unescape_char.
+- Handle ``ValueError`` in ``vBinary.from_ical``.
 
 6.0.0a0 (2024-07-03)
 --------------------

src/icalendar/prop.py

@@ -97,7 +97,7 @@ def to_ical(self):
     def from_ical(ical):
         try:
             return base64.b64decode(ical)
-        except UnicodeError:
+        except (ValueError, UnicodeError):
             raise ValueError('Not valid base 64 encoding.')
 
     def __eq__(self, other):

src/icalendar/tests/prop/test_unit.py

@@ -44,6 +44,9 @@ def test_prop_vDDDLists(self):
 
         dt_list = vDDDLists([datetime(2000, 1, 1), datetime(2000, 11, 11)])
         self.assertEqual(dt_list.to_ical(), b'20000101T000000,20001111T000000')
+
+        instance = vDDDLists([])
+        self.assertFalse(instance == "value")
 
     def test_prop_vDate(self):
         from icalendar.prop import vDate
@@ -110,6 +113,7 @@ def test_prop_vFrequency(self):
         self.assertRaises(ValueError, vFrequency, 'bad test')
         self.assertEqual(vFrequency('daily').to_ical(), b'DAILY')
         self.assertEqual(vFrequency('daily').from_ical('MONTHLY'), 'MONTHLY')
+        self.assertRaises(ValueError, vFrequency.from_ical, 234)
 
     def test_prop_vRecur(self):
         from icalendar.prop import vRecur

src/icalendar/tests/prop/test_vBinary.py

@@ -1,4 +1,6 @@
 """Test vBinary"""
+import pytest
+
 from icalendar import vBinary
 from icalendar.parser import Parameters
 
@@ -27,3 +29,13 @@ def test_long_data():
     txt_ical = b'YWFh' * 33
     assert (vBinary(txt).to_ical() == txt_ical)
     assert (vBinary.from_ical(txt_ical) == txt)
+
+def test_repr():
+    instance = vBinary("value")
+    assert repr(instance) == "vBinary('b'value'')"
+
+def test_from_ical():
+    with pytest.raises(ValueError, match='Not valid base 64 encoding.'):
+        vBinary.from_ical("value")
+    with pytest.raises(ValueError, match='Not valid base 64 encoding.'):
+        vBinary.from_ical("áèਮ")

src/icalendar/tests/prop/test_vCalAddress.py

@@ -17,3 +17,8 @@ def test_params():
 
 def test_from_ical():
     assert vCalAddress.from_ical(txt) == 'MAILTO:maxm@mxm.dk'
+
+
+def test_repr():
+    instance = vCalAddress("value")
+    assert repr(instance) == "vCalAddress('b'value'')"