it_requires_a_mapped_object() - Code Metrics - Inspection of "Allow create and find to be called with instances" - jacebrowning/yorm - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( f1f88a...e832cb )

by Jace

created 2016-04-13 22:11 UTC

it_requires_a_mapped_object() A

↳ Parent: it_can_overwrite_files()

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_can_overwrite_files(model_class, instance):
        instance.__mapper__.create()

        utilities.create(model_class, 'foo', 'bar', overwrite=True)

    def it_can_also_be_called_with_an_instance(instance):
        expect(yorm.create(instance)) == instance

    def it_requires_a_mapped_class_or_instance():
        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_can_also_be_called_with_an_instance(instance):
        expect(yorm.find(instance, create=True)) == instance

    def it_requires_a_mapped_class_or_instance():
        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_instance():
        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_instance():
        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_instance():
        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	View Code Duplication	def describe_create():
		0 ignored issues – show Duplication introduced 2016-04-13 22:13 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
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_can_overwrite_files(model_class, instance):
52		instance.__mapper__.create()
53
54		utilities.create(model_class, 'foo', 'bar', overwrite=True)
55
56		def it_can_also_be_called_with_an_instance(instance):
57		expect(yorm.create(instance)) == instance
58
59		def it_requires_a_mapped_class_or_instance():
60		with expect.raises(TypeError):
61		utilities.create(Mock)
62
63
64		def describe_find():
65
66		def it_returns_object_when_found(model_class, instance):
67		instance.__mapper__.create()
68
69		expect(utilities.find(model_class, 'foo', 'bar')) == instance
70
71		def it_returns_none_when_no_match(model_class):
72		expect(utilities.find(model_class, 'not', 'here')) == None
73
74		def it_allows_objects_to_be_created(model_class):
75		expect(utilities.find(model_class, 'new', 'one', create=True)) == \
76		model_class('new', 'one')
77
78		def it_can_also_be_called_with_an_instance(instance):
79		expect(yorm.find(instance, create=True)) == instance
80
81		def it_requires_a_mapped_class_or_instance():
82		with expect.raises(TypeError):
83		utilities.find(Mock)
84
85
86		def describe_find_all():
87
88		def it_is_not_yet_implemented():
89		with expect.raises(NotImplementedError):
90		utilities.find_all(Mock)
91
92
93		def describe_load():
94
95		def it_marks_files_as_unmodified(instance):
96		instance.__mapper__.create()
97		instance.__mapper__.modified = True
98
99		utilities.load(instance)
100
101		expect(instance.__mapper__.modified) == False
102
103		def it_requires_a_mapped_instance():
104		with expect.raises(TypeError):
105		utilities.load(Mock)
106
107
108	View Code Duplication	def describe_save():
		0 ignored issues – show Duplication introduced 2016-04-13 22:13 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
109
110		def it_creates_files(instance):
111		utilities.save(instance)
112
113		expect(instance.__mapper__.exists) == True
114
115		def it_marks_files_as_modified(instance):
116		instance.__mapper__.create()
117		instance.__mapper__.modified = False
118
119		utilities.save(instance)
120
121		expect(instance.__mapper__.modified) == True
122
123		def it_expects_the_file_to_not_be_deleted(instance):
124		instance.__mapper__.delete()
125
126		with expect.raises(exceptions.DeletedFileError):
127		utilities.save(instance)
128
129		def it_requires_a_mapped_instance():
130		with expect.raises(TypeError):
131		utilities.save(Mock)
132
133
134		def describe_delete():
135
136		def it_deletes_files(instance):
137		utilities.delete(instance)
138
139		expect(instance.__mapper__.exists) == False
140		expect(instance.__mapper__.deleted) == True
141
142		def it_requires_a_mapped_instance():
143		with expect.raises(TypeError):
144		utilities.delete(Mock)
145

jacebrowning / yorm

Push — develop ( f1f88a...e832cb )

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