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 等）应该仅用于指示失败的前提条件 - 而不是失败的后置条件或内部错误（使用 {{1 }}）。
- 我个人认为网络服务客户端不应该永远为任何响应抛出异常，除非它实际上是一个“异常”响应或情况。
- 如果您使用 NSwag 生成网络服务客户端，那么它会为您生成 InvalidOperationException，这会更有用。
- 尽管我的偏好是返回所有合理可能响应（即ApiException<TResponse>声明的任何内容）的歧视联合。

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);

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

2 个答案:

不过……