Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A fresh new and simpler approach to Wirings as a combination of a PropertySourceChain and a FactoryChain #109

Closed
wants to merge 5 commits into from

Conversation

rchodava
Copy link
Owner

@rchodava rchodava commented Apr 14, 2017

Improved the error reporting for when Wirings fail
Wirings now support qualifiers

Also, improve the error when a wiring involving a named property fails
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 69.128% when pulling d995c13 on rc/constants-class-property-source into 6c3c989 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 68.678% when pulling 1c63ae4 on rc/constants-class-property-source into 6c3c989 on master.

…dlers

Improve the construction of property source chains - allow transformers for any property source, add computed and delegating property sources, change defaults to an immediate properties source
…composed of a PropertySourceChain and a FactoryChain

Wirings now support qualifiers and scopes
Wirings now report much more detail when they fail to construct instances
@rchodava rchodava changed the title Allow using a class with constants as a property source A fresh new and simpler approach to Wirings as a combination of a PropertySourceChain and a FactoryChain Apr 16, 2017
…tance

Super of isn't necessarily going to return an instance of a class derived from the class it is registered with - for example (registering super of Class1, but a call to the factory can request construction of an interface of Class1, for which the factory returns Class2, which implements the interface but is not a sub-class of Class1)
Add additional unit tests
Fixed bug so that the the singleton scope respects qualifiers
Added lots of documentation
@coveralls
Copy link

Coverage Status

Coverage increased (+1.6%) to 70.735% when pulling b4af376 on rc/constants-class-property-source into 6c3c989 on master.

@rchodava rchodava closed this Apr 17, 2017
@rchodava rchodava deleted the rc/constants-class-property-source branch April 17, 2017 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants