如何断言异常类型和异常消息？

Question

这是我在 XUnit 中的测试方法。

    [Fact]
    public async Task AddCampaign_ReturnBadRequestWhenDateIsInvalid()
    {
        var client = _factory.CreateClient();
        string title = string.Format("Test Add Campaign {0}", Guid.NewGuid());
        var campaignAddDto = new CampaignDTORequest
        {
            Title = title
        };
        var encodedContent = new StringContent(JsonConvert.SerializeObject(campaignAddDto), Encoding.UTF8, "application/json");

        var response = await client.PostAsync("/api/Campaign/add", encodedContent);
        var responseString = await response.Content.ReadAsStringAsync();
        var result = JsonConvert.DeserializeObject<CampaignDTOResponse>(responseString);

        Assert.False(response.IsSuccessStatusCode);
        Assert.ThrowsAsync<ArgumentNullException>(()=> client.PostAsync("/api/Campaign/add", encodedContent));
    }

第一个断言是有效的。我坚持第二个断言。如何断言异常类型 (ArgumentNullException) 及其异常消息？

这是服务方法

    public async Task<Campaign> AddCampaignAsync(Campaign campaign)
    {            
        if (campaign.StartDate.Equals(DateTime.MinValue)) {
            throw new ArgumentNullException("Start Date cannot be null or empty.");
        }
        
        await _context.Campaigns.AddAsync(campaign);
        await _context.SaveChangesAsync();
        return campaign;
    }

根据雷洋的线索更新。

var exceptionDetails = Assert.ThrowsAsync<ArgumentNullException>(() => client.PostAsync("/api/Campaign/add", encodedContent));
Assert.Equal("Start Date cannot be null or empty.", exceptionDetails.Result.Message);

但仍然不起作用。

System.AggregateException ：发生一个或多个错误。(Assert.Throws() 预期失败：typeof(System.ArgumentNullException) 实际：(没有抛出异常))

尝试了 Dai 的解决方案，但仍然出错。

Assert.Throws() Failure
Expected: typeof(System.ArgumentNullException)
Actual:   (No exception was thrown)

这是我的 API 方法。

 public async Task<ActionResult<CampaignDTOResponse>> AddCampaign([FromBody] CampaignDTORequest newCampaign)
    {
        try
        {
            var campaign = _mapper.Map<Campaign>(newCampaign);
            campaign = await _campaignService.AddCampaignAsync(campaign);
            var campaignDtoResponse = _mapper.Map<CampaignDTOResponse>(campaign);
            return CreatedAtAction(nameof(GetCampaignById), new { id = campaignDtoResponse.Id }, campaignDtoResponse);
        }
        catch (Exception ex)
        {
            _logger.LogError(0, ex, ex.Message);
            return Problem(ex.Message);
        }
    }

更新：我将检查从服务转移到 api。

if (newCampaign.StartDate.Equals(DateTime.MinValue))
{
    return BadRequest("Start Date cannot be null or empty.");
}

我像下面那样断言它们。

Assert.Equal(HttpStatusCode.BadRequest, response.StatusCode);
Assert.Equal("Start Date cannot be null or empty.", responseString);

Answer 1

Assert.ThrowsAsync仍然是一个async Task方法，因此您需要await 确保任务继续（执行实际断言）可以正确运行：

[Fact]
public async Task AddCampaign_Return_bad_request_when_date_is_invalid()
{
    [...]

    Assert.False(response.IsSuccessStatusCode);
    await Assert.ThrowsAsync<ArgumentNullException>(()=> client.PostAsync("/api/Campaign/add", encodedContent));
}

然而...

• ....请重新考虑您的设计：例外应该是例外。
  • 虽然不是每个人都同意：https : //softwareengineering.stackexchange.com/questions/184654/ive-been-told-that-exceptions-should-only-be-used-in-exceptional-cases-how-do
  • 尽管 Java 具有检查异常，但使用异常来表示各种错误条件而不仅仅是异常错误条件是有意义的- 而 C#/.NET 没有检查异常，因此需要人们阅读手写文档以查看声明的异常是什么（如果有） - 这使得.NET 中的正确错误处理变得相当痛苦，除非您同意仅在异常情况下抛出并使用返回类型来表示非异常错误情况。
• 即使你想抛出，你也不应该抛出ArgumentNullException代表 HTTP 400 错误请求响应。
  • 本ArgumentException类及其子类（ArgumentNullException，ArgumentOutOfRangeException，等）应只用来表示失败的先决条件-不是一个失败的后置条件，也没有内部错误（使用InvalidOperationException了这一点）。
  • 我个人不认为Web服务客户端应该不断抛出异常的任何响应，除非它是真正的“特殊”的反应或情况。
  • 如果您使用 NSwag 生成 Web 服务客户端，那么它会ApiException<TResponse>为您生成，这将更有用。
  • 虽然我的偏好是返回所有合理可能的响应（即，无论 声明的任何内容[ProducesResponseType]）的歧视联合。