test_tabular_operators.test_tabular_interfaces() - Code Metrics - Inspection of "Mpeta" - boromir674/so-magic - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — dev (#30)

by Konstantinos

created 2021-05-28 16:18 UTC

test_tabular_operators.test_tabular_interfaces() B

↳ Parent: test_tabular_operators

Complexity

Conditions

Size

Total Lines	54
Code Lines	39

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
eloc	39
nop	2
dl	0
loc	54
rs	8.4773
c	0
b	0
f	0

How to fix Long Method

import pytest
import inspect


@pytest.fixture
def tabular_operators():
    from so_magic.data.backend.panda_handling.df_backend import PDTabularRetriever, PDTabularIterator, PDTabularMutator
    return {
        'retriever': PDTabularRetriever,
        'iterator': PDTabularIterator,
        'mutator': PDTabularMutator,
    }


@pytest.fixture
def tabular_interfaces():
    from so_magic.data.backend.engine_specs import EngineTabularRetriever, EngineTabularIterator, EngineTabularMutator
    return {
        'retriever': EngineTabularRetriever,
        'iterator': EngineTabularIterator,
        'mutator': EngineTabularMutator,
    }


def test_tabular_interfaces(tabular_interfaces, tabular_operators):
    expected_contract = {
        'retriever': ['column', 'row', 'nb_columns', 'nb_rows', 'get_numerical_attributes'],
        'iterator': ['columnnames', 'itercolumns', 'iterrows'],
    }

    retr_members = ['column', 'row', 'nb_columns', 'nb_rows', 'get_numerical_attributes']

    

    pd_tabular_retriever1 = tabular_operators['retriever']()
    pd_tabular_retriever2 = tabular_operators['retriever']()

    member1 = list(x[0] for x in inspect.getmembers(pd_tabular_retriever1))
    member2 = list(x[0] for x in inspect.getmembers(pd_tabular_retriever2))

    for exp_member in retr_members:
        assert exp_member in member1
        assert exp_member in member2

    class A:
        def go(self, x):
            return x + 1
        @staticmethod
        def ela(y):
            return y - 1
        @classmethod
        def pame(cls, z):
            return z * 2
    
    a = A()

    go_sig = inspect.signature(a.go)
    ela_sig = inspect.signature(a.ela)
    pame_sig = inspect.signature(a.pame)

    assert str(go_sig) == '(x)'
    assert str(ela_sig) == '(y)'
    assert str(pame_sig) == '(z)'
    
    for method in retr_members[:2]:
        sig = inspect.signature(getattr(pd_tabular_retriever1, method))
        assert str(sig) == '(identifier, data)'

    for method in retr_members[2:]:
        sig = inspect.signature(getattr(pd_tabular_retriever1, method))
        assert str(sig) == '(data)'

    assert id(pd_tabular_retriever1._delegate) != id(pd_tabular_retriever2._delegate) 

    for function in expected_contract['retriever']:
        assert function in dir(pd_tabular_retriever1)
        assert id(getattr(pd_tabular_retriever1, function)) != id(getattr(pd_tabular_retriever2, function))
        assert id(getattr(pd_tabular_retriever1._delegate, function)) != id(getattr(pd_tabular_retriever2._delegate, function))

    # pd_tabular_iterator1 = tabular_operators['iterator']()
    # pd_tabular_iterator2 = tabular_operators['iterator']()
    # assert id(pd_tabular_iterator1._delegate) != id(pd_tabular_iterator2._delegate)
    # for f in expected_contract['iterator']:
    #     assert f in dir(pd_tabular_iterator1)
    #     assert f in dir(pd_tabular_iterator2)
    #     assert id(getattr(pd_tabular_iterator1, f)) != id(getattr(pd_tabular_iterator2, f))
    #     assert id(getattr(pd_tabular_iterator1._delegate, f)) != id(getattr(pd_tabular_iterator2._delegate, f))

1			import pytest
2			import inspect
3
4
5			@pytest.fixture
6			def tabular_operators():
7			from so_magic.data.backend.panda_handling.df_backend import PDTabularRetriever, PDTabularIterator, PDTabularMutator
8			return {
9			'retriever': PDTabularRetriever,
10			'iterator': PDTabularIterator,
11			'mutator': PDTabularMutator,
12			}
13
14
15			@pytest.fixture
16			def tabular_interfaces():
17			from so_magic.data.backend.engine_specs import EngineTabularRetriever, EngineTabularIterator, EngineTabularMutator
18			return {
19			'retriever': EngineTabularRetriever,
20			'iterator': EngineTabularIterator,
21			'mutator': EngineTabularMutator,
22			}
23
24
25			def test_tabular_interfaces(tabular_interfaces, tabular_operators):
26			expected_contract = {
27			'retriever': ['column', 'row', 'nb_columns', 'nb_rows', 'get_numerical_attributes'],
28			'iterator': ['columnnames', 'itercolumns', 'iterrows'],
29			}
30
31			retr_members = ['column', 'row', 'nb_columns', 'nb_rows', 'get_numerical_attributes']
32
33
34
35			pd_tabular_retriever1 = tabular_operators['retriever']()
36			pd_tabular_retriever2 = tabular_operators['retriever']()
37
38			member1 = list(x[0] for x in inspect.getmembers(pd_tabular_retriever1))
39			member2 = list(x[0] for x in inspect.getmembers(pd_tabular_retriever2))
40
41			for exp_member in retr_members:
42			assert exp_member in member1
43			assert exp_member in member2
44
45			class A:
46			def go(self, x):
47			return x + 1
48			@staticmethod
49			def ela(y):
50			return y - 1
51			@classmethod
52			def pame(cls, z):
53			return z * 2
54
55			a = A()
56
57			go_sig = inspect.signature(a.go)
58			ela_sig = inspect.signature(a.ela)
59			pame_sig = inspect.signature(a.pame)
60
61			assert str(go_sig) == '(x)'
62			assert str(ela_sig) == '(y)'
63			assert str(pame_sig) == '(z)'
64
65			for method in retr_members[:2]:
66			sig = inspect.signature(getattr(pd_tabular_retriever1, method))
67			assert str(sig) == '(identifier, data)'
68
69			for method in retr_members[2:]:
70			sig = inspect.signature(getattr(pd_tabular_retriever1, method))
71			assert str(sig) == '(data)'
72
73			assert id(pd_tabular_retriever1._delegate) != id(pd_tabular_retriever2._delegate)
74
75			for function in expected_contract['retriever']:
76			assert function in dir(pd_tabular_retriever1)
77			assert id(getattr(pd_tabular_retriever1, function)) != id(getattr(pd_tabular_retriever2, function))
78			assert id(getattr(pd_tabular_retriever1._delegate, function)) != id(getattr(pd_tabular_retriever2._delegate, function))
79
80			# pd_tabular_iterator1 = tabular_operators['iterator']()
81			# pd_tabular_iterator2 = tabular_operators['iterator']()
82			# assert id(pd_tabular_iterator1._delegate) != id(pd_tabular_iterator2._delegate)
83			# for f in expected_contract['iterator']:
84			# assert f in dir(pd_tabular_iterator1)
85			# assert f in dir(pd_tabular_iterator2)
86			# assert id(getattr(pd_tabular_iterator1, f)) != id(getattr(pd_tabular_iterator2, f))
87			# assert id(getattr(pd_tabular_iterator1._delegate, f)) != id(getattr(pd_tabular_iterator2._delegate, f))

boromir674 / so-magic

Pull Request — dev (#30)

test_tabular_operators.test_tabular_interfaces() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like