Package: release.debian.org Severity: normal User: release.debian....@packages.debian.org Usertags: unblock
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Please unblock package pristine-lfs [ Reason ] Not knowing that pristine-lfs is considered a key package, I made changes upstream and uploaded a new release into Debian, fixing a few bugs and improving the exported API to make the future integration with git-buildpackage easier. [ Impact ] The direct impact in the event the unblock is not granted is low, since the new features aren’t used yet and the bugs have been previously worked around specifically in Debian bullseye. On the other hand, for the outlined reasons the impact on unblocking will also be very low. [ Tests ] - - There’s a test of tests running upstream on four Python releases (3.7, 3.8, 3.9 and 3.10-rc) and in Debian on the current Python release at build time; these tests cover all features but test them directly in Python - - The Debian package ships autopkgtests testing some of the basic features through the command-line interface. - - The upstream Ci runs mypy static type checks. [ Risks ] The changes are not trivial but OTOH not big and covered by tests; the overall test coverage has increased since the last upload, hence the risks of accepting this package into bullseye should be relatively low. [ Checklist ] [x] all changes are documented in the d/changelog [x] I reviewed all changes and I approve them [x] attach debdiff against the package in testing unblock pristine-lfs/20210404.0-1 -----BEGIN PGP SIGNATURE----- iHUEARYIAB0WIQSD3NF/RLIsyDZW7aHoRGtKyMdyYQUCYGsP6AAKCRDoRGtKyMdy Yc+rAQCsmONKT1dDAIwolvhF5uMJXtR9KiqIOyQi1DSosoJ1ZgEA+bf359mV6UDQ +g2N6APILtvME0Agpz62ZKDz4YqI6wo= =5H1H -----END PGP SIGNATURE-----
diff --git a/PKG-INFO b/PKG-INFO index 8db2f11..5b3e484 100644 --- a/PKG-INFO +++ b/PKG-INFO @@ -1,6 +1,6 @@ Metadata-Version: 2.1 Name: pristine-lfs -Version: 20210222.0 +Version: 20210404.0 Summary: a pristine-tar replacement that works with Git LFS Home-page: https://salsa.debian.org/pristine-lfs-team/pristine-lfs Author: Andrej Shadura diff --git a/debian/changelog b/debian/changelog index 9804ad5..1afd619 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +pristine-lfs (20210404.0-1) unstable; urgency=medium + + * New upstream release. + + -- Andrej Shadura <andrew.shad...@collabora.co.uk> Sun, 04 Apr 2021 22:15:42 +0200 + pristine-lfs (20210222.0-1) unstable; urgency=medium * New upstream release. diff --git a/pristine_lfs.egg-info/PKG-INFO b/pristine_lfs.egg-info/PKG-INFO index 8db2f11..5b3e484 100644 --- a/pristine_lfs.egg-info/PKG-INFO +++ b/pristine_lfs.egg-info/PKG-INFO @@ -1,6 +1,6 @@ Metadata-Version: 2.1 Name: pristine-lfs -Version: 20210222.0 +Version: 20210404.0 Summary: a pristine-tar replacement that works with Git LFS Home-page: https://salsa.debian.org/pristine-lfs-team/pristine-lfs Author: Andrej Shadura diff --git a/pristine_lfs.egg-info/SOURCES.txt b/pristine_lfs.egg-info/SOURCES.txt index 70a9467..2f79c38 100644 --- a/pristine_lfs.egg-info/SOURCES.txt +++ b/pristine_lfs.egg-info/SOURCES.txt @@ -6,8 +6,10 @@ pristine-lfs.rst setup.cfg setup.py pristine_lfs/__init__.py +pristine_lfs/errors.py pristine_lfs/gitwrap.py pristine_lfs/gitwrap.pyi +pristine_lfs/log.py pristine_lfs/main.py pristine_lfs/util.py pristine_lfs.egg-info/PKG-INFO diff --git a/pristine_lfs.egg-info/requires.txt b/pristine_lfs.egg-info/requires.txt index b065e88..9a19637 100644 --- a/pristine_lfs.egg-info/requires.txt +++ b/pristine_lfs.egg-info/requires.txt @@ -1,2 +1,2 @@ python-debian -sh +sh>=1.14 diff --git a/pristine_lfs/__init__.py b/pristine_lfs/__init__.py index 6df221a..ded6987 100644 --- a/pristine_lfs/__init__.py +++ b/pristine_lfs/__init__.py @@ -1,5 +1,6 @@ from .main import ( # noqa: F401 do_commit, + do_commit_files, do_checkout, do_list, do_import, diff --git a/pristine_lfs/errors.py b/pristine_lfs/errors.py new file mode 100644 index 0000000..ca1db96 --- /dev/null +++ b/pristine_lfs/errors.py @@ -0,0 +1,67 @@ +# pristine-lfs +# +# errors for pristine-lfs +# +# Copyright (C) 2021 Collabora Ltd +# Copyright (C) 2021 Andrej Shadura <andrew.shad...@collabora.co.uk> +# +# SPDX-License-Identifier: GPL-2.0-or-later + +from __future__ import annotations + +from gettext import gettext as _ + +from sh import ErrorReturnCode as CommandFailed # noqa: F401 + + +class DifferentFilesExist(Exception): + files: list[str] + + def __init__(self, files: list[str]): + self.files = files + + def __str__(self): + return _("would overwrite files: {files}").format(files=', '.join(self.files)) + + +class UnsupportedHashAlgorithm(Exception): + algo: str + + def __init__(self, algo: str): + self.algo = algo + + def __str__(self): + return _("unsupported hash algorithm {algo}").format( + algo=self.algo, + ) + + +class GitError(Exception): + pass + + +class GitFileNotFound(GitError): + filename: str + branch: str + + def __init__(self, filename: str, branch: str): + self.filename = filename + self.branch = branch + + def __str__(self): + return _('{filename} not found on branch {branch}').format( + filename=self.filename, + branch=self.branch, + ) + + +class GitBranchNotFound(GitError): + branch: str + + def __init__(self, branch: str): + self.branch = branch + + def __str__(self): + return _('No branch {branch} found, not even among remote branches').format( + branch=self.branch, + ) diff --git a/pristine_lfs/gitwrap.py b/pristine_lfs/gitwrap.py index 0b0a83a..824988d 100644 --- a/pristine_lfs/gitwrap.py +++ b/pristine_lfs/gitwrap.py @@ -1,7 +1,7 @@ # Wrapper for Git and Git LFS # # Copyright (C) 2021 Collabora Ltd -# Andrej Shadura <andrew.shad...@collabora.co.uk> +# Copyright (C) 2021 Andrej Shadura <andrew.shad...@collabora.co.uk> # # SPDX-License-Identifier: GPL-2.0-or-later diff --git a/pristine_lfs/gitwrap.pyi b/pristine_lfs/gitwrap.pyi index 200b212..c760b45 100644 --- a/pristine_lfs/gitwrap.pyi +++ b/pristine_lfs/gitwrap.pyi @@ -2,7 +2,7 @@ # Extend as needed. # # Copyright (C) 2021 Collabora Ltd -# Andrej Shadura <andrew.shad...@collabora.co.uk> +# Copyright (C) 2021 Andrej Shadura <andrew.shad...@collabora.co.uk> from sh.contrib import git as sh_git diff --git a/pristine_lfs/log.py b/pristine_lfs/log.py new file mode 100644 index 0000000..a5b3cac --- /dev/null +++ b/pristine_lfs/log.py @@ -0,0 +1,13 @@ +# pristine-lfs +# +# logging +# +# Copyright (C) 2021 Collabora Ltd +# Copyright (C) 2021 Andrej Shadura <andrew.shad...@collabora.co.uk> +# +# SPDX-License-Identifier: GPL-2.0-or-later + +import logging + + +logger = logging.getLogger("pristine-lfs") diff --git a/pristine_lfs/main.py b/pristine_lfs/main.py index 5cbaf66..921c312 100644 --- a/pristine_lfs/main.py +++ b/pristine_lfs/main.py @@ -3,7 +3,7 @@ # store pristine tarballs in Git LFS # # Copyright (C) 2019—2021 Collabora Ltd -# Andrej Shadura <andrew.shad...@collabora.co.uk> +# Copyright (C) 2019—2021 Andrej Shadura <andrew.shad...@collabora.co.uk> # # SPDX-License-Identifier: GPL-2.0-or-later @@ -18,6 +18,7 @@ from typing import ( IO, Iterable, Optional, + Sequence, Union ) @@ -25,9 +26,14 @@ import sh from debian import deb822 from debian.changelog import Changelog, Version +from .errors import ( + CommandFailed, + DifferentFilesExist, + GitError, + UnsupportedHashAlgorithm, +) +from .log import logger from .util import ( - Abort, - GitFileNotFound, check_branch, checkout_lfs_file, checkout_package, @@ -53,13 +59,29 @@ def do_commit(tarball: IO[bytes], branch: str, message: Optional[str] = None, fo commit_lfs_file(tarball, branch, message, overwrite=force_overwrite) -def do_checkout(branch: str, tarball: Optional[str] = None, outdir: Union[str, Path] = '.', full: bool = False, **kwargs): +def do_commit_files(tarballs: Sequence[IO[bytes]], branch: str, message: Optional[str] = None, + force_overwrite: bool = False, **kwargs): + """ + Commit open files to a branch using Git LFS. + Set force_overwrite to overwrite existing files with same names and different checksums. + Message may contain "%s" which gets replaced with a comma-separate list of the file committed. + """ + if check_branch(branch) is None: + if find_remote_branches(branch): + track_remote_branch(branch) + commit_lfs_files(tarballs, branch, message, overwrite=force_overwrite) + + +def do_checkout(branch: str, tarball: Optional[str] = None, outdir: Union[str, Path] = '.', full: bool = False, + package: Optional[str] = None, version: Union[str, Version, None] = None, **kwargs): """ Check out one or multiple files. If tarball is non-None: * file name only: tarball to check out to outdir. * path with to file: the location where to check out to If tarball is None: + * if package and version are specified, that version + is checked out * a tarball corresponding to the latest entry in debian/changelog is found and checked out @@ -74,10 +96,11 @@ def do_checkout(branch: str, tarball: Optional[str] = None, outdir: Union[str, P if path: outdir = path else: - changelog = Path("debian/changelog") - with changelog.open() as f: - ch = Changelog(f, max_blocks=1) - package, version = ch.package, ch.version + if not package or not version: + changelog = Path("debian/changelog") + with changelog.open() as f: + ch = Changelog(f, max_blocks=1) + package, version = ch.package, ch.version outdir = Path(outdir) outdir.mkdir(parents=True, exist_ok=True) @@ -86,24 +109,24 @@ def do_checkout(branch: str, tarball: Optional[str] = None, outdir: Union[str, P if tarball: dsc_file = tarball else: - fver = Version(ch.version) + fver = Version(version) fver.epoch = None dsc_file = f'{package}_{fver}.dsc' - logging.info(_("Checking out file {} in {}").format(dsc_file, outdir)) + logger.info(_("Checking out file {} in {}").format(dsc_file, outdir)) checkout_lfs_file(branch, dsc_file, outdir) if dsc_file.endswith('.dsc'): with (outdir / dsc_file).open('r') as dsc: d = deb822.Dsc(dsc) - package = d['Source'] + package = str(d['Source']) version = Version(d['Version']) files = [f["name"] for f in d["Files"]] checkout_package(package, version, branch, outdir, files) else: if tarball: - logging.info(_("Checking out file {} in {}").format(tarball, outdir)) + logger.info(_("Checking out file {} in {}").format(tarball, outdir)) checkout_lfs_file(branch, tarball, outdir) else: - checkout_package(package, version, branch, outdir) + checkout_package(str(package), Version(version), branch, outdir) def do_list(branch: str, **kwargs) -> Iterable[str]: @@ -116,7 +139,8 @@ def do_list(branch: str, **kwargs) -> Iterable[str]: yield f -def do_import(dsc: IO[str], branch: str, message: Optional[str] = None, force_overwrite: bool = False, full: bool = False, **kwargs): +def do_import(dsc: IO[str], branch: str, message: Optional[str] = None, force_overwrite: bool = False, + full: bool = False, **kwargs): """ Import all tarballs and detached signatures related to an open .dsc file. Set force_overwrite to overwrite an existing file with the same name and a different checksum. @@ -135,12 +159,13 @@ def do_import(dsc: IO[str], branch: str, message: Optional[str] = None, force_ov if find_remote_branches(branch): track_remote_branch(branch) - tarballs = [os.path.join(dsc_dir, f['name']) for f in d['Files'] if full or fnmatch(f['name'], tarball_glob) or fnmatch(f['name'], component_tarball_glob)] + tarballs = [os.path.join(dsc_dir, f['name']) for f in d['Files'] + if full or fnmatch(f['name'], tarball_glob) or fnmatch(f['name'], component_tarball_glob)] if full: tarballs += [dsc.name] if tarballs: - logging.info("Importing: %s" % " ".join(tarballs)) + logger.info("Importing: %s" % " ".join(tarballs)) commit_lfs_files([open(tarball, 'rb') for tarball in tarballs], branch, message, overwrite=force_overwrite) @@ -156,7 +181,10 @@ def do_verify(branch: str, tarball: Union[str, Path], **kwargs) -> bool: def main(*args): prog = os.path.basename(sys.argv[0]) - parser = argparse.ArgumentParser(description=_('store pristine tarballs in Git LFS'), prog=prog, exit_on_error=not args) + parser = argparse.ArgumentParser(description=_('store pristine tarballs in Git LFS'), prog=prog) + if args and hasattr(parser, 'exit_on_error'): + parser.exit_on_error = False + parser.add_argument('-v', '--verbose', action='count', help=_('be more verbose')) parser.add_argument('--debug', action='store_const', const=2, dest='verbose', help=_('be debuggingly verbose')) parser.set_defaults(verbose=0, func=lambda *x, **kw: parser.print_usage(file=sys.stderr)) @@ -173,10 +201,12 @@ def main(*args): # we have to do some trickery since argparse doesn’t support this syntax natively parser_checkout = subparsers.add_parser('checkout', help=_('checkout a tarball')) parser_checkout.add_argument('-b', '--branch', default='pristine-lfs', help=_('branch to store metadata on')) - parser_checkout.add_argument('--full', default=False, action='store_true', help=_('also check out all related files of the Debian package')) + parser_checkout.add_argument('--full', default=False, action='store_true', + help=_('also check out all related files of the Debian package')) parser_checkout.add_argument('-o', '--outdir', default='.', help=_('output directory for the tarball')) checkout_group = parser_checkout.add_mutually_exclusive_group(required=True) - checkout_group.add_argument('--auto', default=False, action='store_true', help=_('check out all tarballs required by the currently checked out Debian package')) + checkout_group.add_argument('--auto', default=False, action='store_true', + help=_('check out all tarballs required by the currently checked out Debian package')) checkout_group.add_argument('tarball', nargs='?', default=None, help=_('tarball to check out')) parser_checkout.set_defaults(func=do_checkout) @@ -187,7 +217,8 @@ def main(*args): parser_import = subparsers.add_parser('import-dsc', help=_('import tarballs and their signatures from a .dsc')) parser_import.add_argument('dsc', type=argparse.FileType('r'), help='.dsc file to use') parser_import.add_argument('--force-overwrite', action='store_true', help=_('overwrite already stored files')) - parser_import.add_argument('--full', default=False, action='store_true', help=_('also import all related files of the Debian package')) + parser_import.add_argument('--full', default=False, action='store_true', + help=_('also import all related files of the Debian package')) parser_import.add_argument('-m', '--message', default=None, help=_('commit message')) parser_import.add_argument('-b', '--branch', default='pristine-lfs', help=_('branch to store metadata on')) parser_import.set_defaults(func=do_import) @@ -211,7 +242,7 @@ def main(*args): print(item) elif isinstance(ret, bool): return 0 if ret else 1 - except sh.ErrorReturnCode as e: + except CommandFailed as e: print(_('Failed to run %s:') % e.full_cmd, file=sys.stderr) print(e.stderr.decode(sh.DEFAULT_ENCODING, "replace"), file=sys.stderr) return e.exit_code @@ -225,6 +256,6 @@ def main(*args): print(file=sys.stderr) print(_('about: Interrupted by user'), file=sys.stderr) return 1 - except (Abort, GitFileNotFound) as e: + except (DifferentFilesExist, GitError, UnsupportedHashAlgorithm) as e: print(_("abort: %s\n") % e, file=sys.stderr) return 1 diff --git a/pristine_lfs/util.py b/pristine_lfs/util.py index ba7f908..48b07ef 100644 --- a/pristine_lfs/util.py +++ b/pristine_lfs/util.py @@ -4,14 +4,13 @@ # This requires Git and git-lfs to be installed. # # Copyright (C) 2019—2021 Collabora Ltd -# Andrej Shadura <andrew.shad...@collabora.co.uk> +# Copyright (C) 2019—2021 Andrej Shadura <andrew.shad...@collabora.co.uk> # # SPDX-License-Identifier: GPL-2.0-or-later from __future__ import annotations import hashlib -import logging import os from contextlib import contextmanager from fnmatch import fnmatch, fnmatchcase @@ -22,7 +21,14 @@ from typing import IO, Any, Generator, Iterable, Mapping, Optional, Sequence, Tu import sh from debian.changelog import Version +from .errors import ( + DifferentFilesExist, + GitBranchNotFound, + GitFileNotFound, + UnsupportedHashAlgorithm, +) from .gitwrap import git +from .log import logger gitattributes = """*.tar.* filter=lfs diff=lfs merge=lfs -text @@ -133,22 +139,6 @@ def parse_git_attributes(s: str) -> Iterable[tuple[str, Mapping[str, AttributeVa default_gitattributes = list(parse_git_attributes(gitattributes)) -class Abort(Exception): - pass - - -class GitFileNotFound(Exception): - filename: str - branch: str - - def __init__(self, filename: str, branch: str): - self.filename = filename - self.branch = branch - - def __str__(self): - return _('%s not found on branch %s') % (self.filename, self.branch) - - def check_branch(name: str) -> Optional[str]: """ Check a branch exists, return the hash it points at, if it does. @@ -187,9 +177,9 @@ def find_remote_branches(name: str) -> list[tuple[str, str]]: def preferred_remote_branch(remote_branches: list[tuple[str, str]]) -> tuple[str, str]: - logging.debug("Remote branches: %r", remote_branches) + logger.debug("Remote branches: %r", remote_branches) current_remote = branch_remote(git_head()) - logging.debug("Current remote: %r", current_remote) + logger.debug("Current remote: %r", current_remote) remote_branches = [ (commit, ref) for (commit, ref) in remote_branches if current_remote and ref.startswith('refs/remotes/' + current_remote) @@ -211,7 +201,7 @@ def find_branch(branch: str) -> str: if remote_branches: commit, branch = preferred_remote_branch(remote_branches) else: - raise Abort(_('No branch {branch} found, not even among remote branches').format(branch=branch)) + raise GitBranchNotFound(branch) return branch @@ -307,7 +297,7 @@ def commit_lfs_files(ios: Sequence[IO[bytes]], branch: str, template: str = None hook_path.write_text(pre_push_hook) hook_path.chmod(0o755) except IOError as e: - logging.warning(_('Failed to set up pre-push hook: %s') % e.strerror) + logger.warning(_('Failed to set up pre-push hook: %s') % e.strerror) with open_index("pristine-lfs") as index: # make sure we include all previously committed files @@ -328,12 +318,12 @@ def commit_lfs_files(ios: Sequence[IO[bytes]], branch: str, template: str = None if check_branch(branch) is not None: diff = git('diff-index', '--cached', branch, index=index).strip().splitlines() if not diff: - logging.info(_("Nothing to commit")) + logger.info(_("Nothing to commit")) return parsed_diff = [parse_diff_entry(d) for d in diff] overwritten = [d['srcname'] for d in parsed_diff if d['srchash'] != ('0' * 40) and d['srcname'] != '.gitattributes'] if any(overwritten) and not overwrite: - raise Abort(_('would overwrite files: %s') % ', '.join(overwritten)) + raise DifferentFilesExist(overwritten) if not template: template = "pristine-lfs data for %s" @@ -399,7 +389,7 @@ def verify_lfs_file(branch: str, tarball: Path) -> bool: oid = parsed_metadata['oid'] algo, hashsum = oid.split(':', 1) if algo not in supported_lfs_hashsums: - raise Abort(_("unsupported hash algorithm %s, cannot verify") % algo) + raise UnsupportedHashAlgorithm(algo) h = getattr(hashlib, algo)() with open(tarball, mode='rb') as f: @@ -410,9 +400,9 @@ def verify_lfs_file(branch: str, tarball: Path) -> bool: h.update(chunk) calc_hashsum = h.hexdigest() if hashsum == calc_hashsum: - logging.info(f"{algo} hash for the tarball: {hashsum}, matches the stored one") + logger.info(f"{algo} hash for the tarball: {hashsum}, matches the stored one") else: - logging.warning(_("%(tarball)s does not match stored hash (expected %(stored_hash)s, got %(tarball_hash)s)") % { + logger.warning(_("%(tarball)s does not match stored hash (expected %(stored_hash)s, got %(tarball_hash)s)") % { 'tarball': filename, 'stored_hash': hashsum, 'tarball_hash': calc_hashsum, @@ -420,8 +410,12 @@ def verify_lfs_file(branch: str, tarball: Path) -> bool: return hashsum == calc_hashsum -def checkout_package(package: str, version: Version, branch: str, outdir: Union[str, Path], requested: Optional[Sequence[str]] = None): - logging.info(_("Checking out files for {package} version {version} to {outdir}:").format(package=package, version=version, outdir=outdir)) +def checkout_package(package: str, version: Version, branch: str, outdir: Union[str, Path], + requested: Optional[Sequence[str]] = None): + logger.info(_("Checking out files for {package} version {version} to {outdir}:").format( + package=package, + version=version, outdir=outdir + )) tarball_glob = f'{package}_{version.upstream_version}.orig.tar.*' component_tarball_glob = f'{package}_{version.upstream_version}.orig-*.tar.*' @@ -433,9 +427,9 @@ def checkout_package(package: str, version: Version, branch: str, outdir: Union[ tarballs = [f for f in files if fnmatch(f, tarball_glob) or fnmatch(f, component_tarball_glob)] for f in tarballs: - logging.info(" ... {}".format(f)) + logger.info(" ... {}".format(f)) checkout_lfs_file(branch, f, outdir) - logging.info(_("Done.")) + logger.info(_("Done.")) def parse_pointer(pointer: IO[str]) -> Iterable[tuple[str, str]]: diff --git a/setup.cfg b/setup.cfg index 04de573..16535af 100644 --- a/setup.cfg +++ b/setup.cfg @@ -1,6 +1,6 @@ [metadata] name = pristine-lfs -version = 20210222.0 +version = 20210404.0 author = Andrej Shadura author_email = andrew.shad...@collabora.co.uk url = https://salsa.debian.org/pristine-lfs-team/pristine-lfs @@ -28,7 +28,7 @@ packages = find: setup_requires = docutils >= 0.12 install_requires = - sh + sh >= 1.14 python-debian include_package_data = True tests_require = @@ -43,9 +43,11 @@ pristine_lfs = stubs/* [mypy] allow_redefinition = True +junit_xml = mypy.xml [tool:pytest] -addopts = --doctest-modules +addopts = --doctest-modules --junit-xml=test-results.xml +junit_family = xunit2 doctest_optionflags = NORMALIZE_WHITESPACE markers = smoke @@ -54,7 +56,7 @@ markers = doctests = yes max-line-length = 130 exclude = .git,build,__pycache__,setup.py -ignore = E121,E123,E126,E133,E226,E241,E242,E704,E501,E301,E261,E127,E128,W391,W503,W504 +ignore = E121,E123,E126,E133,E226,E241,E242,E704,E261,E127,E128,W503,W504 [isort] multi_line_output = 3 @@ -64,6 +66,9 @@ line_length = 130 reverse_relative = true default_section = THIRDPARTY +[pylint.FORMAT] +max-line-length = 130 + [egg_info] tag_build = tag_date = 0 diff --git a/tests/test_checkout.py b/tests/test_checkout.py index f3c289a..9582527 100644 --- a/tests/test_checkout.py +++ b/tests/test_checkout.py @@ -1,4 +1,5 @@ from pristine_lfs import do_checkout +from pristine_lfs.util import Version def test_pristine_lfs_simple_checkout(fake_pristine_lfs): @@ -16,15 +17,33 @@ def test_pristine_lfs_auto_checkout(test_git_repo): outdir = repo / 'tmp-explicit' do_checkout('pristine-lfs', tarball=tarball.name, outdir=outdir) - assert len(list(outdir.glob('**'))) == 1, list(outdir.glob('**')) + assert len(list(outdir.glob('*'))) == 1, list(outdir.glob('*')) assert (outdir / tarball.name).is_file() assert (outdir / tarball.name).stat().st_size == size outdir = repo / 'tmp-auto' do_checkout('pristine-lfs', tarball=None, outdir=outdir) - assert len(list(outdir.glob('**'))) == 1, list(outdir.glob('**')) + assert len(list(outdir.glob('*'))) == 1, list(outdir.glob('*')) assert (outdir / tarball.name).is_file(), 'Extracted tarball not found' assert (outdir / tarball.name).stat().st_size == size, 'Extracted tarball of a wrong size' + outdir = repo / 'tmp-empty' + + do_checkout('pristine-lfs', package='true', version='1', outdir=outdir) + assert len(list(outdir.glob('*'))) == 0, list(outdir.glob('*')) + assert not (outdir / tarball.name).is_file(), 'Found a tarball which should not be there' + + do_checkout('pristine-lfs', package='true', version=Version('1'), outdir=outdir) + assert len(list(outdir.glob('*'))) == 0, list(outdir.glob('*')) + assert not (outdir / tarball.name).is_file(), 'Found a tarball which should not be there' + do_checkout('pristine-lfs', package='true', version='0', outdir=outdir) + assert len(list(outdir.glob('*'))) == 1, list(outdir.glob('*')) + assert (outdir / tarball.name).is_file(), 'Extracted tarball not found' + + outdir = repo / 'tmp-empty-2' + + do_checkout('pristine-lfs', package='true', version='0', outdir=outdir) + assert len(list(outdir.glob('*'))) == 1, list(outdir.glob('*')) + assert (outdir / tarball.name).is_file(), 'Extracted tarball not found' diff --git a/tests/test_commit.py b/tests/test_commit.py index c578d86..207b662 100644 --- a/tests/test_commit.py +++ b/tests/test_commit.py @@ -1,8 +1,10 @@ from textwrap import dedent +import pytest from sh.contrib import git from pristine_lfs import do_commit +from pristine_lfs.errors import DifferentFilesExist def test_pristine_lfs_commit(fake_tarball): @@ -24,3 +26,13 @@ def test_pristine_lfs_commit(fake_tarball): stored = repo / '.git' / 'lfs' / 'objects' / sha[:2] / sha[2:4] / sha assert stored.is_file(), 'Object has not been stored by LFS' + + +def test_pristine_lfs_commit_overwrite(fake_tarball): + repo, tarball, size, sha = fake_tarball + + do_commit(tarball.open('rb'), branch='pristine-lfs') + tarball.write_text('Text') + + with pytest.raises(DifferentFilesExist): + do_commit(tarball.open('rb'), branch='pristine-lfs', message='blip %s %s %s') diff --git a/tests/test_import_dsc.py b/tests/test_import_dsc.py index e5e5a4c..df94fda 100644 --- a/tests/test_import_dsc.py +++ b/tests/test_import_dsc.py @@ -40,4 +40,3 @@ def test_pristine_lfs_import_dsc(fake_tarball): ret = list(do_list(branch='pristine-lfs-source')) assert ['true_0.dsc', 'true_0.orig.tar.gz'] == ret -
