caronc / apprise

Apprise - Push Notifications that work with just about every platform!
https://hub.docker.com/r/caronc/apprise
BSD 2-Clause "Simplified" License
10.9k stars 384 forks source link

Test `test_apprise_trans_add` fails with python3.12 #1102

Closed christian-heusel closed 2 months ago

christian-heusel commented 2 months ago

:mega: Notification Service(s) Impacted

none

:lady_beetle: Describe the bug

When building apprise with python 3.12 im getting a test failure. This rebuild was done as part of the Arch Linux python3.12 rebuild.

:bulb: Screenshots and Logs

=================================== FAILURES ===================================
____________________________ test_apprise_trans_add ____________________________

    @pytest.mark.skipif(
        'gettext' not in sys.modules, reason="Requires gettext")
    def test_apprise_trans_add():
        """
        API: Apprise() Gettext add

        """

        # This throws internally but we handle it gracefully
        al = AppriseLocale.AppriseLocale()
        with environ('LANGUAGE', 'LC_ALL', 'LC_CTYPE', 'LANG'):
            # English is the default/fallback type
>           assert al.add('en') is True
E           AssertionError: assert False is True
E            +  where False = <bound method AppriseLocale.add of <apprise.AppriseLocale.AppriseLocale object at 0x76c64173e1e0>>('en')
E            +    where <bound method AppriseLocale.add of <apprise.AppriseLocale.AppriseLocale object at 0x76c64173e1e0>> = <apprise.AppriseLocale.AppriseLocale object at 0x76c64173e1e0>.add

test/test_apprise_translations.py:212: AssertionError

:computer: Your System Details:

:crystal_ball: Additional context Add any other context about the problem here.


cc @polyzen

caronc commented 2 months ago

Issue is resolved; I'll try to do another official build soon