mirror of
https://github.com/Stone-Red-Code/dvc.git
synced 2026-11-02 07:18:25 +01:00
737 lines
21 KiB
Python
737 lines
21 KiB
Python
import filecmp
|
|
import logging
|
|
import os
|
|
import posixpath
|
|
import shutil
|
|
import time
|
|
|
|
import colorama
|
|
import pytest
|
|
from mock import call, patch
|
|
|
|
import dvc as dvc_module
|
|
from dvc.cache import Cache
|
|
from dvc.dvcfile import DVC_FILE_SUFFIX
|
|
from dvc.exceptions import (
|
|
DvcException,
|
|
OutputDuplicationError,
|
|
OverlappingOutputPathsError,
|
|
RecursiveAddingWhileUsingFilename,
|
|
StageFileCorruptedError,
|
|
)
|
|
from dvc.main import main
|
|
from dvc.output.base import OutputAlreadyTrackedError, OutputIsStageFileError
|
|
from dvc.remote import LocalRemote
|
|
from dvc.repo import Repo as DvcRepo
|
|
from dvc.stage import Stage
|
|
from dvc.system import System
|
|
from dvc.utils import LARGE_DIR_SIZE, file_md5, relpath
|
|
from dvc.utils.fs import path_isin
|
|
from dvc.utils.stage import load_stage_file
|
|
from tests.basic_env import TestDvc
|
|
from tests.utils import get_gitignore_content
|
|
|
|
|
|
def test_add(tmp_dir, dvc):
|
|
(stage,) = tmp_dir.dvc_gen({"foo": "foo"})
|
|
md5, _ = file_md5("foo")
|
|
|
|
assert stage is not None
|
|
|
|
assert isinstance(stage, Stage)
|
|
assert os.path.isfile(stage.path)
|
|
assert len(stage.outs) == 1
|
|
assert len(stage.deps) == 0
|
|
assert stage.cmd is None
|
|
assert stage.outs[0].info["md5"] == md5
|
|
assert stage.md5 == "411854a9fc55b3cebda28704e41b23b3"
|
|
|
|
assert load_stage_file("foo.dvc") == {
|
|
"md5": "411854a9fc55b3cebda28704e41b23b3",
|
|
"outs": [{"md5": "acbd18db4cc2f85cedef654fccc4a4d8", "path": "foo"}],
|
|
}
|
|
|
|
|
|
def test_add_unicode(tmp_dir, dvc):
|
|
with open("\xe1", "wb") as fd:
|
|
fd.write(b"something")
|
|
|
|
(stage,) = dvc.add("\xe1")
|
|
|
|
assert os.path.isfile(stage.path)
|
|
|
|
|
|
def test_add_unsupported_file(dvc):
|
|
with pytest.raises(DvcException):
|
|
dvc.add("unsupported://unsupported")
|
|
|
|
|
|
def test_add_directory(tmp_dir, dvc):
|
|
(stage,) = tmp_dir.dvc_gen({"dir": {"file": "file"}})
|
|
|
|
assert stage is not None
|
|
assert len(stage.deps) == 0
|
|
assert len(stage.outs) == 1
|
|
|
|
md5 = stage.outs[0].info["md5"]
|
|
|
|
dir_info = dvc.cache.local.load_dir_cache(md5)
|
|
for info in dir_info:
|
|
assert "\\" not in info["relpath"]
|
|
|
|
|
|
class TestAddDirectoryRecursive(TestDvc):
|
|
def test(self):
|
|
stages = self.dvc.add(self.DATA_DIR, recursive=True)
|
|
self.assertEqual(len(stages), 2)
|
|
|
|
|
|
class TestAddCmdDirectoryRecursive(TestDvc):
|
|
def test(self):
|
|
ret = main(["add", "--recursive", self.DATA_DIR])
|
|
self.assertEqual(ret, 0)
|
|
|
|
def test_warn_about_large_directories(self):
|
|
warning = (
|
|
"You are adding a large directory 'large-dir' recursively,"
|
|
" consider tracking it as a whole instead.\n"
|
|
"{purple}HINT:{nc} Remove the generated DVC-file and then"
|
|
" run `{cyan}dvc add large-dir{nc}`".format(
|
|
purple=colorama.Fore.MAGENTA,
|
|
cyan=colorama.Fore.CYAN,
|
|
nc=colorama.Style.RESET_ALL,
|
|
)
|
|
)
|
|
|
|
os.mkdir("large-dir")
|
|
|
|
# Create a lot of files
|
|
for iteration in range(LARGE_DIR_SIZE + 1):
|
|
path = os.path.join("large-dir", str(iteration))
|
|
with open(path, "w") as fobj:
|
|
fobj.write(path)
|
|
|
|
with self._caplog.at_level(logging.WARNING, logger="dvc"):
|
|
assert main(["add", "--recursive", "large-dir"]) == 0
|
|
assert warning in self._caplog.messages
|
|
|
|
|
|
class TestAddDirectoryWithForwardSlash(TestDvc):
|
|
def test(self):
|
|
dname = "directory/"
|
|
os.mkdir(dname)
|
|
self.create(os.path.join(dname, "file"), "file")
|
|
stages = self.dvc.add(dname)
|
|
self.assertEqual(len(stages), 1)
|
|
stage = stages[0]
|
|
self.assertTrue(stage is not None)
|
|
self.assertEqual(os.path.abspath("directory.dvc"), stage.path)
|
|
|
|
|
|
def test_add_tracked_file(tmp_dir, scm, dvc):
|
|
path = "tracked_file"
|
|
tmp_dir.scm_gen(path, "...", commit="add tracked file")
|
|
msg = f""" output '{path}' is already tracked by SCM \\(e.g. Git\\).
|
|
You can remove it from Git, then add to DVC.
|
|
To stop tracking from Git:
|
|
git rm -r --cached '{path}'
|
|
git commit -m "stop tracking {path}" """
|
|
|
|
with pytest.raises(OutputAlreadyTrackedError, match=msg):
|
|
dvc.add(path)
|
|
|
|
|
|
class TestAddDirWithExistingCache(TestDvc):
|
|
def test(self):
|
|
dname = "a"
|
|
fname = os.path.join(dname, "b")
|
|
os.mkdir(dname)
|
|
shutil.copyfile(self.FOO, fname)
|
|
|
|
stages = self.dvc.add(self.FOO)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
stages = self.dvc.add(dname)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
|
|
class TestAddModifiedDir(TestDvc):
|
|
def test(self):
|
|
stages = self.dvc.add(self.DATA_DIR)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
os.unlink(self.DATA)
|
|
|
|
time.sleep(2)
|
|
|
|
stages = self.dvc.add(self.DATA_DIR)
|
|
self.assertEqual(len(stages), 1)
|
|
self.assertTrue(stages[0] is not None)
|
|
|
|
|
|
def test_add_file_in_dir(tmp_dir, dvc):
|
|
tmp_dir.gen({"dir": {"subdir": {"subdata": "subdata content"}}})
|
|
subdir_path = os.path.join("dir", "subdir", "subdata")
|
|
|
|
(stage,) = dvc.add(subdir_path)
|
|
|
|
assert stage is not None
|
|
assert len(stage.deps) == 0
|
|
assert len(stage.outs) == 1
|
|
assert stage.relpath == subdir_path + ".dvc"
|
|
|
|
# Current dir should not be taken into account
|
|
assert stage.wdir == os.path.dirname(stage.path)
|
|
assert stage.outs[0].def_path == "subdata"
|
|
|
|
|
|
class TestAddExternalLocalFile(TestDvc):
|
|
def test(self):
|
|
dname = TestDvc.mkdtemp()
|
|
fname = os.path.join(dname, "foo")
|
|
shutil.copyfile(self.FOO, fname)
|
|
|
|
stages = self.dvc.add(fname)
|
|
self.assertEqual(len(stages), 1)
|
|
stage = stages[0]
|
|
self.assertNotEqual(stage, None)
|
|
self.assertEqual(len(stage.deps), 0)
|
|
self.assertEqual(len(stage.outs), 1)
|
|
self.assertEqual(stage.relpath, "foo.dvc")
|
|
self.assertEqual(len(os.listdir(dname)), 1)
|
|
self.assertTrue(os.path.isfile(fname))
|
|
self.assertTrue(filecmp.cmp(fname, "foo", shallow=False))
|
|
|
|
|
|
class TestAddLocalRemoteFile(TestDvc):
|
|
def test(self):
|
|
"""
|
|
Making sure that 'remote' syntax is handled properly for local outs.
|
|
"""
|
|
cwd = os.getcwd()
|
|
remote = "myremote"
|
|
|
|
ret = main(["remote", "add", remote, cwd])
|
|
self.assertEqual(ret, 0)
|
|
|
|
self.dvc = DvcRepo()
|
|
|
|
foo = f"remote://{remote}/{self.FOO}"
|
|
ret = main(["add", foo])
|
|
self.assertEqual(ret, 0)
|
|
|
|
d = load_stage_file("foo.dvc")
|
|
self.assertEqual(d["outs"][0]["path"], foo)
|
|
|
|
bar = os.path.join(cwd, self.BAR)
|
|
ret = main(["add", bar])
|
|
self.assertEqual(ret, 0)
|
|
|
|
d = load_stage_file("bar.dvc")
|
|
self.assertEqual(d["outs"][0]["path"], self.BAR)
|
|
|
|
|
|
class TestCmdAdd(TestDvc):
|
|
def test(self):
|
|
ret = main(["add", self.FOO])
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["add", "non-existing-file"])
|
|
self.assertNotEqual(ret, 0)
|
|
|
|
|
|
class TestDoubleAddUnchanged(TestDvc):
|
|
def test_file(self):
|
|
ret = main(["add", self.FOO])
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["add", self.FOO])
|
|
self.assertEqual(ret, 0)
|
|
|
|
def test_dir(self):
|
|
ret = main(["add", self.DATA_DIR])
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["add", self.DATA_DIR])
|
|
self.assertEqual(ret, 0)
|
|
|
|
|
|
def test_should_update_state_entry_for_file_after_add(mocker, dvc, tmp_dir):
|
|
file_md5_counter = mocker.spy(dvc_module.remote.local, "file_md5")
|
|
tmp_dir.gen("foo", "foo")
|
|
|
|
ret = main(["config", "cache.type", "copy"])
|
|
assert ret == 0
|
|
|
|
ret = main(["add", "foo"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 1
|
|
|
|
ret = main(["status"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 1
|
|
|
|
ret = main(["run", "--single-stage", "-d", "foo", "echo foo"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 1
|
|
|
|
os.rename("foo", "foo.back")
|
|
ret = main(["checkout"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 1
|
|
|
|
ret = main(["status"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 1
|
|
|
|
|
|
def test_should_update_state_entry_for_directory_after_add(
|
|
mocker, dvc, tmp_dir
|
|
):
|
|
file_md5_counter = mocker.spy(dvc_module.remote.local, "file_md5")
|
|
|
|
tmp_dir.gen({"data/data": "foo", "data/data_sub/sub_data": "foo"})
|
|
|
|
ret = main(["config", "cache.type", "copy"])
|
|
assert ret == 0
|
|
|
|
ret = main(["add", "data"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 3
|
|
|
|
ret = main(["status"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 3
|
|
|
|
ls = "dir" if os.name == "nt" else "ls"
|
|
ret = main(
|
|
["run", "--single-stage", "-d", "data", "{} {}".format(ls, "data")]
|
|
)
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 3
|
|
|
|
os.rename("data", "data" + ".back")
|
|
ret = main(["checkout"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 3
|
|
|
|
ret = main(["status"])
|
|
assert ret == 0
|
|
assert file_md5_counter.mock.call_count == 3
|
|
|
|
|
|
class TestAddCommit(TestDvc):
|
|
def test(self):
|
|
ret = main(["add", self.FOO, "--no-commit"])
|
|
self.assertEqual(ret, 0)
|
|
self.assertTrue(os.path.isfile(self.FOO))
|
|
self.assertFalse(os.path.exists(self.dvc.cache.local.cache_dir))
|
|
|
|
ret = main(["commit", self.FOO + ".dvc"])
|
|
self.assertEqual(ret, 0)
|
|
self.assertTrue(os.path.isfile(self.FOO))
|
|
self.assertEqual(len(os.listdir(self.dvc.cache.local.cache_dir)), 1)
|
|
|
|
|
|
def test_should_collect_dir_cache_only_once(mocker, tmp_dir, dvc):
|
|
tmp_dir.gen({"data/data": "foo"})
|
|
get_dir_checksum_counter = mocker.spy(LocalRemote, "get_dir_checksum")
|
|
ret = main(["add", "data"])
|
|
assert ret == 0
|
|
|
|
ret = main(["status"])
|
|
assert ret == 0
|
|
|
|
ret = main(["status"])
|
|
assert ret == 0
|
|
assert get_dir_checksum_counter.mock.call_count == 1
|
|
|
|
|
|
class SymlinkAddTestBase(TestDvc):
|
|
def _get_data_dir(self):
|
|
raise NotImplementedError
|
|
|
|
def _prepare_external_data(self):
|
|
data_dir = self._get_data_dir()
|
|
|
|
self.data_file_name = "data_file"
|
|
external_data_path = os.path.join(data_dir, self.data_file_name)
|
|
with open(external_data_path, "w+") as f:
|
|
f.write("data")
|
|
|
|
self.link_name = "data_link"
|
|
System.symlink(data_dir, self.link_name)
|
|
|
|
def _test(self):
|
|
self._prepare_external_data()
|
|
|
|
ret = main(["add", os.path.join(self.link_name, self.data_file_name)])
|
|
self.assertEqual(0, ret)
|
|
|
|
stage_file = self.data_file_name + DVC_FILE_SUFFIX
|
|
self.assertTrue(os.path.exists(stage_file))
|
|
|
|
d = load_stage_file(stage_file)
|
|
relative_data_path = posixpath.join(
|
|
self.link_name, self.data_file_name
|
|
)
|
|
self.assertEqual(relative_data_path, d["outs"][0]["path"])
|
|
|
|
|
|
class TestShouldAddDataFromExternalSymlink(SymlinkAddTestBase):
|
|
def _get_data_dir(self):
|
|
return self.mkdtemp()
|
|
|
|
def test(self):
|
|
self._test()
|
|
|
|
|
|
class TestShouldAddDataFromInternalSymlink(SymlinkAddTestBase):
|
|
def _get_data_dir(self):
|
|
return self.DATA_DIR
|
|
|
|
def test(self):
|
|
self._test()
|
|
|
|
|
|
class TestShouldPlaceStageInDataDirIfRepositoryBelowSymlink(TestDvc):
|
|
def test(self):
|
|
def is_symlink_true_below_dvc_root(path):
|
|
if path == os.path.dirname(self.dvc.root_dir):
|
|
return True
|
|
return False
|
|
|
|
with patch.object(
|
|
System, "is_symlink", side_effect=is_symlink_true_below_dvc_root
|
|
):
|
|
|
|
ret = main(["add", self.DATA])
|
|
self.assertEqual(0, ret)
|
|
|
|
stage_file_path_on_data_below_symlink = (
|
|
os.path.basename(self.DATA) + DVC_FILE_SUFFIX
|
|
)
|
|
self.assertFalse(
|
|
os.path.exists(stage_file_path_on_data_below_symlink)
|
|
)
|
|
|
|
stage_file_path = self.DATA + DVC_FILE_SUFFIX
|
|
self.assertTrue(os.path.exists(stage_file_path))
|
|
|
|
|
|
class TestShouldThrowProperExceptionOnCorruptedStageFile(TestDvc):
|
|
def test(self):
|
|
ret = main(["add", self.FOO])
|
|
assert 0 == ret
|
|
|
|
foo_stage = relpath(self.FOO + DVC_FILE_SUFFIX)
|
|
|
|
# corrupt stage file
|
|
with open(foo_stage, "a+") as file:
|
|
file.write("this will break yaml file structure")
|
|
|
|
self._caplog.clear()
|
|
|
|
ret = main(["add", self.BAR])
|
|
assert 1 == ret
|
|
|
|
expected_error = (
|
|
"unable to read DVC-file: {} "
|
|
"YAML file structure is corrupted".format(foo_stage)
|
|
)
|
|
|
|
assert expected_error in self._caplog.text
|
|
|
|
|
|
class TestAddFilename(TestDvc):
|
|
def test(self):
|
|
ret = main(["add", self.FOO, self.BAR, "-f", "error.dvc"])
|
|
self.assertNotEqual(0, ret)
|
|
|
|
ret = main(["add", "-R", self.DATA_DIR, "-f", "error.dvc"])
|
|
self.assertNotEqual(0, ret)
|
|
|
|
with self.assertRaises(RecursiveAddingWhileUsingFilename):
|
|
self.dvc.add(self.DATA_DIR, recursive=True, fname="error.dvc")
|
|
|
|
ret = main(["add", self.DATA_DIR, "-f", "data_directory.dvc"])
|
|
self.assertEqual(0, ret)
|
|
self.assertTrue(os.path.exists("data_directory.dvc"))
|
|
|
|
ret = main(["add", self.FOO, "-f", "bar.dvc"])
|
|
self.assertEqual(0, ret)
|
|
self.assertTrue(os.path.exists("bar.dvc"))
|
|
self.assertFalse(os.path.exists("foo.dvc"))
|
|
|
|
os.remove("bar.dvc")
|
|
|
|
ret = main(["add", self.FOO, "--file", "bar.dvc"])
|
|
self.assertEqual(0, ret)
|
|
self.assertTrue(os.path.exists("bar.dvc"))
|
|
self.assertFalse(os.path.exists("foo.dvc"))
|
|
|
|
|
|
def test_failed_add_cleanup(tmp_dir, scm, dvc):
|
|
tmp_dir.gen({"foo": "foo", "bar": "bar"})
|
|
|
|
# Add and corrupt a stage file
|
|
dvc.add("foo")
|
|
tmp_dir.gen("foo.dvc", "- broken\nyaml")
|
|
|
|
with pytest.raises(StageFileCorruptedError):
|
|
dvc.add("bar")
|
|
|
|
assert not os.path.exists("bar.dvc")
|
|
|
|
gitignore_content = get_gitignore_content()
|
|
assert "/bar" not in gitignore_content
|
|
|
|
|
|
def test_should_not_track_git_internal_files(mocker, dvc, tmp_dir):
|
|
stage_creator_spy = mocker.spy(dvc_module.repo.add, "_create_stages")
|
|
|
|
ret = main(["add", "-R", dvc.root_dir])
|
|
assert ret == 0
|
|
|
|
created_stages_filenames = stage_creator_spy.mock.call_args[0][1]
|
|
for fname in created_stages_filenames:
|
|
assert ".git" not in fname
|
|
|
|
|
|
class TestAddUnprotected(TestDvc):
|
|
def test(self):
|
|
ret = main(["config", "cache.type", "hardlink"])
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["add", self.FOO])
|
|
self.assertEqual(ret, 0)
|
|
|
|
self.assertFalse(os.access(self.FOO, os.W_OK))
|
|
self.assertTrue(System.is_hardlink(self.FOO))
|
|
|
|
ret = main(["unprotect", self.FOO])
|
|
self.assertEqual(ret, 0)
|
|
|
|
ret = main(["add", self.FOO])
|
|
self.assertEqual(ret, 0)
|
|
|
|
self.assertFalse(os.access(self.FOO, os.W_OK))
|
|
self.assertTrue(System.is_hardlink(self.FOO))
|
|
|
|
|
|
@pytest.fixture
|
|
def temporary_windows_drive(tmp_path_factory):
|
|
import string
|
|
import win32api
|
|
from ctypes import windll
|
|
from win32con import DDD_REMOVE_DEFINITION
|
|
|
|
drives = [
|
|
s[0].upper()
|
|
for s in win32api.GetLogicalDriveStrings().split("\000")
|
|
if len(s) > 0
|
|
]
|
|
|
|
new_drive_name = [
|
|
letter for letter in string.ascii_uppercase if letter not in drives
|
|
][0]
|
|
new_drive = f"{new_drive_name}:"
|
|
|
|
target_path = tmp_path_factory.mktemp("tmp_windows_drive")
|
|
|
|
set_up_result = windll.kernel32.DefineDosDeviceW(
|
|
0, new_drive, os.fspath(target_path)
|
|
)
|
|
if set_up_result == 0:
|
|
raise RuntimeError("Failed to mount windows drive!")
|
|
|
|
# NOTE: new_drive has form of `A:` and joining it with some relative
|
|
# path might result in non-existing path (A:path\\to)
|
|
yield os.path.join(new_drive, os.sep)
|
|
|
|
tear_down_result = windll.kernel32.DefineDosDeviceW(
|
|
DDD_REMOVE_DEFINITION, new_drive, os.fspath(target_path)
|
|
)
|
|
if tear_down_result == 0:
|
|
raise RuntimeError("Could not unmount windows drive!")
|
|
|
|
|
|
@pytest.mark.skipif(os.name != "nt", reason="Windows specific")
|
|
def test_windows_should_add_when_cache_on_different_drive(
|
|
tmp_dir, dvc, temporary_windows_drive
|
|
):
|
|
dvc.config["cache"]["dir"] = temporary_windows_drive
|
|
dvc.cache = Cache(dvc)
|
|
|
|
(stage,) = tmp_dir.dvc_gen({"file": "file"})
|
|
cache_path = stage.outs[0].cache_path
|
|
|
|
assert path_isin(cache_path, temporary_windows_drive)
|
|
assert os.path.isfile(cache_path)
|
|
filecmp.cmp("file", cache_path)
|
|
|
|
|
|
def test_readding_dir_should_not_unprotect_all(tmp_dir, dvc, mocker):
|
|
tmp_dir.gen("dir/data", "data")
|
|
|
|
dvc.cache.local.cache_types = ["symlink"]
|
|
|
|
dvc.add("dir")
|
|
tmp_dir.gen("dir/new_file", "new_file_content")
|
|
|
|
unprotect_spy = mocker.spy(LocalRemote, "unprotect")
|
|
dvc.add("dir")
|
|
|
|
assert not unprotect_spy.mock.called
|
|
assert System.is_symlink(os.path.join("dir", "new_file"))
|
|
|
|
|
|
def test_should_not_checkout_when_adding_cached_copy(tmp_dir, dvc, mocker):
|
|
dvc.cache.local.cache_types = ["copy"]
|
|
|
|
tmp_dir.dvc_gen({"foo": "foo", "bar": "bar"})
|
|
|
|
shutil.copy("bar", "foo")
|
|
|
|
copy_spy = mocker.spy(dvc.cache.local, "copy")
|
|
|
|
dvc.add("foo")
|
|
|
|
assert copy_spy.mock.call_count == 0
|
|
|
|
|
|
@pytest.mark.parametrize(
|
|
"link,new_link,link_test_func",
|
|
[
|
|
("hardlink", "copy", lambda path: not System.is_hardlink(path)),
|
|
("symlink", "copy", lambda path: not System.is_symlink(path)),
|
|
("copy", "hardlink", System.is_hardlink),
|
|
("copy", "symlink", System.is_symlink),
|
|
],
|
|
)
|
|
def test_should_relink_on_repeated_add(
|
|
link, new_link, link_test_func, tmp_dir, dvc
|
|
):
|
|
from dvc.path_info import PathInfo
|
|
|
|
dvc.config["cache"]["type"] = link
|
|
|
|
tmp_dir.dvc_gen({"foo": "foo", "bar": "bar"})
|
|
|
|
os.remove("foo")
|
|
getattr(dvc.cache.local, link)(PathInfo("bar"), PathInfo("foo"))
|
|
|
|
dvc.cache.local.cache_types = [new_link]
|
|
|
|
dvc.add("foo")
|
|
|
|
assert link_test_func("foo")
|
|
|
|
|
|
@pytest.mark.parametrize("link", ["hardlink", "symlink", "copy"])
|
|
def test_should_protect_on_repeated_add(link, tmp_dir, dvc):
|
|
dvc.cache.local.cache_types = [link]
|
|
|
|
tmp_dir.dvc_gen({"foo": "foo"})
|
|
|
|
dvc.unprotect("foo")
|
|
|
|
dvc.add("foo")
|
|
|
|
assert not os.access(
|
|
os.path.join(".dvc", "cache", "ac", "bd18db4cc2f85cedef654fccc4a4d8"),
|
|
os.W_OK,
|
|
)
|
|
|
|
# NOTE: Windows symlink perms don't propagate to the target
|
|
if link == "copy" or (link == "symlink" and os.name == "nt"):
|
|
assert os.access("foo", os.W_OK)
|
|
else:
|
|
assert not os.access("foo", os.W_OK)
|
|
|
|
|
|
def test_escape_gitignore_entries(tmp_dir, scm, dvc):
|
|
fname = "file!with*weird#naming_[1].t?t"
|
|
ignored_fname = r"/file\!with\*weird\#naming_\[1\].t\?t"
|
|
|
|
if os.name == "nt":
|
|
# Some characters are not supported by Windows in the filename
|
|
# https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file
|
|
fname = "file!with_weird#naming_[1].txt"
|
|
ignored_fname = r"/file\!with_weird\#naming_\[1\].txt"
|
|
|
|
tmp_dir.dvc_gen(fname, "...")
|
|
assert ignored_fname in get_gitignore_content()
|
|
|
|
|
|
def test_add_from_data_dir(tmp_dir, scm, dvc):
|
|
tmp_dir.dvc_gen({"dir": {"file1": "file1 content"}})
|
|
|
|
tmp_dir.gen({"dir": {"file2": "file2 content"}})
|
|
|
|
with pytest.raises(OverlappingOutputPathsError) as e:
|
|
dvc.add(os.path.join("dir", "file2"))
|
|
assert str(e.value) == (
|
|
"Cannot add '{out}', because it is overlapping with other DVC "
|
|
"tracked output: 'dir'.\n"
|
|
"To include '{out}' in 'dir', run 'dvc commit dir.dvc'"
|
|
).format(out=os.path.join("dir", "file2"))
|
|
|
|
|
|
def test_not_raises_on_re_add(tmp_dir, dvc):
|
|
tmp_dir.dvc_gen("file", "file content")
|
|
|
|
tmp_dir.gen({"file2": "file2 content", "file": "modified file"})
|
|
dvc.add(["file2", "file"])
|
|
|
|
|
|
@pytest.mark.parametrize("link", ["hardlink", "symlink", "copy"])
|
|
def test_add_empty_files(tmp_dir, dvc, link):
|
|
file = "foo"
|
|
dvc.cache.local.cache_types = [link]
|
|
stages = tmp_dir.dvc_gen(file, "")
|
|
|
|
assert (tmp_dir / file).exists()
|
|
assert (tmp_dir / (file + DVC_FILE_SUFFIX)).exists()
|
|
assert os.path.exists(stages[0].outs[0].cache_path)
|
|
|
|
|
|
def test_add_optimization_for_hardlink_on_empty_files(tmp_dir, dvc, mocker):
|
|
dvc.cache.local.cache_types = ["hardlink"]
|
|
tmp_dir.gen({"foo": "", "bar": "", "lorem": "lorem", "ipsum": "ipsum"})
|
|
m = mocker.spy(LocalRemote, "is_hardlink")
|
|
stages = dvc.add(["foo", "bar", "lorem", "ipsum"])
|
|
|
|
assert m.call_count == 1
|
|
assert m.call_args != call(tmp_dir / "foo")
|
|
assert m.call_args != call(tmp_dir / "bar")
|
|
|
|
for stage in stages[:2]:
|
|
# hardlinks are not created for empty files
|
|
assert not System.is_hardlink(stage.outs[0].path_info)
|
|
|
|
for stage in stages[2:]:
|
|
assert System.is_hardlink(stage.outs[0].path_info)
|
|
|
|
for stage in stages:
|
|
assert os.path.exists(stage.path)
|
|
assert os.path.exists(stage.outs[0].cache_path)
|
|
|
|
|
|
def test_output_duplication_for_pipeline_tracked(tmp_dir, dvc, run_copy):
|
|
tmp_dir.dvc_gen("foo", "foo")
|
|
run_copy("foo", "bar", name="copy-foo-bar")
|
|
with pytest.raises(OutputDuplicationError):
|
|
dvc.add("bar")
|
|
|
|
|
|
def test_add_pipeline_file(tmp_dir, dvc, run_copy):
|
|
from dvc.dvcfile import PIPELINE_FILE
|
|
|
|
tmp_dir.dvc_gen("foo", "foo")
|
|
run_copy("foo", "bar", name="copy-foo-bar")
|
|
|
|
with pytest.raises(OutputIsStageFileError):
|
|
dvc.add(PIPELINE_FILE)
|