Skip to content

Commit

Permalink
Merge pull request #3072 from pared/2896_import_url
Browse files Browse the repository at this point in the history
import url: test: migrate to dir helpers
  • Loading branch information
efiop authored Jan 9, 2020
2 parents 277d28c + ad14506 commit 238b4e8
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions tests/func/test_import_url.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import filecmp
import os
from uuid import uuid4

Expand Down Expand Up @@ -90,15 +89,16 @@ def test(self):


@pytest.mark.parametrize("dname", [".", "dir", "dir/subdir"])
def test_import_url_to_dir(dname, repo_dir, dvc_repo):
src = repo_dir.DATA
def test_import_url_to_dir(dname, tmp_dir, dvc):
tmp_dir.gen({"data_dir": {"file": "file content"}})
src = os.path.join("data_dir", "file")

makedirs(dname, exist_ok=True)

stage = dvc_repo.imp_url(src, dname)
stage = dvc.imp_url(src, dname)

dst = os.path.join(dname, os.path.basename(src))
dst = tmp_dir / dname / "file"

assert stage.outs[0].fspath == os.path.abspath(dst)
assert stage.outs[0].path_info == dst
assert os.path.isdir(dname)
assert filecmp.cmp(repo_dir.DATA, dst, shallow=False)
assert dst.read_text() == "file content"

0 comments on commit 238b4e8

Please sign in to comment.