Merge pull request aspnetboilerplate#5516 from aspnetboilerplate/maliming/patch-2

Restore DeleteOrphansTiming value of ChangeTracker.

Author: ismcagdas

src/Abp.EntityFrameworkCore/EntityFrameworkCore/AbpDbContext.cs

@@ -253,7 +253,6 @@ public virtual void Initialize(AbpEfDbContextInitializationContext initializatio
             }
 
             ChangeTracker.CascadeDeleteTiming = CascadeTiming.OnSaveChanges;
-            ChangeTracker.DeleteOrphansTiming = CascadeTiming.OnSaveChanges;
         }
 
         protected virtual EntityChangeReport ApplyAbpConcepts()

test/Abp.EntityFrameworkCore.Tests/Tests/Repository_Filtering_Tests.cs

@@ -6,13 +6,15 @@
 using System;
 using System.Linq;
 using System.Threading.Tasks;
+using Microsoft.EntityFrameworkCore;
 using Xunit;
 
 namespace Abp.EntityFrameworkCore.Tests.Tests
 {
     public class Repository_Filtering_Tests : EntityFrameworkCoreModuleTestBase
     {
         private readonly IRepository<Post, Guid> _postRepository;
+        private readonly IRepository<Blog> _blogRepository;
         private readonly IRepository<Ticket> _ticketRepository;
         private readonly IUnitOfWorkManager _unitOfWorkManager;
         private readonly IRepository<TicketListItem> _ticketListItemRepository;
@@ -22,6 +24,7 @@ public Repository_Filtering_Tests()
             _unitOfWorkManager = Resolve<IUnitOfWorkManager>();
 
             _postRepository = Resolve<IRepository<Post, Guid>>();
+            _blogRepository = Resolve<IRepository<Blog>>();
             _ticketRepository = Resolve<IRepository<Ticket>>();
             _ticketListItemRepository = Resolve<IRepository<TicketListItem>>();
         }
@@ -123,5 +126,28 @@ public async Task Should_Filter_View_With_MustHaveTenantId()
             ticketsDefault.Any(t => t.TenantId == 42).ShouldBeTrue();
             ticketsDefault.Any(t => t.TenantId != 42).ShouldBeFalse();
         }
+        
+        [Fact]
+        public async Task Navigation_Properties_Cascade_Delete_Test()
+        {
+            await WithUnitOfWorkAsync(async () =>
+            {
+                var blog = await _blogRepository.GetAll().Include(x => x.Posts).FirstOrDefaultAsync(b => b.Name == "test-blog-1");
+                blog.Posts.ShouldNotBeEmpty();
+
+                blog.Posts.Clear();
+                await _blogRepository.UpdateAsync(blog);
+            });
+            
+            await WithUnitOfWorkAsync(async () =>
+            {
+                using (_unitOfWorkManager.Current.DisableFilter(AbpDataFilters.SoftDelete))
+                {
+                    var blog = await _blogRepository.GetAll().Include(x => x.Posts).FirstOrDefaultAsync(b => b.Name == "test-blog-1");
+                    blog.Posts.ShouldNotBeEmpty();
+                    blog.Posts.ShouldAllBe(x => x.IsDeleted);
+                }
+            });
+        }
     }
 }