Add TypeFactory

[AliveDevil]

Full interface for TypeFactory:

// Convention Scoped
AddScopedTypeFactory<TFactory>()
AddScopedTypeFactory<TFactory, TProxy>()
AddScopedTypeFactory<TFactory, TInterceptor>()

// Convention Singleton
AddSingletonTypeFactory<TFactory>()
AddSingletonTypeFactory<TFactory, TProxy>()
AddSingletonTypeFactory<TFactory, TInterceptor>()

AddTypeFactory<TFactory>()
AddTypeFactory<TFactory, TProxy>()
AddTypeFactory<TFactory, TInterceptor>()
AddTypeFactory<TFactory, TProxy, TInterceptor>()
AddTypeFactory(Type, Type)
AddTypeFactory(Type, Type, Type)

UseTypeFactories()

Remarks:

• All methods are extensions and comply to IServiceCollection(IServiceCollection, …, ServiceLifetime) By convention (AddScoped, AddSingleton) or by parameter)
• Whenever TFactory is mentioned, assume where TFactory : class, IRequiresAsyncProxy
• Whenever TProxy is mentioned, assume where TProxy : TFactory
• Whenever TInterceptor is mentioned, assume where TInterceptor : Interceptor

Resolves #649

[alexyakunin]

Hi, I'll merge this. One q: do you think we can come up with something better than "MethodBased"? Just want to know if you have some ideas on that - I tend to dislike unnecessarily long names, but struggle to find a name that describes this well :)

[alexyakunin]

Maybe something like "AutoFactory"?

[alexyakunin]

(just throwing some ideas, i.e. this is not my preference)

[AliveDevil]

So, my ideas:

• Drop the IMethodBasedFactory, and make Add*Factory<T>() T : class, IRequiresAsyncProxy (taking any interface)
• My name ideas:
  • ClassFactory
  • TypeFactory

So AddClassFactoy, AddTypeFactory, AddAutoFactory. With "Add(Singleton|Scoped)". Don't know whether Transient would be necessary.

Auto Factory is fine as well. Not that good either with naming things.

[AliveDevil]

Renamed to TypeFactory and made some changes to how this extension can be consumed (overwrite either Proxy or Interceptor type). TypeFactory now takes class, IRequiresAsyncProxy, which should be sufficient. Adding an interface just for annotation seemed a bit too much, as there already is Requires Async Proxy.

[alexyakunin]

LGTM, will merge this PR today or tomorrow - thanks!

[alexyakunin]

@AliveDevil I slightly edited this, but no changes to the way it works: https://github.com/servicetitan/Stl.Fusion/commit/49c60c13e080acbe64627aa4c552f468c79aa875

Mostly, it's "TypeFactory" -> "TypedFactory", + I removed a bunch of excessive methods :)

[alexyakunin]

And added one more test.