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

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 is still an async Task method，因此您需要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));
}

---

然而……

• ....请重新考虑您的设计：Exceptions should be exceptional。
  • 虽然不是所有人都同意：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] 声明的任何响应）的有区别的联合。

Answer 2

您可以使用ThrowsAsync 捕获异常并声明它：

// Act
var exception = await Assert.ThrowsAsync<ArgumentNullException>(()=> client.PostAsync("/api/Campaign/add", encodedContent));

// Assert
Assert.Equal(exception.Message, "message to compare");

或

// Act
Action action = async () => await client.PostAsync("/api/Campaign/add", encodedContent);
var ex = Record.Exception(action);

// Assert
Assert.NotNull(ex);
Assert.IsType<ArgumentNullException>(ex);