Autofixture: 'async'로 표시된 메서드를 지원하는 대체 GuardClauseAssertion을 만듭니다.

async 키워드로 표시된 메소드를 다룰 때 _fail fast_ 동작을 구현하는 것은 불가능해 보입니다.

어때요? 예를 들어 줄 수 있습니까?

@ploeh 기본적으로 _any_ async 메서드는 실패하지 않습니다. 다음 예를 고려하십시오.

```c#
공개 비동기 작업GuardedOpenGenericMethodReturningTaskResult(객체 인수)
{
if (arg == null) throw new ArgumentNullException(nameof(arg));

await Task.Yield();

return default;

}
```

이 메서드는 호출하면 던지지 않습니다. null 됩니다. 대신 실패한 Task 가 반환됩니다. 따라서 현재 형식의 GuardClauseAssertion은 실패하고 가드 절 누락에 대해 불평합니다.

실제 애플리케이션에서 거의 모든 코드는 async 이므로 각 호출은 awaited 이고 예외는 즉시 발생합니다. 따라서 예외를 보려면 await Task를 수행해야 하는 형식에서 "빠른 실패" 원칙을 위반하는 것은 완전히 괜찮습니다.

물론 코드를 다시 작성하여 즉시 실패하도록 할 수 있지만( async 키워드로 private 함수를 만들고 모든 논리를 거기로 이동) 그렇게 하는 것은 실용적이지 않습니다.

실제 응용 프로그램에서 거의 모든 코드는 async 이므로 각 호출은 awaited 이고 예외는 즉시 발생합니다.

그것은 정당한 지적입니다. 나는 그 주장을 받아들인다.

나는 누군가가 (거의 증거 없이) 무언가가 _불가능_하다고 주장하거나 암시할 때 때때로 길을 잃습니다...

누군가가 무언가가 불가능하다고 주장하거나 암시할 때 (증거가 거의 없음) 때때로 길을 잃습니다...

예, 충분히 공평합니다 😉

이제 부울 플래그를 사용하지 않도록 GuardClauseAssertion 를 다시 디자인하는 방법을 생각해 보겠습니다. 나중에 PR에서 뵙겠습니다.

이제 부울 플래그가 없도록 GuardClauseAssertion 를 다시 디자인하는 방법을 생각해 보겠습니다.

@moodmosaic이 제안한 것처럼 새 클래스가 더 나은 디자인일 수 있다고 생각합니다. 문제를 자세히 살펴보지는 않았지만 틀릴 수 있는 권리가 있습니다 😄

다음은 내가 사용한 솔루션입니다.

1. 메서드 서명의 async 키워드 제거
2. async 키워드를 사용하여 중첩 메서드 만들기
3. 메서드의 바깥 부분에 가드 절을 넣습니다.
4. 중첩된 메서드를 호출하고 결과를 반환합니다.

예시:

public Task<int> GetNumberAsync(MyObject obj)
{
    // Fail-fast behavior
    if (request is null)
        throw new ArgumentNullException(nameof(obj));

    async Task<int> work()
    {
        var items = await obj.GetItemsAsync(); // whatever you need to call
        return items.Count();
    }

    return work();
}

그것은 저에게 좋은 해결책이 되었습니다. 당신의 도움을 주셔서 감사합니다.

@mniak @zvirja 가 여기에서 제안한 것에 대한 구체적인 예가 흥미 롭습니다 .

개인적으로 테스트 프레임워크의 이점을 위해 "속임수"로 메서드를 오염시키기 때문에 이 방식이 마음에 들지 않습니다.

기본 방식으로 지원을 구현하려면 열어 두십시오. 어느 날. 가까운 미래에. 아니면 단 하루.

안녕하세요. 우리는 정확히 이 문제를 우연히 발견했고 현재까지 좋은 해결책이 없다는 사실에 놀랐습니다.
이를 위해 새 클래스를 구현하지 않을 이유가 있습니까?
AutoFixture의 내부 세부 사항을 모르지만 Method-to-Test에 대한 GuardClauseAssertion 클래스 내에서 MethodInfo를 사용할 수 있는 경우 CustomAttributes를 살펴보고 이것이 비동기 메서드인지 여부를 식별할 수 있습니다(https:/ /stackoverflow.com/a/20350646). 그래서 아마도 스위치(bool vs. new class)가 필요하지 않을 수도 있습니다.

@zvirja @aivascu 이것을 재고할 수 있습니까? 2021년에 비동기 메서드를 제대로 테스트할 수 없는 이유를 정말 모르겠습니다.

@Kralizek 상황을 제대로 읽었다면 요청이 거부되지 않았고, 심지어 그것을 수정하는 PR도 있습니다.
나는 PR을 간략하게 살펴보았고 받은 피드백에 동의한다고 말해야 합니다.
따라서 남은 것은 피드백을 구현하거나 구현된 피드백과 함께 새 PR을 제출하는 것뿐입니다.

모바일 앱에서 링크된 PR이 안보여요 😂