如何解决SonarQube检测到的ValueTask实例未直接等待的代码异味问题？

我明白你想在测试里保留given/when/then的清晰结构，同时解决SonarQube的代码异味提示——这个问题的核心是ValueTask的设计特性：它的实例只能被消费一次，存储到局部变量或字段里很容易引发重复等待、意外的Task分配等问题，所以SonarQube会把这种用法标记为风险点。

下面给你两种既符合规范又能保留测试结构的解决方案：

方案1：用委托保留when/then分离结构 #

如果你不想打破given/when/then的步骤划分，可以把获取ValueTask的逻辑封装成一个委托，而不是直接存储ValueTask实例：

[Fact]
public async Task SampleMethod()
{
    // given
    Guid someRoleId = Guid.NewGuid();
    NpgsqlException npgsqlException = GetNpsqlException();
    var failedRoleStorageException = new FailedRoleStorageException(npgsqlException);
    var expectedRoleDependencyException = new RoleDependencyException(failedRoleStorageException);

    this.dataMock.SelectRoleByIdAsync(Arg.Any<Guid>()).Throws(npgsqlException);

    // when
    Func<Task<Role>> getRoleOperation = () => this.roleService.RoleByIdAsync(someRoleId).AsTask();

    // then
    RoleDependencyException actualServiceException = await Assert.ThrowsAsync<RoleDependencyException>(getRoleOperation);
    actualServiceException.Should().BeEquivalentTo(expectedRoleDependencyException);

    await this.dataMock.Received(1).SelectRoleByIdAsync(Arg.Any<Guid>());
    this.dataMock.ClearReceivedCalls();
}

这里我们存储的是一个委托函数，而不是ValueTask实例。当Assert.ThrowsAsync调用这个委托时，才会触发RoleByIdAsync获取ValueTask并转换为Task，完全符合SonarQube对ValueTask必须直接消费的要求，同时保留了测试步骤的清晰性。

方案2：直接在断言中使用ValueTask（推荐） #

如果你的xUnit版本是2.4.0及以上，其实Assert.ThrowsAsync已经支持ValueTask的重载，你可以直接在断言委托里await方法调用，不需要转换为Task，代码更简洁：

[Fact]
public async Task SampleMethod()
{
    // given
    Guid someRoleId = Guid.NewGuid();
    NpgsqlException npgsqlException = GetNpsqlException();
    var failedRoleStorageException = new FailedRoleStorageException(npgsqlException);
    var expectedRoleDependencyException = new RoleDependencyException(failedRoleStorageException);

    this.dataMock.SelectRoleByIdAsync(Arg.Any<Guid>()).Throws(npgsqlException);

    // when & then
    RoleDependencyException actualServiceException = await Assert.ThrowsAsync<RoleDependencyException>(
        async () => await this.roleService.RoleByIdAsync(someRoleId));
    
    actualServiceException.Should().BeEquivalentTo(expectedRoleDependencyException);

    await this.dataMock.Received(1).SelectRoleByIdAsync(Arg.Any<Guid>());
    this.dataMock.ClearReceivedCalls();
}

这种写法直接在断言委托里awaitValueTask，完全没有存储ValueTask实例的问题，同时代码更紧凑。如果你觉得when & then合并在一起不舒服，也可以把断言的委托单独放在when块里，本质和方案1是一样的。

为什么SonarQube会报这个错？ #

再补充下背景：ValueTask是为了优化性能而生的——当操作可以同步完成时，它会直接返回结果而不分配Task对象；但这种优化的代价是它的实例只能被消费一次（要么await，要么调用AsTask()一次）。如果把ValueTask存储到变量里，很容易出现重复await、多次调用AsTask()的情况，导致不可预期的行为（比如重复执行操作、抛出异常），所以SonarQube会把这种用法标记为代码异味。

备注：内容来源于stack exchange，提问作者thomsan