IDisposable on Scope doesn't call Dispose on cached instances

[davidmatson]

Based on reading the code; haven't tested, but I think the repro would be something like:

class SpyDisposable : IDisposable
{
    public bool Disposed { get; private set; }

    public void Dispose()
    {
        Disposed = true;
    }
}

Container.Register<SpyDisposable>(typeof(SpyDisposable));
SpyDisposable spy;

using (var scope = Container.CreateScope())
{
    spy = scope.Resolve<SpyDisposable>();
}

Assert.True(spy.Disposed);

[vladris]

Note there is a small bug in the repro above. SpyDisposable should be registered as PerScope, otherwise the container won't hold on to the resolved instance:

-Container.Register<SpyDisposable>(typeof(SpyDisposable));
+Container.Register<SpyDisposable>(typeof(SpyDisposable)).PerScope();