Showing 490 of 907 total issues
Function cleanup
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def cleanup(self):
"""
Cleanup the temporary job handlers (dirs, global setting, ...)
"""
self.__stop_job_logging()
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Avoid deeply nested control flow statements. Open
if parts[1] == self.mountpoint:
return parts[1] # The mountpoint where it's mounted
return None
Avoid deeply nested control flow statements. Open
if m.group(2).upper().startswith("SKIP"):
if count > 0:
yield self.Error("invalid SKIP directive for plan")
skipped = True
else:
Function discover
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def discover(self):
resolutions = []
for ext in self.extensions:
try:
results = ext.obj.discover()
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function find_free_port
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
def find_free_port(
Function _create_connection
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def _create_connection(self):
"""
Creates connection with `self.uri` based on `socket.create_connection`
"""
if ":" in self.uri:
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function start
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def start(self):
"""Log all collectibles at the start of the event."""
os.environ["AVOCADO_SYSINFODIR"] = self.pre_dir
for log_hook in self.start_collectibles:
# log daemons in profile directory
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function end
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def end(self, status=""):
"""
Logging hook called whenever a job finishes.
"""
optimized = self.config.get("sysinfo.collect.optimize")
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Avoid deeply nested control flow statements. Open
if not must_flat.issubset(test_tags):
continue
Function from_avocado_config
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
def from_avocado_config(
Avoid deeply nested control flow statements. Open
if not _must_key_val_matches(
must_key_val, test_tags, include_empty_key
):
continue
Function merge_with_arguments
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def merge_with_arguments(self, arg_parse_config):
"""Merge the current settings with the command-line args.
After parsing argument options this method should be executed to have
an unified settings.
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function __init__
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
def __init__(
Function emit
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def emit(self, record):
try:
msg = self.format(record)
if record.levelno < logging.INFO: # Most messages are INFO
pass
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Function __init__
has 6 arguments (exceeds 4 allowed). Consider refactoring. Open
def __init__(self, kind, uri, *args, config=None, identifier=None, **kwargs):
Function identifier
has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring. Open
def identifier(self):
"""Runnable identifier respecting user's format string.
This is still experimental and we have room for improvements.
- Read upRead up
Cognitive Complexity
Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.
A method's cognitive complexity is based on a few simple rules:
- Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
- Code is considered more complex for each "break in the linear flow of the code"
- Code is considered more complex when "flow breaking structures are nested"
Further reading
Consider simplifying this complex logical expression. Open
if len(line) > 2 and (line[:3] == "+++" or line[:3] == "---"):
continue
# ignore line range information in the output
elif len(line) > 1 and line[:2] == "@@":
continue
Function test_send_signal_sudo_enabled_with_exception
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
def test_send_signal_sudo_enabled_with_exception(
Function test_send_signal_sudo_enabled
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
def test_send_signal_sudo_enabled(self, run, get_children, get_pid, sudo, _):
Function _test
has 5 arguments (exceeds 4 allowed). Consider refactoring. Open
def _test(self, name, content, exp_str, mode=MODE_0664, count=1):