gdm.test.it_opens_files() - Code Metrics - Inspection of "Add tests for platform detection" - jacebrowning/gdm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( c9c7a4...e6cfa4 )

by Jace

created 2016-02-28 00:31 UTC

gdm.test.it_opens_files() A

↳ Parent: gdm.test.describe_launch()

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Metric	Value
cc	1
dl	0
loc	5
rs	9.4285

# pylint: disable=unused-variable,expression-not-assigned
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

from unittest.mock import patch, call, Mock

from expecter import expect
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

from gdm import system


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

    @patch('platform.system', Mock(return_value="Windows"))
    @patch('gdm.system._launch_windows')
    def it_opens_files(startfile):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        system.launch("fake/path")
        expect(startfile.mock_calls) == [call("fake/path")]

    @patch('platform.system', Mock(return_value="fake"))
    def it_raises_an_exception_when_platform_is_unknown():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        with expect.raises(RuntimeError):
            system.launch(None)


Push — develop ( c9c7a4...e6cfa4 )

gdm.test.it_opens_files() A

Complexity

Size

Duplication

1. Missing Dependencies

2. Missing init.py files

1			# pylint: disable=unused-variable,expression-not-assigned
			0 ignored issues – show Coding Style introduced 2016-02-28 00:33 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			from unittest.mock import patch, call, Mock
4
5			from expecter import expect
			0 ignored issues – show Configuration introduced 2016-02-28 00:33 UTC by Report Bug Copy Issue Report The import `expecter` 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...
6
7			from gdm import system
8
9
10			def describe_launch():
			0 ignored issues – show Coding Style introduced 2016-02-28 00:33 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...
11
12			@patch('platform.system', Mock(return_value="Windows"))
13			@patch('gdm.system._launch_windows')
14			def it_opens_files(startfile):
			0 ignored issues – show Coding Style introduced 2016-02-28 00:33 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...
15			system.launch("fake/path")
16			expect(startfile.mock_calls) == [call("fake/path")]
17
18			@patch('platform.system', Mock(return_value="fake"))
19			def it_raises_an_exception_when_platform_is_unknown():
			0 ignored issues – show Coding Style introduced 2016-02-28 00:33 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...
20			with expect.raises(RuntimeError):
21			system.launch(None)
22

jacebrowning / gdm

Push — develop ( c9c7a4...e6cfa4 )

gdm.test.it_opens_files() A

Complexity

Size

Duplication

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files