비동기 void 메서드를 테스트하는 방법

Question

아래와 같이 코드를 고려하십시오. GetBrands를 호출하면 속성 브랜드에 적절한 데이터가 할당됩니다.

public class BrandsViewModel : ViewModelBase 
{ 
    private IEnumerable<Brand> _brands; 
    public IEnumerable<Brand> Brands 
    { 
     get { return _brands; } 
     set { SetProperty(ref _brands, value); } 
    } 

    public async void GetBrands() 
    { 
     // ...... 

     Brands = await _dataHelper.GetFavoriteBrands(); 

     // ...... 
    } 
} 

그러나 아래와 같이 테스트하면 테스트가 실패합니다. GetBrands 메서드 내에서 비동기 호출을 기다리는 방법은 무엇입니까?

[TestMethod] 
public void AllBrandsTest() 
{ 
    BrandsViewModel viewModel = new BrandsViewModel(); 
    viewModel.GetBrands(); 
    Assert.IsTrue(viewModel.Brands.Any()); 
} 

Answer 1

간단한 대답은 다음과 같습니다. async void하지 마십시오. 실제로 을 만들지 마십시오. 절대적으로 이벤트 처리기로 작동해야하는 경우가 아니면 async void을 만듭니다. async void의 패배는 정확하게입니다. 여기에서 원하는 테스트 (실제 코드에 해당하는 것)가 있습니다.

대신 async Task 메쏘드로 만드십시오. 이제 완료 (시간 초과 포함)/연속 추가 및 성공 또는 예외로 종료했는지 여부를 확인할 수 있습니다.

이 하나의 단어 변화하는 것입니다

public async Task GetBrands() 
{ 
    // ...... 

    Brands = await _dataHelper.GetFavoriteBrands(); 

    // ...... 
} 

다음 테스트 :

[TestMethod] 
public async Task AllBrandsTest() 
{ 
    BrandsViewModel viewModel = new BrandsViewModel(); 
    var task = viewModel.GetBrands(); 
    Assert.IsTrue(task.Wait(YOUR_TIMEOUT), "failed to load in time"); 
    Assert.IsTrue(viewModel.Brands.Any(), "no brands"); 
} 

Answer 2

모델 (DTO) 자체가 채워집니다 (데이터 액세스). 이것은 하나의 클래스가하기에는 너무 많은 것입니다. 대개 자신에게 물을 때 "어떻게이 지구를 테스트 할 수 있습니까?", 리팩토링 할 시간입니다. 별도의 데이터 액세스 클래스를 만듭니다.

BrandsViewModel viewModel = new BrandsViewModel(); 
var brandAccess = new BrandsDataAccess(); 
viewModel.Brands = await brandAccess.GetAllBrands(); 
Assert.IsTrue(viewModel.Brands.Any()); 

이제 BrandsDataAccess.GetAllBrands()을 테스트 할 수 있습니다.