Removed the skipCapability function.

Author: VladislavYar

tests/test_distinct.py

@@ -2,6 +2,7 @@
 
 from tests.testmodels import Tournament
 from tortoise.contrib import test
+from tortoise.contrib.test.condition import NotIn
 from tortoise.exceptions import OperationalError
 
 # ---------------------------------------------------------------------------
@@ -197,7 +198,7 @@ async def test_distinct_on_invalid_order_by(db):
         await Tournament.all().distinct("name").order_by("desc")
 
 
-@test.skipCapability(dialect="postgres")
+@test.requireCapability(dialect=NotIn("postgres"))
 @pytest.mark.asyncio
 async def test_distinct_on_not_supported_outside_postgres(db):
     with pytest.raises(OperationalError):

tortoise/contrib/test/__init__.py

@@ -50,7 +50,6 @@ async def test_sqlite_only(db):
     "TortoiseContext",
     "tortoise_test_context",
     "requireCapability",
-    "skipCapability",
     "truncate_all_models",
     "init_memory_sqlite",
     "SkipTest",
@@ -236,69 +235,6 @@ def skip_wrapper(*args: typing.Any, **kwargs: typing.Any) -> typing.Any:
     return decorator
 
 
-def skipCapability(
-    connection_name: str = "models", **conditions: typing.Any
-) -> Callable[[_FT], _FT]:
-    """
-    Skip a test if the specified capabilities are matched.
-
-    This is the inverse of :func:`requireCapability`.
-
-    Usage:
-
-    .. code-block:: python3
-
-        @skipCapability(dialect='postgres')
-        @pytest.mark.asyncio
-        async def test_skip_on_postgres(db):
-            ...
-
-    :param connection_name: name of the connection to retrieve capabilities from.
-    :param conditions: capability tests — if all match, the test is skipped.
-    """
-
-    def decorator(test_item: _FT) -> _FT:
-        if not isinstance(test_item, type):
-
-            def check_capabilities() -> None:
-                db = get_connection(connection_name)
-                if all(getattr(db.capabilities, key) == val for key, val in conditions.items()):
-                    raise SkipTest(f"Skipped because capabilities match: {conditions}")
-
-            if inspect.iscoroutinefunction(test_item):
-
-                @wraps(test_item)
-                async def skip_wrapper(*args: typing.Any, **kwargs: typing.Any) -> typing.Any:
-                    check_capabilities()
-                    return await test_item(*args, **kwargs)
-
-            else:
-
-                @wraps(test_item)
-                def skip_wrapper(*args: typing.Any, **kwargs: typing.Any) -> typing.Any:
-                    check_capabilities()
-                    return test_item(*args, **kwargs)
-
-            return cast(_FT, skip_wrapper)
-
-        # Assume a class is decorated
-        funcs = {
-            var: f
-            for var in dir(test_item)
-            if var.startswith("test_") and callable(f := getattr(test_item, var))
-        }
-        for name, func in funcs.items():
-            setattr(
-                test_item,
-                name,
-                skipCapability(connection_name=connection_name, **conditions)(func),
-            )
-
-        return test_item
-
-    return decorator
-
-
 @typing.overload
 def init_memory_sqlite(models: ModulesConfigType | None = None) -> AsyncFuncDeco: ...