dotnet/runtime

Rewrite 3 MemoryCache tests to not be timing sensitive

Fermée

#72 890 ouverte le 26 juil. 2022

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)C# (5 445 forks)batch import
area-Extensions-Cachingdisabled-testhelp wanted

Métriques du dépôt

Stars
 (17 886 étoiles)
Métriques de merge PR
 (Merge moyen 12j 11h) (661 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Concentrez vous sur le fichier `src/libraries/Microsoft.Extensions.Caching.Memory/tests/MemoryCacheSetAndRemoveTests.cs` et les trois méthodes de test désactivées : `GetAndSet AreThreadSafe AndUpdatesNeverLeavesNullValues`, `OvercapacityPurge AreThreadSafe` et `AddAndReplaceEntries AreThreadSafe`. Les tests sont sensibles au timing et provoquent des échecs intermittents. Examinez le code de test existant pour identifier où le timing est utilisé (par exemple, des délais ou des timeouts) et remplacez les par des mécanismes de synchronisation déterministes comme `ManualResetEventSlim` ou `TaskCompletionSource` pour contrôler l'entrelacement des threads. Consultez d'autres tests dans le même fichier qui ne sont pas sensibles au timing pour des exemples de modèles appropriés.
Stack technique
csharp
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#unit testingthreading basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70