Merge pull request #14284 from abpframework/enisn/cmskit-fix-tests

CmsKit fix tests

modules/cms-kit/src/Volo.CmsKit.Admin.Application/Volo/CmsKit/Admin/Pages/PageAdminAppService.cs

@@ -93,7 +93,5 @@ public virtual async Task SetAsHomePageAsync(Guid id)
         var page = await PageRepository.GetAsync(id);
 
 		await PageManager.SetHomePageAsync(page);
-
-		await PageRepository.UpdateAsync(page);
     }
 }

modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Pages/PageManager.cs

@@ -56,7 +56,8 @@ public virtual async Task SetHomePageAsync(Page page)
 		}
 
 		await PageRepository.UpdateManyAsync(currentHomePages);
-
+		await PageRepository.UpdateAsync(page);
+
 		page.SetIsHomePage(true);
 	}
 

modules/cms-kit/test/Volo.CmsKit.Domain.Tests/Pages/PageManager_Test.cs

@@ -72,12 +72,14 @@ public async Task SetSlugAsync_ShouldThrowException_WithExistingSlug()
 	[Fact]
 	public async Task SetHomePageAsync_ShouldWorkProperly_IfExistHomePage()
 	{
-		var page = await pageRepository.GetAsync(testData.Page_1_Id);
-
-		await pageManager.SetHomePageAsync(page);
+		await WithUnitOfWorkAsync(async ()=>
+		{
+			var page = await pageRepository.GetAsync(testData.Page_1_Id);
 
-		await pageRepository.UpdateAsync(page);
+			await pageManager.SetHomePageAsync(page);
+		});
 
+		var page = await pageRepository.GetAsync(testData.Page_1_Id);
 		page.IsHomePage.ShouldBeTrue();
 
 		var pageSetAsHomePageAsFalse = await pageRepository.GetAsync(testData.Page_2_Id);