Remove __ne__ where it's a simple wrapper of __eq__

src/coaster/sqlalchemy/markdown.py

@@ -92,10 +92,6 @@ def __eq__(self, other: t.Any) -> bool:
             self.__composite_values__() == other.__composite_values__()
         )
 
-    def __ne__(self, other: t.Any) -> bool:
-        """Compare for inequality."""
-        return not self.__eq__(other)
-
     # Pickle support methods implemented as per SQLAlchemy documentation, but not
     # tested here as we don't use them.
     # https://docs.sqlalchemy.org/en/13/orm/extensions/mutable.html#id1

src/coaster/sqlalchemy/roles.py

@@ -538,9 +538,6 @@ def __eq__(self, other: t.Any) -> bool:
             return self.obj == other.obj and self.actor == other.actor
         return self._contents() == other
 
-    def __ne__(self, other: t.Any) -> bool:
-        return not self.__eq__(other)
-
     def __and__(self, other: t.Iterable[str]) -> t.Set[str]:
         """Faster implementation that avoids lazy lookups where not needed."""
         return {role for role in other if self._role_is_present(role)}
@@ -714,10 +711,6 @@ def __eq__(self, other: t.Any) -> bool:
             and self.attr == other.attr
         )
 
-    def __ne__(self, other: t.Any) -> bool:
-        # This method is required as abc.Set provides a less efficient version
-        return not self.__eq__(other)
-
 
 class RoleAccessProxy(abc.Mapping, t.Generic[RoleMixinType]):
     """
@@ -986,10 +979,6 @@ def __eq__(self, other: t.Any) -> bool:
             return True
         return super().__eq__(other)
 
-    def __ne__(self, other: t.Any) -> bool:
-        # Don't call __eq__ directly, it may return NotImplemented
-        return not self == other
-
     def __bool__(self) -> bool:
         return bool(self._obj)
 

src/coaster/sqlalchemy/statemanager.py

@@ -520,9 +520,6 @@ def __eq__(self, other: t.Any) -> bool:
             and self._obj == other._obj
         )
 
-    def __ne__(self, other: t.Any) -> bool:
-        return not self.__eq__(other)
-
     def __bool__(self) -> bool:
         return self._mstate.is_current_in(self._obj)
 

src/coaster/views/classview.py

@@ -663,9 +663,6 @@ def current_handler(self) -> ViewMethodBind:
     def __eq__(self, other: t.Any) -> bool:
         return type(other) is type(self)
 
-    def __ne__(self, other: t.Any) -> bool:
-        return type(other) is not type(self)
-
     def dispatch_request(
         self, view: t.Callable[..., ResponseReturnValue], view_args: t.Dict[str, t.Any]
     ) -> BaseResponse:
@@ -954,9 +951,6 @@ def __init_subclass__(cls) -> None:
     def __eq__(self, other: t.Any) -> bool:
         return type(other) is type(self) and other.obj == self.obj
 
-    def __ne__(self, other: t.Any) -> bool:
-        return not self.__eq__(other)
-
     def dispatch_request(
         self, view: t.Callable[..., ResponseReturnValue], view_args: t.Dict[str, t.Any]
     ) -> BaseResponse: