Browse Source
Merge pull request #14284 from abpframework/enisn/cmskit-fix-tests
CmsKit fix tests
pull/14294/head
liangshiwei
4 years ago
committed by
GitHub
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with
8 additions and
7 deletions
-
modules/cms-kit/src/Volo.CmsKit.Admin.Application/Volo/CmsKit/Admin/Pages/PageAdminAppService.cs
-
modules/cms-kit/src/Volo.CmsKit.Domain/Volo/CmsKit/Pages/PageManager.cs
-
modules/cms-kit/test/Volo.CmsKit.Domain.Tests/Pages/PageManager_Test.cs
|
|
|
@ -93,7 +93,5 @@ public class PageAdminAppService : CmsKitAdminAppServiceBase, IPageAdminAppServi |
|
|
|
var page = await PageRepository.GetAsync(id); |
|
|
|
|
|
|
|
await PageManager.SetHomePageAsync(page); |
|
|
|
|
|
|
|
await PageRepository.UpdateAsync(page); |
|
|
|
} |
|
|
|
} |
|
|
|
|
|
|
|
@ -56,7 +56,8 @@ public class PageManager : DomainService |
|
|
|
} |
|
|
|
|
|
|
|
await PageRepository.UpdateManyAsync(currentHomePages); |
|
|
|
|
|
|
|
await PageRepository.UpdateAsync(page); |
|
|
|
|
|
|
|
page.SetIsHomePage(true); |
|
|
|
} |
|
|
|
|
|
|
|
|
|
|
|
@ -72,12 +72,14 @@ public class PageManager_Test : CmsKitDomainTestBase |
|
|
|
[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); |
|
|
|
|