test_debug_turned_off() - Code Metrics - Inspection of "Testing no debug" - euri10/gols - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 7fc4fe...8068d1 )

by benoit

created 2017-09-07 05:05 UTC

test_debug_turned_off() A

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
dl	0
loc	5
rs	9.2
c	1
b	0
f	0

import os
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

import pytest

from click.testing import CliRunner

from testfixtures import LogCapture


from gols.cli import main

from gols.cli import upload



@pytest.fixture(scope='function')
def runner():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    yield CliRunner()


@pytest.fixture(scope='function')

def fs():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    if not os.path.exists(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'directory_fit')):

        os.makedirs(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'directory_fit'))

    yield os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'directory_fit')


@pytest.fixture(scope='function')

def cdf():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    if not os.path.exists(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'conf_dir_fit')):

        os.makedirs(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'conf_dir_fit'))

    yield os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'conf_dir_fit')


def test_debug_turned_on(runner):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    with LogCapture() as l:

        result = runner.invoke(main, ['--debug', 'upload'])
        assert result.exit_code == 2
        assert 'Debug level set on' in str(l)


def test_debug_turned_off(runner):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    with LogCapture() as l:

        result = runner.invoke(main, ['upload'])
        assert result.exit_code == 2
        assert 'Info level set on' in str(l)


def test_upload_help(runner):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    result = runner.invoke(upload, ['--help'])
    assert result.exit_code == 0
    assert 'Usage' in result.output


def test_required_fit_directory(runner, fs):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    result = runner.invoke(upload, ['upload', fs])
    assert result.exit_code == 2
    assert 'Error: Missing option' in result.output


def test_upload_fit(runner, fs, cdf):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    username = '[email protected]'
    password = 'G0lsG0ls'
    with LogCapture() as l:

        # logger = logging.getLogger()
        result = runner.invoke(main, ['--debug', 'upload', '-d', fs, '-c', cdf, '-u', username, '-p', password])  # noqa

        print(result.output)
        print(l)
        assert result.exit_code == 0
        assert 'Done uploading' in str(l)


1		import os
		0 ignored issues – show Coding Style introduced 2017-09-06 08:41 UTC by Report Bug Copy Issue Report This module 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...
2
3		import pytest
		0 ignored issues – show introduced 2017-09-06 08:41 UTC by Report Bug Copy Issue Report Unable to import 'pytest' Loading history...
4		from click.testing import CliRunner
		0 ignored issues – show introduced 2017-09-06 08:41 UTC by Report Bug Copy Issue Report Unable to import 'click.testing' Loading history...
5		from testfixtures import LogCapture
		0 ignored issues – show introduced 2017-09-06 08:41 UTC by Report Bug Copy Issue Report Unable to import 'testfixtures' Loading history...
6
7		from gols.cli import main
		0 ignored issues – show introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report Unable to import 'gols.cli' Loading history...
8		from gols.cli import upload
		0 ignored issues – show introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report Unable to import 'gols.cli' Loading history...
9
10
11		@pytest.fixture(scope='function')
12		def runner():
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This function 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...
13		yield CliRunner()
14
15
16	View Code Duplication	@pytest.fixture(scope='function')
		0 ignored issues – show Duplication introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
17		def fs():
		0 ignored issues – show Coding Style Naming introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report The name `fs` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[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... Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This function 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...
18		if not os.path.exists(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'directory_fit')):
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (110/100). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
19		os.makedirs(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'directory_fit'))
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (103/100). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
20		yield os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'directory_fit')
21
22
23	View Code Duplication	@pytest.fixture(scope='function')
		0 ignored issues – show Duplication introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
24		def cdf():
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This function 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...
25		if not os.path.exists(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'conf_dir_fit')):
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (109/100). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
26		os.makedirs(os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'conf_dir_fit'))
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (102/100). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
27		yield os.path.join(os.path.dirname(os.path.dirname(__file__)), 'tests', 'conf_dir_fit')
28
29
30		def test_debug_turned_on(runner):
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This function 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... Comprehensibility Bug introduced 2017-09-06 17:20 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 `12`). 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...
31		with LogCapture() as l:
		0 ignored issues – show Coding Style Naming introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report The name `l` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[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...
32		result = runner.invoke(main, ['--debug', 'upload'])
33		assert result.exit_code == 2
34		assert 'Debug level set on' in str(l)
35
36
37		def test_debug_turned_off(runner):
		0 ignored issues – show Coding Style introduced 2017-09-07 05:06 UTC by Report Bug Copy Issue Report This function 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... Comprehensibility Bug introduced 2017-09-07 05:06 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 `12`). 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...
38		with LogCapture() as l:
		0 ignored issues – show Coding Style Naming introduced 2017-09-07 05:06 UTC by Report Bug Copy Issue Report The name `l` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[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...
39		result = runner.invoke(main, ['upload'])
40		assert result.exit_code == 2
41		assert 'Info level set on' in str(l)
42
43
44		def test_upload_help(runner):
		0 ignored issues – show Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This function 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... Comprehensibility Bug introduced 2017-09-06 17:20 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 `12`). 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...
45		result = runner.invoke(upload, ['--help'])
46		assert result.exit_code == 0
47		assert 'Usage' in result.output
48
49
50		def test_required_fit_directory(runner, fs):
		0 ignored issues – show Coding Style Naming introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report The name `fs` does not conform to the argument naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[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... Coding Style introduced 2017-09-06 08:41 UTC by Report Bug Copy Issue Report This function 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... Comprehensibility Bug introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report `fs` is re-defining a name which is already available in the outer-scope (previously defined on line `17`). 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... Comprehensibility Bug introduced 2017-09-06 17:20 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 `12`). 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...
51		result = runner.invoke(upload, ['upload', fs])
52		assert result.exit_code == 2
53		assert 'Error: Missing option' in result.output
54
55
56		def test_upload_fit(runner, fs, cdf):
		0 ignored issues – show Coding Style Naming introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report The name `fs` does not conform to the argument naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[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... Coding Style introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report This function 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... Comprehensibility Bug introduced 2017-09-06 17:20 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 `12`). 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... Comprehensibility Bug introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report `fs` is re-defining a name which is already available in the outer-scope (previously defined on line `17`). 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... Comprehensibility Bug introduced 2017-09-06 17:20 UTC by Report Bug Copy Issue Report `cdf` is re-defining a name which is already available in the outer-scope (previously defined on line `24`). 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...
57		username = '[email protected]'
58		password = 'G0lsG0ls'
59		with LogCapture() as l:
		0 ignored issues – show Coding Style Naming introduced 2017-09-06 08:41 UTC by Report Bug Copy Issue Report The name `l` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[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...
60		# logger = logging.getLogger()
61		result = runner.invoke(main, ['--debug', 'upload', '-d', fs, '-c', cdf, '-u', username, '-p', password]) # noqa
		0 ignored issues – show Coding Style introduced 2017-09-07 03:59 UTC by Report Bug Copy Issue Report This line is too long as per the coding-style (120/100). This check looks for lines that are too long. You can specify the maximum line length. Loading history...
62		print(result.output)
63		print(l)
64		assert result.exit_code == 0
65		assert 'Done uploading' in str(l)
66

euri10 / gols

Push — master ( 7fc4fe...8068d1 )

test_debug_turned_off() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like