Skip to content
This repository has been archived by the owner on Dec 19, 2023. It is now read-only.

OverridingContainer should extend base enumerables #42

Closed
LordZoltan opened this issue May 14, 2017 · 0 comments
Closed

OverridingContainer should extend base enumerables #42

LordZoltan opened this issue May 14, 2017 · 0 comments
Assignees
Labels
Milestone

Comments

@LordZoltan
Copy link
Member

The Auto-built enumerables should concatenate any objects from an overriding container with those of the base container, so:

var container1 = new Container();
container1.RegisterObject(1);
var container2 = new OverridingContainer(container1);
container2.RegisterObject(2);

var result = container2.Resolve<IEnumerable<int>>();

// result should be [1, 2]
@LordZoltan LordZoltan added this to the 1.2 milestone May 14, 2017
@LordZoltan LordZoltan self-assigned this May 14, 2017
@LordZoltan LordZoltan changed the title Overriding containers should extend enumerables OverridingContainer should extend base enumerables May 14, 2017
LordZoltan added a commit that referenced this issue May 14, 2017
New test added to this which currently fails DELIBERATELY.
LordZoltan added a commit that referenced this issue May 15, 2017
…this functionality) now breaks - however there's more to do now with making the new behaviour optional and reinventing how a target container creates other target containers internally. Initial idea for #36, too, which I think needs a bit more work.
@LordZoltan LordZoltan modified the milestones: 1.3, 1.2 Jul 3, 2017
@LordZoltan LordZoltan mentioned this issue Sep 29, 2017
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant