From 7cfe2115c0b82e1f607bf3f91109c7f92d0d828b Mon Sep 17 00:00:00 2001 From: Lars-Erik Aabech Date: Fri, 4 Jan 2019 00:15:06 +0100 Subject: [PATCH] Actually uses a scope in the ScopeBuilderCreatesScopedCollection tests --- .../Composing/CollectionBuildersTests.cs | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/Umbraco.Tests/Composing/CollectionBuildersTests.cs b/src/Umbraco.Tests/Composing/CollectionBuildersTests.cs index 7b04c52150..4c262fbf82 100644 --- a/src/Umbraco.Tests/Composing/CollectionBuildersTests.cs +++ b/src/Umbraco.Tests/Composing/CollectionBuildersTests.cs @@ -303,7 +303,7 @@ namespace Umbraco.Tests.Composing [Test] public void ScopeBuilderCreatesScopedCollection() { - _composition.WithCollectionBuilder() + _composition.WithCollectionBuilder() .Append() .Append(); @@ -313,13 +313,17 @@ namespace Umbraco.Tests.Composing var factory = _composition.CreateFactory(); - var col1 = factory.GetInstance(); - AssertCollection(col1, typeof(Resolved1), typeof(Resolved2)); + using (factory.BeginScope()) + { + var col1 = factory.GetInstance(); + AssertCollection(col1, typeof(Resolved1), typeof(Resolved2)); - var col2 = factory.GetInstance(); - AssertCollection(col2, typeof(Resolved1), typeof(Resolved2)); + var col2 = factory.GetInstance(); + AssertCollection(col2, typeof(Resolved1), typeof(Resolved2)); + + AssertSameCollection(col1, col2); + } - AssertSameCollection(col1, col2); } [Test]