Source.run_scripts() - Code Metrics - Inspection of "Support post-install scripts" - jacebrowning/gitman - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#148)

by Jace

created 2017-01-20 22:39 UTC

Source.run_scripts() C

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	19
CRAP Score	7.0422

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	7
dl	0
loc	29
ccs	19
cts	21
cp	0.9048
crap	7.0422
rs	5.5
c	1
b	0
f	0

"""Wrappers for the dependency configuration files."""

import os
import logging
import warnings

import yorm
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from yorm.types import String, List, AttributeDictionary
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

from .. import common, exceptions, shell, git


log = logging.getLogger(__name__)



@yorm.attr(name=String)
@yorm.attr(repo=String)
@yorm.attr(rev=String)
@yorm.attr(link=String)
@yorm.attr(scripts=List.of_type(String))
class Source(AttributeDictionary):
    """A dictionary of `git` and `ln` arguments."""

    DIRTY = '<dirty>'
    UNKNOWN = '<unknown>'

    def __init__(self, repo, name, rev='master', link=None, scripts=None):

        super().__init__()
        self.repo = repo
        self.name = name
        self.rev = rev
        self.link = link
        self.scripts = scripts or []
        if not self.repo:
            msg = "'repo' missing on {}".format(repr(self))
            raise exceptions.InvalidConfig(msg)
        if not self.name:
            msg = "'name' missing on {}".format(repr(self))
            raise exceptions.InvalidConfig(msg)

    def __repr__(self):
        return "<source {}>".format(self)

    def __str__(self):
        pattern = "'{r}' @ '{v}' in '{d}'"
        if self.link:
            pattern += " <- '{s}'"
        return pattern.format(r=self.repo, v=self.rev, d=self.name, s=self.link)

    def __eq__(self, other):
        return self.name == other.name

    def __ne__(self, other):
        return self.name != other.name

    def __lt__(self, other):
        return self.name < other.name

    def update_files(self, force=False, fetch=False, clean=True):
        """Ensure the source matches the specified revision."""
        log.info("Updating source files...")

        # Clone the repository if needed
        if not os.path.exists(self.name):
            git.clone(self.repo, self.name)

        # Enter the working tree
        shell.cd(self.name)
        if not git.valid():
            raise self._invalid_repository

        # Check for uncommitted changes
        if not force:
            log.debug("Confirming there are no uncommitted changes...")
            if git.changes(include_untracked=clean):
                msg = "Uncommitted changes in {}".format(os.getcwd())
                raise exceptions.UncommittedChanges(msg)

        # Fetch the desired revision
        if fetch or self.rev not in (git.get_branch(),
                                     git.get_hash(),
                                     git.get_tag()):
            git.fetch(self.repo, self.rev)

        # Update the working tree to the desired revision
        git.update(self.rev, fetch=fetch, clean=clean)

    def create_link(self, root, force=False):
        """Create a link from the target name to the current directory."""
        if not self.link:
            return

        log.info("Creating a symbolic link...")

        if os.name == 'nt':
            warnings.warn("Symbolic links are not supported on Windows")
            return

        target = os.path.join(root, self.link)
        source = os.path.relpath(os.getcwd(), os.path.dirname(target))

        if os.path.islink(target):
            os.remove(target)
        elif os.path.exists(target):
            if force:
                shell.rm(target)
            else:
                msg = "Preexisting link location at {}".format(target)
                raise exceptions.UncommittedChanges(msg)

        shell.ln(source, target)

    def run_scripts(self, force=False):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        log.info("Running install scripts...")

        # Enter the working tree
        shell.cd(self.name)
        if not git.valid():
            raise self._invalid_repository

        # Check for scripts
        if not self.scripts:
            common.show("(no scripts to run)", color='shell_info')
            common.newline()
            return

        # Run all scripts
        for script in self.scripts:
            try:
                lines = shell.call(script, _shell=True)
            except exceptions.ShellError as exc:
                common.show(*exc.output, color='shell_error')
                cmd = exc.program
                if force:
                    log.debug("Ignored error from call to '%s'", cmd)
                else:
                    msg = "Command '{}' failed in {}".format(cmd, os.getcwd())
                    raise exceptions.ScriptFailure(msg)
            else:
                common.show(*lines, color='shell_output')

        common.newline()

    def identify(self, allow_dirty=True, allow_missing=True):
        """Get the path and current repository URL and hash."""
        if os.path.isdir(self.name):

            shell.cd(self.name)
            if not git.valid():
                raise self._invalid_repository

            path = os.getcwd()
            url = git.get_url()
            if git.changes(display_status=not allow_dirty, _show=True):
                if not allow_dirty:
                    msg = "Uncommitted changes in {}".format(os.getcwd())
                    raise exceptions.UncommittedChanges(msg)

                common.show(self.DIRTY, color='git_dirty', log=False)
                common.newline()
                return path, url, self.DIRTY
            else:
                rev = git.get_hash(_show=True)
                common.show(rev, color='git_rev', log=False)
                common.newline()
                return path, url, rev

        elif allow_missing:

            return os.getcwd(), '<missing>', self.UNKNOWN

        else:

            raise self._invalid_repository

    def lock(self, rev=None):
        """Return a locked version of the current source."""
        if rev is None:
            _, _, rev = self.identify(allow_dirty=False, allow_missing=False)
        source = self.__class__(self.repo, self.name, rev,
                                self.link, self.scripts)
        return source

    @property
    def _invalid_repository(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        path = os.path.join(os.getcwd(), self.name)
        msg = "Not a valid repository: {}".format(path)
        return exceptions.InvalidRepository(msg)


1		"""Wrappers for the dependency configuration files."""
2
3	1	import os
4	1	import logging
5	1	import warnings
6
7	1	import yorm
		0 ignored issues – show Configuration introduced 2016-03-14 20:50 UTC by Report Bug Copy Issue Report The import `yorm` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
8	1	from yorm.types import String, List, AttributeDictionary
		0 ignored issues – show Configuration introduced 2016-03-31 00:54 UTC by Report Bug Copy Issue Report The import `yorm.types` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
9
10	1	from .. import common, exceptions, shell, git
11
12
13	1	log = logging.getLogger(__name__)
		0 ignored issues – show Coding Style Naming introduced 2016-04-22 01:39 UTC by Report Bug Copy Issue Report The name `log` does not conform to the constant naming conventions (`(([A-Z_][A-Z0-9_])\|(__.__))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
14
15
16	1	@yorm.attr(name=String)
17	1	@yorm.attr(repo=String)
18	1	@yorm.attr(rev=String)
19	1	@yorm.attr(link=String)
20	1	@yorm.attr(scripts=List.of_type(String))
21	1	class Source(AttributeDictionary):
22		"""A dictionary of `git` and `ln` arguments."""
23
24	1	DIRTY = '<dirty>'
25	1	UNKNOWN = '<unknown>'
26
27	1	def __init__(self, repo, name, rev='master', link=None, scripts=None):
		0 ignored issues – show best-practice introduced 2017-01-20 02:55 UTC by Report Bug Copy Issue Report Too many arguments (6/5) Loading history...
28	1	super().__init__()
29	1	self.repo = repo
30	1	self.name = name
31	1	self.rev = rev
32	1	self.link = link
33	1	self.scripts = scripts or []
34	1	if not self.repo:
35	1	msg = "'repo' missing on {}".format(repr(self))
36	1	raise exceptions.InvalidConfig(msg)
37	1	if not self.name:
38	1	msg = "'name' missing on {}".format(repr(self))
39	1	raise exceptions.InvalidConfig(msg)
40
41	1	def __repr__(self):
42	1	return "<source {}>".format(self)
43
44	1	def __str__(self):
45	1	pattern = "'{r}' @ '{v}' in '{d}'"
46	1	if self.link:
47	1	pattern += " <- '{s}'"
48	1	return pattern.format(r=self.repo, v=self.rev, d=self.name, s=self.link)
49
50	1	def __eq__(self, other):
51	1	return self.name == other.name
52
53	1	def __ne__(self, other):
54	1	return self.name != other.name
55
56	1	def __lt__(self, other):
57	1	return self.name < other.name
58
59	1	def update_files(self, force=False, fetch=False, clean=True):
60		"""Ensure the source matches the specified revision."""
61	1	log.info("Updating source files...")
62
63		# Clone the repository if needed
64	1	if not os.path.exists(self.name):
65	1	git.clone(self.repo, self.name)
66
67		# Enter the working tree
68	1	shell.cd(self.name)
69	1	if not git.valid():
70	1	raise self._invalid_repository
71
72		# Check for uncommitted changes
73	1	if not force:
74	1	log.debug("Confirming there are no uncommitted changes...")
75	1	if git.changes(include_untracked=clean):
76		msg = "Uncommitted changes in {}".format(os.getcwd())
77		raise exceptions.UncommittedChanges(msg)
78
79		# Fetch the desired revision
80	1	if fetch or self.rev not in (git.get_branch(),
81		git.get_hash(),
82		git.get_tag()):
83	1	git.fetch(self.repo, self.rev)
84
85		# Update the working tree to the desired revision
86	1	git.update(self.rev, fetch=fetch, clean=clean)
87
88	1	def create_link(self, root, force=False):
89		"""Create a link from the target name to the current directory."""
90	1	if not self.link:
91	1	return
92
93	1	log.info("Creating a symbolic link...")
94
95	1	if os.name == 'nt':
96		warnings.warn("Symbolic links are not supported on Windows")
97		return
98
99	1	target = os.path.join(root, self.link)
100	1	source = os.path.relpath(os.getcwd(), os.path.dirname(target))
101
102	1	if os.path.islink(target):
103	1	os.remove(target)
104	1	elif os.path.exists(target):
105	1	if force:
106	1	shell.rm(target)
107		else:
108	1	msg = "Preexisting link location at {}".format(target)
109	1	raise exceptions.UncommittedChanges(msg)
110
111	1	shell.ln(source, target)
112
113	1	def run_scripts(self, force=False):
		0 ignored issues – show Coding Style introduced 2017-01-20 02:55 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
114	1	log.info("Running install scripts...")
115
116		# Enter the working tree
117	1	shell.cd(self.name)
118	1	if not git.valid():
119		raise self._invalid_repository
120
121		# Check for scripts
122	1	if not self.scripts:
123	1	common.show("(no scripts to run)", color='shell_info')
124	1	common.newline()
125	1	return
126
127		# Run all scripts
128	1	for script in self.scripts:
129	1	try:
130	1	lines = shell.call(script, _shell=True)
131	1	except exceptions.ShellError as exc:
132	1	common.show(*exc.output, color='shell_error')
133	1	cmd = exc.program
134	1	if force:
135	1	log.debug("Ignored error from call to '%s'", cmd)
136		else:
137	1	msg = "Command '{}' failed in {}".format(cmd, os.getcwd())
138	1	raise exceptions.ScriptFailure(msg)
139		else:
140		common.show(*lines, color='shell_output')
		0 ignored issues – show Coding Style introduced 2017-01-20 02:55 UTC by Report Bug Copy Issue Report Usage of `` or `` arguments should usually be done with care. Generally, there is nothing wrong with usage of `` or `**` arguments. For readability of the code base, we suggest to not over-use these language constructs though. For more information, we can recommend this blog post from Ned Batchelder including its comments which also touches this aspect. Loading history...
141	1	common.newline()
142
143	1	def identify(self, allow_dirty=True, allow_missing=True):
144		"""Get the path and current repository URL and hash."""
145	1	if os.path.isdir(self.name):
146
147	1	shell.cd(self.name)
148	1	if not git.valid():
149	1	raise self._invalid_repository
150
151	1	path = os.getcwd()
152	1	url = git.get_url()
153	1	if git.changes(display_status=not allow_dirty, _show=True):
154	1	if not allow_dirty:
155	1	msg = "Uncommitted changes in {}".format(os.getcwd())
156	1	raise exceptions.UncommittedChanges(msg)
157
158		common.show(self.DIRTY, color='git_dirty', log=False)
159		common.newline()
160		return path, url, self.DIRTY
161		else:
162	1	rev = git.get_hash(_show=True)
163	1	common.show(rev, color='git_rev', log=False)
164	1	common.newline()
165	1	return path, url, rev
166
167	1	elif allow_missing:
168
169	1	return os.getcwd(), '<missing>', self.UNKNOWN
170
171		else:
172
173	1	raise self._invalid_repository
174
175	1	def lock(self, rev=None):
176		"""Return a locked version of the current source."""
177	1	if rev is None:
178	1	_, _, rev = self.identify(allow_dirty=False, allow_missing=False)
179	1	source = self.__class__(self.repo, self.name, rev,
180		self.link, self.scripts)
181	1	return source
182
183	1	@property
184		def _invalid_repository(self):
		0 ignored issues – show Coding Style introduced 2017-01-11 18:37 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
185	1	path = os.path.join(os.getcwd(), self.name)
186	1	msg = "Not a valid repository: {}".format(path)
187		return exceptions.InvalidRepository(msg)
188

Pull Request — develop (#148)

Source.run_scripts() C

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

jacebrowning / gitman

Pull Request — develop (#148)

Source.run_scripts() C

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing Dependencies

2. Missing __init__.py files

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files

2. Missing init.py files