fixed: #316

Author: maximv

src/DryIoc/Container.cs

@@ -4737,7 +4737,7 @@ internal static bool TryGetUsedInstance(this IResolverContext r, Type serviceTyp
         // todo: @perf no need to check for IDisposable in TrackDisposable
         /// <summary>A bit if sugar to track disposable in the current scope or in the singleton scope as a fallback</summary>
         public static T TrackDisposable<T>(this IResolverContext r, T instance) where T : IDisposable =>
-            (T)(r.SingletonScope ?? r.CurrentScope).TrackDisposable(instance); // todo: @fix check the scope first
+            (T)(r.CurrentScope ?? r.SingletonScope).TrackDisposable(instance);
     }
 
     /// <summary>The result delegate generated by DryIoc for service creation.</summary>

test/DryIoc.UnitTests/ReuseTests.cs

@@ -558,6 +558,19 @@ public void Disposing_container_should_dispose_the_registered_instance_even_not_
             Assert.IsTrue(service.IsDisposed);
         }
 
+        [Test]
+        public void Disposing_the_scope_should_dispose_the_registered_instance_even_not_resolved()
+        {
+            var container = new Container();
+            var service = new SomethingDisposable();
+            using (var scope = container.OpenScope())
+            {
+                scope.Use(scope.TrackDisposable(service));
+            }
+
+            Assert.IsTrue(service.IsDisposed);
+        }
+
         [Test]
         public void Dispose_should_happen_in_reverse_resolution_order()
         {