Better error handling

[gen-xu]

Currently if the callable provider fails due to missing arguments or other errors, it may be hard to find out which guy is missing arguments if one provider is being referred many times.

src/dependency_injector/providers.pxd:422: in dependency_injector.providers.__provide_keyword_args
    ???
src/dependency_injector/providers.pxd:345: in dependency_injector.providers.__get_value
    ???
src/dependency_injector/providers.pyx:195: in dependency_injector.providers.Provider.__call__
    ???
src/dependency_injector/providers.pyx:197: in dependency_injector.providers.Provider.__call__
    ???
src/dependency_injector/providers.pyx:2755: in dependency_injector.providers.Singleton._provide
    ???
src/dependency_injector/providers.pxd:620: in dependency_injector.providers.__factory_call
    ???
src/dependency_injector/providers.pxd:606: in dependency_injector.providers.__callable_call
    ???
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

>   ???
E   TypeError: __init__() missing 1 required positional argument: 'config'

src/dependency_injector/providers.pxd:579: TypeError

it may be hard to find out which __init__, consider re-raise the exception with the function reference

[rmk135]

Hi @gen-xu. Yes, I agree that it turns to be a problem. Thanks for the PR. I'll look at it later today.

[rmk135]

@gen-xu , commented on the PR.

[whysage]

@rmk135 Is there are any way to run dependency-injector in pure python for debug?

[rmk135]

@whysage , no, you can only run it as a C extension

[chbndrhnns]

I was troubleshooting a missing argument today in a FastAPI application and I agree that the error handling makes it hard to see where the error occurred. The only hint I get is the token_service.

../../../../Library/Caches/pypoetry/virtualenvs/xxx-auth-v2-7R6iqtGQ-py3.9/lib/python3.9/site-packages/dependency_injector/wiring.py:628: in _patched
    provide = provider()
src/dependency_injector/providers.pyx:207: in dependency_injector.providers.Provider.__call__
    ???
src/dependency_injector/providers.pyx:2385: in dependency_injector.providers.Factory._provide
    ???
src/dependency_injector/providers.pxd:620: in dependency_injector.providers.__factory_call
    ???
src/dependency_injector/providers.pxd:606: in dependency_injector.providers.__callable_call
    ???
src/dependency_injector/providers.pxd:552: in dependency_injector.providers.__call
    ???
src/dependency_injector/providers.pxd:422: in dependency_injector.providers.__provide_keyword_args
    ???
src/dependency_injector/providers.pxd:345: in dependency_injector.providers.__get_value
    ???
src/dependency_injector/providers.pyx:685: in dependency_injector.providers.Dependency.__call__
    ???
src/dependency_injector/providers.pyx:207: in dependency_injector.providers.Provider.__call__
    ???
src/dependency_injector/providers.pyx:2822: in dependency_injector.providers.Singleton._provide
    ???
src/dependency_injector/providers.pxd:620: in dependency_injector.providers.__factory_call
    ???
src/dependency_injector/providers.pxd:606: in dependency_injector.providers.__callable_call
    ???
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

>   ???
E   TypeError: __init__() got an unexpected keyword argument 'token_service'

src/dependency_injector/providers.pxd:579: TypeError

[attentive]

Nothing much to add, just having my first experiences with dependency-injector after using lots of other DI stacks, I have about 20 items in my container and am currently having trouble locating the place init() is failing.

Library seems great otherwise 👏

[thatguysimon]

Hey @rmk135, error handling still seems to be an issue. Any updates on this? I see there are a couple of PRs attempting to solve this.

[dharmagetic]

@rmk135 I also request this issue