Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test_tcp_connection_with_forwarding failure with dbus 1.14.4 #135

Open
SFrijters opened this issue Oct 24, 2022 · 2 comments
Open

test_tcp_connection_with_forwarding failure with dbus 1.14.4 #135

SFrijters opened this issue Oct 24, 2022 · 2 comments

Comments

@SFrijters
Copy link

=================================== FAILURES ===================================
_____________________ test_tcp_connection_with_forwarding ______________________

event_loop = <_UnixSelectorEventLoop running=False closed=False debug=False>

    @pytest.mark.asyncio
    async def test_tcp_connection_with_forwarding(event_loop):
        closables = []
        host = '127.0.0.1'
        port = '55556'
    
        addr_info = parse_address(os.environ.get('DBUS_SESSION_BUS_ADDRESS'))
        assert addr_info
>       assert 'abstract' in addr_info[0][1]
E       AssertionError: assert 'abstract' in {'guid': '80575dbdd24507fd614f7dfa63569e61', 'path': '/tmp/dbus-nXrJp9b9fo'}

test/test_tcp_address.py:18: AssertionError

It seems that dbus 1.14.4 made a backwards incompatible change:

Excerpt from https://gitlab.freedesktop.org/dbus/dbus/-/blob/dbus-1.14.4/NEWS :

Behaviour changes:

• On Linux, dbus-daemon and other uses of DBusServer now create a
  path-based Unix socket, unix:path=..., when asked to listen on a
  unix:tmpdir=... address. This makes unix:tmpdir=... equivalent to
  unix:dir=... on all platforms.
  Previous versions would have created an abstract socket, unix:abstract=...,
  in this situation.

I don't know if the test needs fixing or anything else needs to be changed, just reporting what I found so far.

@dvzrv
Copy link

dvzrv commented Apr 8, 2023

Hi! I also just stumbled upon this issue with the ongoing Python 3.11 rebuild on Arch Linux.
Is there a fix besides disabling this test?

archlinux-github pushed a commit to archlinux/svntogit-community that referenced this issue Apr 8, 2023
Remove unnecessary quotes and curly braces.
Disable broken test: altdesktop/python-dbus-next#135
Switch to PEP517.

git-svn-id: file:///srv/repos/svn-community/svn@1442916 9fca08f4-af9d-4005-b8df-a31f2cc04f65
archlinux-github pushed a commit to archlinux/svntogit-community that referenced this issue Apr 8, 2023
Remove unnecessary quotes and curly braces.
Disable broken test: altdesktop/python-dbus-next#135
Switch to PEP517.

git-svn-id: file:///srv/repos/svn-community/svn@1442916 9fca08f4-af9d-4005-b8df-a31f2cc04f65
@elParaguayo
Copy link
Contributor

@dvzrv I haven't seen the maintainer around for quite a few months now so skipping that test (which I can see you've done) is probably the best approach for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants