tests.test_main.test_main_succeeds() - Code Metrics - Inspection of "configurations updated" - johnpatrickroach/trending-homebrew - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — main ( b75c67...4e8f49 )

by John Patrick

created 2023-01-03 22:29 UTC

tests.test_main.test_main_succeeds() A

↳ Parent: tests.test_main

Complexity

Conditions

Size

Total Lines	4
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	3
nop	1
dl	0
loc	4
rs	10
c	0
b	0
f	0

"""Test cases for the __main__ module."""
import pytest

from click.testing import CliRunner


from trending_homebrew import __main__



@pytest.fixture
def runner() -> CliRunner:
    """Fixture for invoking command-line interfaces."""
    return CliRunner()


def test_main_succeeds(runner: CliRunner) -> None:
param = 5

class Foo:
    def __init__(self, param):   # "param" would be flagged here
        self.param = param
    """It exits with a status code of zero."""
    result = runner.invoke(__main__.main)
    assert result.exit_code == 0


1			"""Test cases for the __main__ module."""
2			import pytest
			0 ignored issues – show introduced 2023-01-03 22:31 UTC by Report Bug Copy Issue Report Unable to import 'pytest' Loading history...
3			from click.testing import CliRunner
			0 ignored issues – show introduced 2023-01-03 22:31 UTC by Report Bug Copy Issue Report Unable to import 'click.testing' Loading history...
4
5			from trending_homebrew import __main__
			0 ignored issues – show introduced 2023-01-03 22:31 UTC by Report Bug Copy Issue Report Unable to import 'trending_homebrew' Loading history...
6
7
8			@pytest.fixture
9			def runner() -> CliRunner:
10			"""Fixture for invoking command-line interfaces."""
11			return CliRunner()
12
13
14			def test_main_succeeds(runner: CliRunner) -> None:
			0 ignored issues – show Comprehensibility Bug introduced 2023-01-03 22:31 UTC by Report Bug Copy Issue Report `runner` is re-defining a name which is already available in the outer-scope (previously defined on line `9`). It is generally a bad practice to shadow variables from the outer-scope. In most cases, this is done unintentionally and might lead to unexpected behavior: param = 5 class Foo: def __init__(self, param): # "param" would be flagged here self.param = param Loading history...
15			"""It exits with a status code of zero."""
16			result = runner.invoke(__main__.main)
17			assert result.exit_code == 0
18

johnpatrickroach / trending-homebrew

Push — main ( b75c67...4e8f49 )

tests.test_main.test_main_succeeds() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like