fix(parser): handle bytes input in escape_char using to_unicode

CHANGES.rst

@@ -42,6 +42,7 @@ New features
 Bug fixes
 ~~~~~~~~~
 
+- Fixed :func:`~icalendar.parser.string.escape_char` to accept :class:`bytes` input by converting it to :class:`str` with :func:`~icalendar.parser_tools.to_unicode` before applying replacements. :issue:`1226`
 - ...
 
 Documentation

src/icalendar/parser/string.py

@@ -2,10 +2,10 @@
 
 import re
 
-from icalendar.parser_tools import DEFAULT_ENCODING
+from icalendar.parser_tools import DEFAULT_ENCODING, to_unicode
 
 
-def escape_char(text: str | bytes) -> str | bytes:
+def escape_char(text: str | bytes) -> str:
     r"""Format value according to iCalendar TEXT escaping rules.
 
     Escapes special characters in text values according to :rfc:`5545#section-3.3.11`
@@ -29,7 +29,7 @@ def escape_char(text: str | bytes) -> str | bytes:
         6. ``"\n"`` -> ``r"\n"`` (transform a newline character to a literal, or raw,
            newline character)
     """
-    assert isinstance(text, (str, bytes))
+    text = to_unicode(text)
     # NOTE: ORDER MATTERS!
     return (
         text.replace(r"\N", "\n")

src/icalendar/tests/test_parsing.py

@@ -9,7 +9,7 @@
 from icalendar.cal.calendar import Calendar
 from icalendar.cal.component_factory import ComponentFactory
 from icalendar.cal.event import Event
-from icalendar.parser import Contentline, Parameters, unescape_char
+from icalendar.parser import Contentline, Parameters, unescape_char, escape_char
 
 
 @pytest.mark.parametrize(
@@ -280,3 +280,14 @@ def test_create_a_component():
     my_component_class = factory.get_component_class("My-Component")
     assert my_component_class.name == "MY-COMPONENT"
     assert my_component_class.__name__ == "MyComponent"
+
+
+def test_escape_char_bytes():
+    """Test that escape_char accepts bytes input and returns str."""
+    assert escape_char(b"hello,world") == "hello\\,world"
+    assert escape_char(b"hello;world") == "hello\\;world"
+    assert escape_char(b"hello\\world") == "hello\\\\world"
+    assert escape_char(b"hello\nworld") == "hello\\nworld"
+    assert escape_char(b"hello\r\nworld") == "hello\\nworld"
+    assert escape_char(b"hello\\Nworld") == "hello\\nworld"
+    assert escape_char("hello,world") == "hello\\,world"