dotnet/runtime

Rewrite 3 MemoryCache tests to not be timing sensitive

Geschlossen

#72.890 geöffnet am 26.07.2022

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (5.445 Forks)batch import
area-Extensions-Cachingdisabled-testhelp wanted

Repository-Metriken

Stars
 (17.886 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 12T 11h) (661 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Konzentrieren Sie sich auf die Datei `src/libraries/Microsoft.Extensions.Caching.Memory/tests/MemoryCacheSetAndRemoveTests.cs` und die drei deaktivierten Testmethoden: `GetAndSet AreThreadSafe AndUpdatesNeverLeavesNullValues`, `OvercapacityPurge AreThreadSafe` und `AddAndReplaceEntries AreThreadSafe`. Die Tests sind zeitkritisch und verursachen intermittierende Fehler. Überprüfen Sie den vorhandenen Testcode, um Stellen zu identifizieren, an denen Timing verwendet wird (z. B. Verzögerungen oder Timeouts), und ersetzen Sie diese durch deterministische Synchronisationsmechanismen wie `ManualResetEventSlim` oder `TaskCompletionSource`, um die Thread Verschachtelung zu steuern. Sehen Sie sich andere Tests in derselben Datei an, die nicht zeitkritisch sind, um Beispiele für geeignete Muster zu finden.
Tech Stack
csharp
Domain
backend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#unit testingthreading basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70