test_merged_pull_requests() - Code Metrics - Inspection of "Merge pull request #117 from michaeljoseph/init" - michaeljoseph/changes - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 40bfaf...b38ec2 )

by Michael

created 2017-10-15 21:20 UTC

test_merged_pull_requests() B

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	6
Bugs	0	Features	0

Metric	Value
cc	6
c	6
b	0
f	0
dl	0
loc	29
rs	7.5384

import responses
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3
from semantic_version import Version
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

from changes import models


def test_repository_parses_remote_url(git_repo):

    repository = models.GitRepository()
    assert 'test_app' == repository.repo
    assert 'michaeljoseph' == repository.owner


@responses.activate
def test_merged_pull_requests(git_repo_with_merge_commit):
    responses.add(
        responses.GET,
        'https://api.github.com/repos/michaeljoseph/test_app/pulls/111',
        json={
            'title': 'The title of the pull request',
            'body': 'An optional, longer description.',
            'user': {
                'login': 'someone'
            },
            'labels': [
                {'id': 1, 'name': 'feature'}
            ],
        },
        status=200,
        content_type='application/json'
    )

    repository = models.GitRepository()
    assert 1 == len(repository.pull_requests)

    first_pull_request = repository.pull_requests[0]
    assert '111' == first_pull_request.number
    assert ['feature'] == first_pull_request.labels

    assert [] == repository.versions

    assert Version('0.0.0') == repository.latest_version


Push — master ( 40bfaf...b38ec2 )

test_merged_pull_requests() B

Complexity

Size

Duplication

Importance

1. Missing Dependencies

2. Missing init.py files

1. Missing Dependencies

2. Missing init.py files

1			import responses
			0 ignored issues – show Configuration introduced 2017-10-15 11:55 UTC by Report Bug Copy Issue Report The import `responses` 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...
2			from semantic_version import Version
			0 ignored issues – show Configuration introduced 2017-10-15 09:45 UTC by Report Bug Copy Issue Report The import `semantic_version` 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...
3
4			from changes import models
5
6
7			def test_repository_parses_remote_url(git_repo):
			0 ignored issues – show Coding Style Naming introduced 2017-10-14 17:35 UTC by Report Bug Copy Issue Report The name `test_repository_parses_remote_url` does not conform to the function naming conventions (`[a-z_][a-z0-9_]{2,30}$`). 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...
8			repository = models.GitRepository()
9			assert 'test_app' == repository.repo
10			assert 'michaeljoseph' == repository.owner
11
12
13			@responses.activate
14			def test_merged_pull_requests(git_repo_with_merge_commit):
15			responses.add(
16			responses.GET,
17			'https://api.github.com/repos/michaeljoseph/test_app/pulls/111',
18			json={
19			'title': 'The title of the pull request',
20			'body': 'An optional, longer description.',
21			'user': {
22			'login': 'someone'
23			},
24			'labels': [
25			{'id': 1, 'name': 'feature'}
26			],
27			},
28			status=200,
29			content_type='application/json'
30			)
31
32			repository = models.GitRepository()
33			assert 1 == len(repository.pull_requests)
34
35			first_pull_request = repository.pull_requests[0]
36			assert '111' == first_pull_request.number
37			assert ['feature'] == first_pull_request.labels
38
39			assert [] == repository.versions
40
41			assert Version('0.0.0') == repository.latest_version
42

michaeljoseph / changes

Push — master ( 40bfaf...b38ec2 )

test_merged_pull_requests() B

Complexity

Size

Duplication

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