it_requires_a_mapped_object() - Code Metrics - Inspection of "Merge pull request #117 from jacebrowning/feature/..." - jacebrowning/yorm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 10b422...8844b4 )

by Jace

created 2016-04-13 14:05 UTC

it_requires_a_mapped_object() A

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	3
rs	10
cc	2

# pylint: disable=unused-variable,redefined-outer-name,expression-not-assigned,singleton-comparison


import logging
from unittest.mock import Mock

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

import yorm
from yorm import exceptions
from yorm import utilities

log = logging.getLogger(__name__)


@pytest.fixture
def model_class(tmpdir):
    tmpdir.chdir()

    @yorm.sync("data/{self.kind}/{self.key}.yml", auto_create=False)
    class Model:

        def __init__(self, kind, key):
            self.kind = kind
            self.key = key

        def __eq__(self, other):
            return (self.kind, self.key) == (other.kind, other.key)

    return Model


@pytest.fixture
def instance(model_class):
    return model_class('foo', 'bar')


def describe_create():

    def it_creates_files(model_class):
        instance = utilities.create(model_class, 'foo', 'bar')

        expect(instance.__mapper__.exists) == True

    def it_requires_files_to_not_yet_exist(model_class, instance):
        instance.__mapper__.create()

        with expect.raises(exceptions.DuplicateMappingError):
            utilities.create(model_class, 'foo', 'bar')

    def it_requires_a_mapped_object():
        with expect.raises(TypeError):
            utilities.create(Mock)


def describe_find():

    def it_returns_object_when_found(model_class, instance):
        instance.__mapper__.create()

        expect(utilities.find(model_class, 'foo', 'bar')) == instance

    def it_returns_none_when_no_match(model_class):
        expect(utilities.find(model_class, 'not', 'here')) == None

    def it_allows_objects_to_be_created(model_class):
        expect(utilities.find(model_class, 'new', 'one', create=True)) == \
            model_class('new', 'one')

    def it_requires_a_mapped_object():
        with expect.raises(TypeError):
            utilities.find(Mock)


def describe_find_all():

    def it_is_not_yet_implemented():
        with expect.raises(NotImplementedError):
            utilities.find_all(Mock)


def describe_load():

    def it_marks_files_as_unmodified(instance):
        instance.__mapper__.create()
        instance.__mapper__.modified = True

        utilities.load(instance)

        expect(instance.__mapper__.modified) == False

    def it_requires_a_mapped_object():
        with expect.raises(TypeError):
            utilities.load(Mock)


def describe_save():

    def it_creates_files(instance):
        utilities.save(instance)

        expect(instance.__mapper__.exists) == True

    def it_marks_files_as_modified(instance):
        instance.__mapper__.create()
        instance.__mapper__.modified = False

        utilities.save(instance)

        expect(instance.__mapper__.modified) == True

    def it_expects_the_file_to_not_be_deleted(instance):
        instance.__mapper__.delete()

        with expect.raises(exceptions.DeletedFileError):
            utilities.save(instance)

    def it_requires_a_mapped_object():
        with expect.raises(TypeError):
            utilities.save(Mock)


def describe_delete():

    def it_deletes_files(instance):
        utilities.delete(instance)

        expect(instance.__mapper__.exists) == False
        expect(instance.__mapper__.deleted) == True

    def it_requires_a_mapped_object():
        with expect.raises(TypeError):
            utilities.delete(Mock)


1			# pylint: disable=unused-variable,redefined-outer-name,expression-not-assigned,singleton-comparison
			0 ignored issues – show introduced 2016-04-12 20:58 UTC by Report Bug Copy Issue Report Bad option value 'singleton-comparison' Loading history...
2
3			import logging
4			from unittest.mock import Mock
5
6			import pytest
7			from expecter import expect
			0 ignored issues – show Configuration introduced 2016-03-26 01:02 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...
8
9			import yorm
10			from yorm import exceptions
11			from yorm import utilities
12
13			log = logging.getLogger(__name__)
14
15
16			@pytest.fixture
17			def model_class(tmpdir):
18			tmpdir.chdir()
19
20			@yorm.sync("data/{self.kind}/{self.key}.yml", auto_create=False)
21			class Model:
22
23			def __init__(self, kind, key):
24			self.kind = kind
25			self.key = key
26
27			def __eq__(self, other):
28			return (self.kind, self.key) == (other.kind, other.key)
29
30			return Model
31
32
33			@pytest.fixture
34			def instance(model_class):
35			return model_class('foo', 'bar')
36
37
38			def describe_create():
39
40			def it_creates_files(model_class):
41			instance = utilities.create(model_class, 'foo', 'bar')
42
43			expect(instance.__mapper__.exists) == True
44
45			def it_requires_files_to_not_yet_exist(model_class, instance):
46			instance.__mapper__.create()
47
48			with expect.raises(exceptions.DuplicateMappingError):
49			utilities.create(model_class, 'foo', 'bar')
50
51			def it_requires_a_mapped_object():
52			with expect.raises(TypeError):
53			utilities.create(Mock)
54
55
56			def describe_find():
57
58			def it_returns_object_when_found(model_class, instance):
59			instance.__mapper__.create()
60
61			expect(utilities.find(model_class, 'foo', 'bar')) == instance
62
63			def it_returns_none_when_no_match(model_class):
64			expect(utilities.find(model_class, 'not', 'here')) == None
65
66			def it_allows_objects_to_be_created(model_class):
67			expect(utilities.find(model_class, 'new', 'one', create=True)) == \
68			model_class('new', 'one')
69
70			def it_requires_a_mapped_object():
71			with expect.raises(TypeError):
72			utilities.find(Mock)
73
74
75			def describe_find_all():
76
77			def it_is_not_yet_implemented():
78			with expect.raises(NotImplementedError):
79			utilities.find_all(Mock)
80
81
82			def describe_load():
83
84			def it_marks_files_as_unmodified(instance):
85			instance.__mapper__.create()
86			instance.__mapper__.modified = True
87
88			utilities.load(instance)
89
90			expect(instance.__mapper__.modified) == False
91
92			def it_requires_a_mapped_object():
93			with expect.raises(TypeError):
94			utilities.load(Mock)
95
96
97			def describe_save():
98
99			def it_creates_files(instance):
100			utilities.save(instance)
101
102			expect(instance.__mapper__.exists) == True
103
104			def it_marks_files_as_modified(instance):
105			instance.__mapper__.create()
106			instance.__mapper__.modified = False
107
108			utilities.save(instance)
109
110			expect(instance.__mapper__.modified) == True
111
112			def it_expects_the_file_to_not_be_deleted(instance):
113			instance.__mapper__.delete()
114
115			with expect.raises(exceptions.DeletedFileError):
116			utilities.save(instance)
117
118			def it_requires_a_mapped_object():
119			with expect.raises(TypeError):
120			utilities.save(Mock)
121
122
123			def describe_delete():
124
125			def it_deletes_files(instance):
126			utilities.delete(instance)
127
128			expect(instance.__mapper__.exists) == False
129			expect(instance.__mapper__.deleted) == True
130
131			def it_requires_a_mapped_object():
132			with expect.raises(TypeError):
133			utilities.delete(Mock)
134

jacebrowning / yorm

Push — develop ( 10b422...8844b4 )

it_requires_a_mapped_object() A

Complexity

Size

Duplication

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files