0LL13 / person

personroles/mop_role.py

#!/usr/bin/env python
# -*- coding: utf-8 -*-
"""A set of dataclasses concerning roles of persons and their particulars."""

import os
import sys
from dataclasses import dataclass, field
from typing import List, Set

PACKAGE_PARENT = ".."
SCRIPT_DIR = os.path.dirname(
    os.path.realpath(os.path.join(os.getcwd(), os.path.expanduser(__file__)))
)  # isort:skip # noqa # pylint: disable=wrong-import-position
sys.path.append(
    os.path.normpath(os.path.join(SCRIPT_DIR, PACKAGE_PARENT))
)  # isort: skip # noqa # pylint: disable=wrong-import-position

from personroles.politician_role import Politician  # type: ignore  # noqa
from personroles.resources.helpers import AttrDisplay  # type: ignore # noqa
from personroles.resources.helpers import NotInRange  # type: ignore # noqa
from personroles.resources.mop_tinyDB import MopsDB  # type: ignore # noqa


@dataclass

Class name "_MoP_default" doesn't conform to PascalCase naming style ('[^\\W\\da-z][^\\W_]+$' pattern)

Too many instance attributes (8/7)

class _MoP_default:
    key: str = field(default="")  # noqa
    parl_pres: bool = field(default=False)
    parl_vicePres: bool = field(default=False)

Attribute name "parl_vicePres" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,|_[^\\WA-Z]*|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern)

    parliament_entry: str = field(default="unknown")  # date string: "11.3.2015"  # noqa
    parliament_exit: str = field(default="unknown")  # dto.
    speeches: List[str] = field(
        default_factory=lambda: []
    )  # identifiers for speeches  # noqa
    reactions: List[str] = field(
        default_factory=lambda: []
    )  # identifiers for reactions
    membership: Set[str] = field(
        default_factory=lambda: set()

This lambda might be unnecessary.

    )  # years like ["2010", "2011", ...]


@dataclass

Class name "_MoP_base" doesn't conform to PascalCase naming style ('[^\\W\\da-z][^\\W_]+$' pattern)

class _MoP_base:
    legislature: str
    state: str


@dataclass

Too many ancestors (15/7)

class MoP(_MoP_default, Politician, _MoP_base, AttrDisplay):

    """
    Module mop_role.py covers the role as member of parliament. The role
    integrates the role of politician and adds a federal state (like "NRW" or
    "BY") and legislature (legislative term) as obligatory informations to
    define the role. More informations like speeches held or offices (like
    president) filled can be added. Call politician's __post_init__ to
    initialize wards and voters.
    """

    def __post_init__(self):
        """
        Check if legislature is correct for NRW and add legislature into the
        mop's list of memberships (in case more than one term is spent in
        parliament.
        """
        if int(self.legislature) not in range(14, 18):

Unnecessary "else" after "raise"

            raise NotInRange("Number for legislature not in range")
        else:
            self.membership.add(self.legislature)
        Politician.__post_init__(self)


if __name__ == "__main__":

    mop = MoP(
        "14",
        "NRW",
        "SPD",  # type: ignore
        "Tom",  # type: ignore
        "Schwadronius",
        party_entry="1990",  # type: ignore
        peer_title="Junker von",
        date_of_birth="1950",
    )
    print(mop)

    mop.add_Party("Grüne", party_entry="30.11.1999")
    mop.change_ward("Düsseldorf II")
    print(mop)

    print(mop.__dict__)

    mop_db = MopsDB(".")
    mop_key = mop_db.join(mop)
    print(mop_db.fetch(mop_key))