Instance Creation Issue with Property injection and lazy instantiation

[blsmithers]

Hi, I stumbled on an issue and think it might be similar to the issue described in #666. The below is failing for me with the error message "An item with the same key has already been added.". I was hoping this case would be possible, any help you could give me would be appreciated!

static void Main(string[] args) {
            var builder = new ContainerBuilder();

            builder.RegisterType<A>().SingleInstance();
            builder.RegisterType<B>().SingleInstance().PropertiesAutowired(PropertyWiringOptions.AllowCircularDependencies);

            using (var container = builder.Build())
            {
                var myA = container.Resolve<A>();
            }
        }

        public class A
        {
            public A(Lazy<B> b)
            {
                var myB = b.Value;
            }
        }

        public class B 
        {
            public A A { get; set; }
        }

[tillig]

Verified. It does seem to be around the use of Lazy<T> since removing Lazy<T> and using T directly doesn't encounter this. We'll have to see if we can track this down. Thanks!

[tillig]

While the exception message is different now, this is still an issue. The issue is getting caught in LifetimeScope.GetOrCreateAndShare and it seems the use of Lazy<T> doesn't carry with it the circular property dependency information. I've added a failing unit test (skipped) to the solution.

Note we also have #798 where we're trying to figure out better circular dependency handling. The use of Lazy<T> needs to be considered there.

[tillig]

This appears to have been resolved with the overhaul to the internals in v6. The skipped/failing test for this issue can be marked to run and it now passes.

Fix will be released with v6.