fastapi_14458

{
 "command": "grep -rn \"is_coroutine_callable\" . | head -20"
}

{
 "command": "grep -nE \"is_coroutine_callable|is_async_gen_callable|is_gen_callable\" fastapi/dependencies/models.py"
}

{
 "command": "sed -n '100,180p' fastapi/dependencies/models.py"
}

{
 "command": "cat <<EOF > /tmp/repro.py\nimport inspect\nfrom fastapi import FastAPI, Depends\nfrom fastapi.dependencies.models import Dependant\nfrom typing import Annotated\n\nclass MyDep:\n    async def __call__(self):\n        return \"hello\"\n\napp = FastAPI()\n\n@app.get(\"/\")\nasync def root(dep: Annotated[str, Depends(MyDep)]):\n    return dep\n\n# We want to check if MyDep (the class) is considered a coroutine callable\n# In the current buggy version, it might be.\n\n# To simulate how FastAPI does it:\n# We need to create a Dependant object.\n# Dependant(call=MyDep, ...)\nfrom fastapi.dependencies.models import Dependant\n\ndep_model = Dependant(call=MyDep)\nprint(f\"is_gen_callable: {dep_model.is_gen_callable}\")\nprint(f\"is_async_gen_callable: {dep_model.is_async_gen_callable}\")\nprint(f\"is_coroutine_callable: {dep_model.is_coroutine_callable}\")\n\n# Expected: all should be False because MyDep is a class. \n# The instance of MyDep is a coroutine, but the class itself is not.\nEOF\npython3 /tmp/repro.py"
}

{
 "command": "cat <<EOF > /tmp/repro_gen.py\nimport inspect\nfrom fastapi.dependencies.models import Dependant\n\nclass MyGenDep:\n    def __call__(self):\n        yield \"hello\"\n\nclass MyAsyncGenDep:\n    async def __call__(self):\n        yield \"hello\"\n\ndep_gen = Dependant(call=MyGenDep)\ndep_async_gen = Dependant(call=MyAsyncGenDep)\n\nprint(f\"MyGenDep is_gen_callable: {dep_gen.is_gen_callable}\")\nprint(f\"MyAsyncGenDep is_async_gen_callable: {dep_async_gen.is_async_gen_callable}\")\nEOF\npython3 /tmp/repro_gen.py"
}

{
 "filepath": "fastapi/dependencies/models.py",
 "new_string": "    @cached_property\n    def is_gen_callable(self) -> bool:\n        if self.call is None:\n            return False  # pragma: no cover\n        if inspect.isgeneratorfunction(\n            _impartial(self.call)\n        ) or inspect.isgeneratorfunction(_unwrapped_call(self.call)):\n            return True\n        if inspect.isclass(self.call):\n            return False\n        dunder_call = getattr(_impartial(self.call), \"__call__\", None)  # noqa: B004\n        if dunder_call is None:\n            return False  # pragma: no cover\n        if inspect.isgeneratorfunction(\n            _impartial(dunder_call)\n        ) or inspect.isgeneratorfunction(_unwrapped_call(dunder_call)):\n            return True\n        dunder_unwrapped_call = getattr(_unwrapped_call(self.call), \"__call__\", None)  # noqa: B004\n        if dunder_unwrapped_call is None:\n            return False  # pragma: no cover\n        if inspect.isgeneratorfunction(\n            _impartial(dunder_unwrapped_call)\n        ) or inspect.isgeneratorfunction(_unwrapped_call(dunder_unwrapped_call)):\n            return True\n        return False\n\n    @cached_property\n    def is_async_gen_callable(self) -> bool:\n        if self.call is None:\n            return False  # pragma: no cover\n        if inspect.isasyncgenfunction(\n            _impartial(self.call)\n        ) or inspect.isasyncgenfunction(_unwrapped_call(self.call)):\n            return True\n        if inspect.isclass(self.call):\n            return False\n        dunder_call = getattr(_impartial(self.call), \"__call__\", None)  # noqa: B004\n        if dunder_call is None:\n            return False  # pragma: no cover\n        if inspect.isasyncgenfunction(\n            _impartial(dunder_call)\n        ) or inspect.isasyncgenfunction(_unwrapped_call(dunder_call)):\n            return True\n        dunder_unwrapped_call = getattr(_unwrapped_call(self.call), \"__call__\", None)  # noqa: B004\n        if dunder_unwrapped_call is None:\n            return False  # pragma: no cover\n        if inspect.isasyncgenfunction(\n            _impartial(dunder_unwrapped_call)\n        ) or inspect.isasyncgenfunction(_unwrapped_call(dunder_unwrapped_call)):\n            return True\n        return False\n\n    @cached_property\n    def is_coroutine_callable(self) -> bool:\n        if self.call is None:\n            return False  # pragma: no cover\n        if inspect.isroutine(_impartial(self.call)) and iscoroutinefunction(\n            _impartial(self.call)\n        ):\n            return True\n        if inspect.isroutine(_unwrapped_call(self.call)) and iscoroutinefunction(\n            _unwrapped_call(self.call)\n        ):\n            return True\n        if inspect.isclass(self.call):\n            return False\n        dunder_call = getattr(_impartial(self.call), \"__call__\", None)  # noqa: B004\n        if dunder_call is None:\n            return False  # pragma: no cover\n        if iscoroutinefunction(_impartial(dunder_call)) or iscoroutinefunction(\n            _unwrapped_call(dunder_call)\n        ):\n            return True\n        dunder_unwrapped_call = getattr(_unwrapped_call(self.call), \"__call__\", None)  # noqa: B004\n        if dunder_unwrapped_call is None:\n            return False  # pragma: no cover\n        if iscoroutinefunction(\n            _impartial(dunder_unwrapped_call)\n        ) or iscoroutinefunction(_unwrapped_call(dunder_unwrapped_call)):\n            return True\n        # if inspect.isclass(self.call): False, covered by default return\n        return False",
 "old_string": "    @cached_property\n    def is_gen_callable(self) -> bool:\n        if self.call is None:\n            return False  # pragma: no cover\n        if inspect.isgeneratorfunction(\n            _impartial(self.call)\n        ) or inspect.isgeneratorfunction(_unwrapped_call(self.call)):\n            return True\n        dunder_call = geta

{
 "command": "python3 /tmp/repro.py && python3 /tmp/repro_gen.py"
}

{}