Dependency overrides don't work with post clean up.

[mr-mapache]

Hi! I need to inject resources with post-cleanup. I have this example working:

from fast_depends import Depends
from fast_depends import inject
from fast_depends import Provider

provider = Provider()

def callback():
    try:
        yield lambda count: print(count)
    finally:
        print('done')


@inject(dependency_overrides_provider=provider)
def handle_for_loop(callback = Depends(callback)):
    for i in range(10):
        callback(i)
        
handle_for_loop()

But when trying to make it work with dependency_overrides it fails.

from fast_depends import Depends
from fast_depends import inject
from fast_depends import Provider

provider = Provider()

def callback():...

@inject(dependency_overrides_provider=provider)
def handle_for_loop(callback = Depends(callback)):
    for i in range(10):
        callback(i)

def override_callback():
    try:
        yield lambda count: print(count)
    finally:
        print('done')


provider.dependency_overrides[callback] = override_callback

handle_for_loop()

It gives the error:

raceback (most recent call last):
  File "main.py", line 46, in <module>
    handle_for_loop()
  File "fast_depends/use.py", line 169, in injected_wrapper
    r = real_model.solve(
        ^^^^^^^^^^^^^^^^^
  File "fast_depends/core/model.py", line 416, in solve
    response = call(*final_args, **final_kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "main.py", line 35, in handle_for_loop
    callback(i)
TypeError: 'generator' object is not callable

[mr-mapache]

If I replace:

def callback():
    raise NotImplementedError

for this:

def callback():
    yield

It works again, It seems that Depends is not resolving how to call the generator dependency in runtime but reliying on what was injected fisrt.

[mr-mapache]

I made it work by changing this line:

class CallModel(Generic[P, T]):
    def solve(
            response = call(*final_args, **final_kwargs)

by this:

class CallModel(Generic[P, T]):
    def solve(
            response = call(*final_args, **{key: next(value) if hasattr(value, '__next__') else value for key, value in final_kwargs.items()})

In the FastDepends/fast_depends/use.py

Let me know if I can open a pull request.

[Lancetnik]

Let me know if I can open a pull request.

For sure - if everything else works fine you are welcome to open any PR

[mr-mapache]

Hi, I opened the PR solving the issue, for both sync and async cases, but there are more related issues to this and with automating casting. I think they are part of more general problem and is beyond what can i handle.

What I can do is to write some failing tests for you to see them later if you want. By the way thanks for the library, you did a great job, hope it gets more adoption.

[Lancetnik]

What I can do is to write some failing tests for you to see them later if you want. By the way thanks for the library, you did a great job, hope it gets more adoption.

You are welcome to make any changes and improvements, but please, can you make them to 3.0.0 branch? It is a new major I want to release soon, so all core fixes and changes should be made there