dotnet/runtime

Rewrite 3 MemoryCache tests to not be timing sensitive

Fechada

#72.890 aberto em 26 de jul. de 2022

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)C# (5.445 forks)batch import
area-Extensions-Cachingdisabled-testhelp wanted

Métricas do repositório

Stars
 (17.886 estrelas)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

Description

GetAndSet_AreThreadSafe_AndUpdatesNeverLeavesNullValues OvercapacityPurge_AreThreadSafe AddAndReplaceEntries_AreThreadSafe

These were inherited from aspnet/caching but have been disabled as they need rewriting to not be racy.

example failure

System.AggregateException : One or more errors occurred.
---- System.Threading.Tasks.TaskCanceledException : A task was canceled.


Stack trace
   at System.Threading.Tasks.Task.WaitAll(Task[] tasks, Int32 millisecondsTimeout, CancellationToken cancellationToken)
   at Microsoft.Extensions.Caching.Memory.MemoryCacheSetAndRemoveTests.OvercapacityPurge_AreThreadSafe() in /_/src/libraries/Microsoft.Extensions.Caching.Memory/tests/MemoryCacheSetAndRemoveTests.cs:line 655
----- Inner Stack Trace -----

Guia do colaborador

Direção de pesquisa
Foque no arquivo `src/libraries/Microsoft.Extensions.Caching.Memory/tests/MemoryCacheSetAndRemoveTests.cs` e nos três métodos de teste desabilitados: `GetAndSet AreThreadSafe AndUpdatesNeverLeavesNullValues`, `OvercapacityPurge AreThreadSafe` e `AddAndReplaceEntries AreThreadSafe`. Os testes são sensíveis ao tempo e causam falhas intermitentes. Revise o código de teste existente para identificar onde o tempo é usado (por exemplo, atrasos ou timeouts) e substitua os por mecanismos de sincronização determinísticos, como `ManualResetEventSlim` ou `TaskCompletionSource`, para controlar o intercalamento de threads. Examine outros testes no mesmo arquivo que não são sensíveis ao tempo para obter exemplos de padrões adequados.
Pilha de tecnologia
csharp
Domain
backend
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#unit testingthreading basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70