Source._infer_name() - Code Metrics - Inspection of "Allow 'link' setting to be blank" - jacebrowning/gitman - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( eee1b7...9b8aaf )

by Jace

created 2017-01-24 22:08 UTC

Source._infer_name() A

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
c	1
b	0
f	0
dl	0
loc	5
ccs	4
cts	4
cp	1
crap	1
rs	9.4285

"""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, NullableString, 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=NullableString)
@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=None, rev='master', link=None, scripts=None):

        super().__init__()
        self.repo = repo
        self.name = self._infer_name(repo) if name is None else name
        self.rev = rev
        self.link = link
        self.scripts = scripts or []

        for key in ['name', 'repo', 'rev']:
            if not self[key]:
                msg = "'{}' required for {}".format(key, 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)

    @staticmethod
    def _infer_name(repo):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        filename = repo.split('/')[-1]
        name = filename.split('.')[0]
        return name


Push — develop ( eee1b7...9b8aaf )

Source._infer_name() A

Complexity

Size

Duplication

Code Coverage

Importance

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

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, NullableString, 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=NullableString)
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=None, 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 = self._infer_name(repo) if name is None else name
31	1	self.rev = rev
32	1	self.link = link
33	1	self.scripts = scripts or []
34
35	1	for key in ['name', 'repo', 'rev']:
36	1	if not self[key]:
37	1	msg = "'{}' required for {}".format(key, repr(self))
38	1	raise exceptions.InvalidConfig(msg)
39
40	1	def __repr__(self):
41	1	return "<source {}>".format(self)
42
43	1	def __str__(self):
44	1	pattern = "'{r}' @ '{v}' in '{d}'"
45	1	if self.link:
46	1	pattern += " <- '{s}'"
47	1	return pattern.format(r=self.repo, v=self.rev, d=self.name, s=self.link)
48
49	1	def __eq__(self, other):
50	1	return self.name == other.name
51
52	1	def __ne__(self, other):
53	1	return self.name != other.name
54
55	1	def __lt__(self, other):
56	1	return self.name < other.name
57
58	1	def update_files(self, force=False, fetch=False, clean=True):
59		"""Ensure the source matches the specified revision."""
60	1	log.info("Updating source files...")
61
62		# Clone the repository if needed
63	1	if not os.path.exists(self.name):
64	1	git.clone(self.repo, self.name)
65
66		# Enter the working tree
67	1	shell.cd(self.name)
68	1	if not git.valid():
69	1	raise self._invalid_repository
70
71		# Check for uncommitted changes
72	1	if not force:
73	1	log.debug("Confirming there are no uncommitted changes...")
74	1	if git.changes(include_untracked=clean):
75		msg = "Uncommitted changes in {}".format(os.getcwd())
76		raise exceptions.UncommittedChanges(msg)
77
78		# Fetch the desired revision
79	1	if fetch or self.rev not in (git.get_branch(),
80		git.get_hash(),
81		git.get_tag()):
82	1	git.fetch(self.repo, self.rev)
83
84		# Update the working tree to the desired revision
85	1	git.update(self.rev, fetch=fetch, clean=clean)
86
87	1	def create_link(self, root, force=False):
88		"""Create a link from the target name to the current directory."""
89	1	if not self.link:
90	1	return
91
92	1	log.info("Creating a symbolic link...")
93
94	1	if os.name == 'nt':
95		warnings.warn("Symbolic links are not supported on Windows")
96		return
97
98	1	target = os.path.join(root, self.link)
99	1	source = os.path.relpath(os.getcwd(), os.path.dirname(target))
100
101	1	if os.path.islink(target):
102	1	os.remove(target)
103	1	elif os.path.exists(target):
104	1	if force:
105	1	shell.rm(target)
106		else:
107	1	msg = "Preexisting link location at {}".format(target)
108	1	raise exceptions.UncommittedChanges(msg)
109
110	1	shell.ln(source, target)
111
112	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...
113	1	log.info("Running install scripts...")
114
115		# Enter the working tree
116	1	shell.cd(self.name)
117	1	if not git.valid():
118		raise self._invalid_repository
119
120		# Check for scripts
121	1	if not self.scripts:
122	1	common.show("(no scripts to run)", color='shell_info')
123	1	common.newline()
124	1	return
125
126		# Run all scripts
127	1	for script in self.scripts:
128	1	try:
129	1	lines = shell.call(script, _shell=True)
130	1	except exceptions.ShellError as exc:
131	1	common.show(*exc.output, color='shell_error')
132	1	cmd = exc.program
133	1	if force:
134	1	log.debug("Ignored error from call to '%s'", cmd)
135		else:
136	1	msg = "Command '{}' failed in {}".format(cmd, os.getcwd())
137	1	raise exceptions.ScriptFailure(msg)
138		else:
139		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...
140	1	common.newline()
141
142	1	def identify(self, allow_dirty=True, allow_missing=True):
143		"""Get the path and current repository URL and hash."""
144	1	if os.path.isdir(self.name):
145
146	1	shell.cd(self.name)
147	1	if not git.valid():
148	1	raise self._invalid_repository
149
150	1	path = os.getcwd()
151	1	url = git.get_url()
152	1	if git.changes(display_status=not allow_dirty, _show=True):
153	1	if not allow_dirty:
154	1	msg = "Uncommitted changes in {}".format(os.getcwd())
155	1	raise exceptions.UncommittedChanges(msg)
156
157		common.show(self.DIRTY, color='git_dirty', log=False)
158		common.newline()
159		return path, url, self.DIRTY
160		else:
161	1	rev = git.get_hash(_show=True)
162	1	common.show(rev, color='git_rev', log=False)
163	1	common.newline()
164	1	return path, url, rev
165
166	1	elif allow_missing:
167
168	1	return os.getcwd(), '<missing>', self.UNKNOWN
169
170		else:
171
172	1	raise self._invalid_repository
173
174	1	def lock(self, rev=None):
175		"""Return a locked version of the current source."""
176	1	if rev is None:
177	1	_, _, rev = self.identify(allow_dirty=False, allow_missing=False)
178	1	source = self.__class__(self.repo, self.name, rev,
179		self.link, self.scripts)
180	1	return source
181
182	1	@property
183		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...
184	1	path = os.path.join(os.getcwd(), self.name)
185	1	msg = "Not a valid repository: {}".format(path)
186	1	return exceptions.InvalidRepository(msg)
187
188	1	@staticmethod
189		def _infer_name(repo):
		0 ignored issues – show Coding Style introduced 2017-01-24 22:11 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...
190	1	filename = repo.split('/')[-1]
191	1	name = filename.split('.')[0]
192		return name
193

jacebrowning / gitman

Push — develop ( eee1b7...9b8aaf )

Source._infer_name() A

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