Skip to content

test!: Path ////path causing failures #185

@faretek1

Description

@faretek1

The path ////path is causing issues with most of the test fixtures.

Image

https://github.com/gruns/furl/blob/f53defbf075e6f31c5b910aefb4221abe5b937cf/tests/test_furl.py#L1627C28-L1631C29

This is caused by the output of f.url == '//////path' instead of '////path'. This may be due to a change in the urllib library, but I am not sure.

The output of calling urllib.parse.unparse here:
https://github.com/gruns/furl/blob/f53defbf075e6f31c5b910aefb4221abe5b937cf/furl/furl.py#L1826C9-L1832C11

actually generates the expected number of slashes:

Image

However, because of this code, an extra 2 slashes are prepended, causing the output of f.url to be invalid.

Note

This issue does not appear on python 3.8, but appears on all other tests, so this could be a problem caused by a change in the standard library, in which case, other functionality may need reviewing

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions