Skip to content

Commit

Permalink
fix Windows tests for LocalIO.url2transport_path
Browse files Browse the repository at this point in the history
  • Loading branch information
christian-monch committed May 14, 2024
1 parent 3074c2f commit 585b48d
Showing 1 changed file with 20 additions and 5 deletions.
25 changes: 20 additions & 5 deletions datalad_next/patches/tests/test_add_method_url2transport_path.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,16 @@
http_remote_io_url2transport_path,
)

from datalad.utils import on_windows
from datalad.tests.utils_pytest import skip_if

def test_local_io_url2transport_path():

@skip_if(on_windows)
def test_local_io_url2transport_path_posix():

for url, transport_path in (
('/a/b/c', '/a/b/c'),
('/C:/a/b/c', '/C:/a/b/c'),
('/C:a/b/c', '/C:a/b/c'),
('C:/a/b/c', 'C:/a/b/c'),
):
assert local_io_url2transport_path(
Expand All @@ -23,7 +26,8 @@ def test_local_io_url2transport_path():
) == Path(transport_path)


def test_local_io_url2transport_path_windows_warning(monkeypatch):
@skip_if(not on_windows)
def test_local_io_url2transport_path_windows(monkeypatch):
monkeypatch.setattr(
'datalad_next.patches.add_method_url2transport_path.on_windows',
True,
Expand All @@ -33,8 +37,19 @@ def test_local_io_url2transport_path_windows_warning(monkeypatch):
'datalad_next.patches.add_method_url2transport_path.lgr.warning',
lambda x: warnings.append(x),
)
local_io_url2transport_path(None, PurePosixPath('/C:a/b/c'))
assert len(warnings) == 1

for url, transport_path in (
('/a/b/c', '/a/b/c'),
('C:/a/b/c', 'C:/a/b/c'),
('C:a/b/c', 'C:a/b/c'),
('/C:a/b/c', 'C:a/b/c'),
('/C:/a/b/c', 'C:/a/b/c'),
):
assert local_io_url2transport_path(
None,
PurePosixPath(url)
) == Path(transport_path)
assert len(warnings) > 0


def test_http_remote_io_url2transport_path():
Expand Down

0 comments on commit 585b48d

Please sign in to comment.