We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
現状の単体テストでは、リポジトリに渡される検索条件が単体テストの中で確認されていない。 例えば samples/Dressca/dressca-backend/tests/Dressca.UnitTests.ApplicationCore/ApplicationService/CatalogApplicationServiceTest.cs では以下のように実装されている。
[Fact] public async Task GetCatalogItemsAsync_リポジトリのFindAsyncを1回呼出す() { // Arrange var catalogRepositoryMock = new Mock<ICatalogRepository>(); var catalogBrandRepository = Mock.Of<ICatalogBrandRepository>(); var catalogCategoryRepository = Mock.Of<ICatalogCategoryRepository>(); var logger = this.CreateTestLogger<CatalogApplicationService>(); var service = new CatalogApplicationService(catalogRepositoryMock.Object, catalogBrandRepository, catalogCategoryRepository, logger); const int skip = 1; const int take = 10; // Act _ = await service.GetCatalogItemsAsync(skip, take, 1, 1); // Assert catalogRepositoryMock.Verify( r => r.FindAsync(It.IsAny<Expression<Func<CatalogItem, bool>>>(), skip, take, AnyToken), Times.Once); }
Assertする際、検索条件の検証が It.IsAny<Expression<Func<CatalogItem, bool>>>() で行われており、適切に検索条件がテストされているとは言えない。 検索条件が正しく設定されているかテストする必要がある。
It.IsAny<Expression<Func<CatalogItem, bool>>>()
対象のテストを明らかにしてから着手する必要がある。 必要に応じて Sub-issue を作成して、少しずつ対処する。
The text was updated successfully, but these errors were encountered:
No branches or pull requests
概要
現状の単体テストでは、リポジトリに渡される検索条件が単体テストの中で確認されていない。
例えば samples/Dressca/dressca-backend/tests/Dressca.UnitTests.ApplicationCore/ApplicationService/CatalogApplicationServiceTest.cs では以下のように実装されている。
Assertする際、検索条件の検証が
It.IsAny<Expression<Func<CatalogItem, bool>>>()
で行われており、適切に検索条件がテストされているとは言えない。検索条件が正しく設定されているかテストする必要がある。
詳細 / 機能詳細(オプション)
対象のテストを明らかにしてから着手する必要がある。
必要に応じて Sub-issue を作成して、少しずつ対処する。
完了条件
The text was updated successfully, but these errors were encountered: