Início / coding / Perguntas / 79352698
Accepted
B.Quaink
Asked: 2025-01-13 23:31:36 +0800 CST

Verifique BackgroundJob.Delete no Hangfire no teste de unidade mstest

  • 772

Tenho um método CancelScheduledJob que recebe o Id de um trabalho em segundo plano e usa o JobStorage para recuperar esse Id e cancelar o trabalho hangfire correspondente:

var mon = JobStorage.Current.GetMonitoringApi();
var scheduledJobs = mon.ScheduledJobs(0, int.MaxValue);
var jobsToDelete = scheduledJobs.Where(job => job.Value.Job?.Args?.Any(arg => arg is Guid guid && guid == id) == true).ToList();

jobsToDelete?.ForEach(job => _backgroundJobClient.Delete(job.Key));

É possível verificar um método Enqueue() ou Schedule() verificando o método Create chamado no mock backgroundJobClient, como aqui:

_backgroundJobClientMock.Verify(x => x.Create(
                    It.Is<Job>(job => job.Method.Name == "Run" && Guid.Parse(job.Args[0].ToString()) == input),
                    It.IsAny<ScheduledState>()));

Mas como eu faria para verificar o método Delete? Já estou zombando do JobStorage, mas não consigo encontrar uma maneira de verificar o método Delete(). Atualmente, tenho isto:

_backgroundJobClientMock.Verify(
                x => x.Delete(It.Is<string>(jobId => jobId == "job1")),
                Times.Once
            );

Mas me deparo com o problema comum de que Delete é um método de extensão e não pode ser usado em uma expressão de configuração/verificação.

c#

3 respostas

  1. Best Answer

    Você pode fazer algo "esquisito" e testar o método de instância que é chamado pelo método de extensão - ( código-fonte atual ):

    public static bool Delete(
    [NotNull] this IBackgroundJobClient client,
    [NotNull] string jobId,
    [CanBeNull] string fromState) {
    if (client == null) throw new ArgumentNullException(nameof(client));

    var state = new DeletedState();
    return client.ChangeState(jobId, state, fromState);
}

    Isso seria ChangeState, portanto, Verifycontra isso.

    EDITAR :

    Se você não puder ou (não quiser) encapsular coisas em uma interface personalizada em seu código real como as outras respostas sugerem, você pode verificar esta minha resposta para uma abordagem alternativa. Era para verificar ILoggermétodos de extensão s como LogInformation, mas deve funcionar aqui também se você estiver dependendo de IBackgroundJobClient. Em poucas palavras, você cria uma IBackgroundJobClientExt:IBackgroundJobClientinterface que inclui os métodos de extensão explicitamente. Então use DispatchProxypara criar uma instância dela e passá-la para nosso sut. Dessa forma, você ainda pode usar Verifycontra Deleteem seu código de teste e manter a conexão entre métodos de extensão e métodos de instância que eles chamam em um único lugar - no método DispatchProxy's Invoke.

    • 3

  2. Como Ivan Petrov escreve, você pode testar um método de extensão quando sabe como ele é implementado. Isso permite que você verifique em relação à API subjacente que o método de extensão usa.

    No entanto, isso leva a testes frágeis. Seus testes agora estão vulneráveis ​​a alterações em uma biblioteca de terceiros.

    Uma opção melhor é definir uma API polimórfica (interface ou classe base) que seja definida com base nas necessidades do seu código cliente, em vez de detalhes de implementação. Aqui, consideramos toda a Hangfire API um detalhe de implementação.

    Isso seguiria mais apropriadamente o Princípio de Inversão de Dependência . Você define a abstração contra a qual está programando com base no que seu código cliente precisa, em vez do que alguma API de terceiros oferece.

    Depois de definir a API necessária , implemente a interface ou classe base com os detalhes de implementação: a API Hangfire.

    • 3

  3. Para complementar as respostas existentes, especialmente a resposta de Mark, você deve definir a interface em torno _backgroundJobClientdo tipo '.

    Algo como:

    public interface IBackgroundJobClientWrapper
{
    void Delete(string key);
}

public class BackgroundJobClientWrapper : IBackgroundJobClientWrapper
{
    private readonly IBackgroundJobClient _backgroundJobClient;

    // Assuming that's how you get _backgroundJobClient
    public BackgroundJobClientWrapper(IBackgroundJobClient backgroundJobClient)
    {
        _backgroundJobClient = backgroundJobClient;
    }

    public void Delete(string key);
    {
        _backgroundJobClient.Delete(job.Key);
    }
}

    Então você pode facilmente simular IBackgroundJobClientWrappera interface.

    É claro que você pode ajustar a definição da interface para melhor se adequar ao seu código.

    • 2

relate perguntas