Refactoring tests for readability

Author: kyoto7250

tests/functional/m/method_hidden.py

@@ -1,10 +1,9 @@
 # pylint: disable=too-few-public-methods,missing-docstring
 # pylint: disable=unused-private-member
-# pylint: disable=import-error
 """check method hiding ancestor attribute
 """
 import functools as ft
-import something_else as functools
+import something_else as functools  # pylint: disable=import-error
 
 
 class Abcd:

tests/functional/m/method_hidden.txt

@@ -1,3 +1,3 @@
-method-hidden:20:4:20:12:Cdef.abcd:An attribute defined in functional.m.method_hidden line 14 hides this method:UNDEFINED
-method-hidden:88:4:88:11:One.one:An attribute defined in functional.m.method_hidden line 86 hides this method:UNDEFINED
-method-hidden:116:4:116:18:Child._protected:An attribute defined in functional.m.method_hidden line 111 hides this method:UNDEFINED
+method-hidden:19:4:19:12:Cdef.abcd:An attribute defined in functional.m.method_hidden line 13 hides this method:UNDEFINED
+method-hidden:87:4:87:11:One.one:An attribute defined in functional.m.method_hidden line 85 hides this method:UNDEFINED
+method-hidden:115:4:115:18:Child._protected:An attribute defined in functional.m.method_hidden line 110 hides this method:UNDEFINED

tests/functional/m/method_hidden_py39.py

@@ -1,142 +1,16 @@
 # pylint: disable=too-few-public-methods,missing-docstring
-# pylint: disable=unused-private-member
-# pylint: disable=import-error
 """check method hiding ancestor attribute
 """
-import functools as ft
-import something_else as functools
-
-
-class Abcd:
-    """dummy"""
-
-    def __init__(self):
-        self.abcd = 1
-
-
-class Cdef(Abcd):
-    """dummy"""
-
-    def abcd(self):  # [method-hidden]
-        """test"""
-        print(self)
-
-
-class AbcdMixin:
-    def abcd(self):
-        pass
-
-
-class Dabc(AbcdMixin, Abcd):
-    def abcd(self):
-        pass
-
-
-class CustomProperty:
-    """dummy"""
-
-    def __init__(self, _):
-        pass
-
-    def __get__(self, obj, __):
-        if not obj:
-            return self
-        return 5
-
-    def __set__(self, _, __):
-        pass
-
-
-class Ddef:
-    """dummy"""
-
-    def __init__(self):
-        self.five = "five"
-
-    @CustomProperty
-    def five(self):
-        """Always 5."""
-        return self
-
-
-def my_decorator(*args, **kwargs):
-    return CustomProperty(*args, **kwargs)
-
-
-class Foo:
-    def __init__(self):
-        self._bar = 42
-        self._baz = 84
-
-    @my_decorator
-    def method(self):  # E0202
-        return self._baz
-
-    @method.setter
-    def method(self, value):
-        self._baz = value
-
-    def do_something_with_baz(self, value):
-        self.method = value
-
-
-class One:
-    def __init__(self, one=None):
-        if one is not None:
-            self.one = one
-
-    def one(self):  # [method-hidden]
-        pass
-
-
-class Two(One):
-    def one(self):
-        pass
-
-
-try:
-    import unknown as js
-except ImportError:
-    import json as js
-
-
-class JsonEncoder(js.JSONEncoder):
-    # pylint: disable=useless-super-delegation,super-with-arguments
-    def default(self, o):
-        return super(JsonEncoder, self).default(o)
+import something_else as functools  # pylint: disable=import-error
 
 
 class Parent:
     def __init__(self):
         self._protected = None
-        self._protected_two = None
-        self._protected_three = None
 
 
 class Child(Parent):
-    def _protected(self):  # [method-hidden]
-        pass
-
-
-class CachedChild(Parent):
-    @ft.cached_property
-    def _protected(self):
-        pass
-
-    @functools.cached_property
-    def _protected_two(self):
-        pass
-
     @functools().cached_property
-    def _protected_three(self):
-        pass
-
-
-class ParentTwo:
-    def __init__(self):
-        self.__private = None
-
-
-class ChildTwo(ParentTwo):
-    def __private(self):
+    def _protected(self):
+        # This test case is only valid for python3.9 and above
         pass