Making typing.Type injectable. about injectable HOT 5 CLOSED

I created a gist with a minimal (not) working sample of what i tried. In case you don't know peewee (a sql orm), the most important function in order to understand this gist should be engine.create_tables() which accepts an array of table classes, which it then creates in the database.

https://gist.github.com/romangraef/584f8c075143412dd0f681a5b189e440

from injectable.

Hi @romangraef! Thank you for the issue with implementation suggestions details ♥️

Could you provide a minimal working example of how you're using injectable today? I fear I wasn't fully able to see it through. With a concrete example, it will be easier to spot the issue's root cause and think of the best way out of it.

from injectable.

Thank you for the gist @romangraef. I've run it and it does give me an error due to the Type wrapper. I've removed the Type wrapper and just used tables: Autowired(typing.List[TableBase]) and it worked.

Sorry, I think that I didn't get it when you said you can't use Autowired(List[BaseType]), especially the "no empty __init__" thing. Can you clarify this for me?

from injectable.

Hmm, i thought you couldn't instantiate the tables objects without arguments, but i might have been wrong. Well that saved me that trouble, and I'm sorry for yours. It might still be an interesting feature to be able to inject the Type instead, but (as i said in my OP) there probably aren't that many use cases except for this specific one.

from injectable.

Injectables mustn't depend on mandatory arguments. In case arguments are needed for instantiation one should declare an injectable_factory to correctly instantiate objects.

Currently, I don't see a real benefit in allowing wrapping types with Type. It is odd that one would want an object of type Type and most probably typing.Type isn't intended for that kind of usage.

I'm really glad that you solved your issue 🎉 Please, feel free to open other issues if you find it necessary or to post a question at StackOverflow with the python-injectable tag.

As the issue seems solved I'm closing it. If you believe that it should be reopened just leave a comment and I'll review it.

from injectable.