Remove the sleep-based lease setup from the two Redis reaper tests
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 92/100
- Issue-Typ
- Refactoring
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- python, redis
- Bereich
- backend, testing-qa
Rechercherichtung
Beginne in tests/backends/test_redis.py mit test_running_reaper__fails_expired_tasks und test_stale_acknowledge__is_noop. Lies den Hilfsfunktion _expire_lease(backend, task_id) auf Modulebene und die vorhandenen Reap-Tests, die sie verwenden, und ersetze dann die auf sleep basierende Ablaufkonfiguration in beiden Tests. Führe die beiden genannten Tests aus und bestätige, dass sie ohne die 1,1-Sekunden-sleeps bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Found during the SuperJoe review of the lease-expiry retry patch (4ec9068..d5a1fdb). It is out of the scope of that patch because both tests predate it.
tests/backends/test_redis.py still expires a lease with a sleep of 1.1 seconds:
# tests/backends/test_redis.py
time.sleep(1.1)
# Run the broker to reap the running set
broker = RedisBroker(backend)
broker.main()
The two tests are test_running_reaper__fails_expired_tasks and test_stale_acknowledge__is_noop.
The patch added the module-level _expire_lease(backend, task_id) helper. The helper sets the running-set score back to 0. Every new reap test already uses it. Both old tests can use it too. Then the suite loses about 2.2 seconds of wall clock and the risk of a timing flake.
- Vorherrschende Sprache
- Python
- Sterne
- 19
- Forks
- 1
- Ø Merge
- 14 Std. 37 Min.
- Gemergte PRs (30 T.)
- 22
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus codingjoe/threadmill
-
Add worker pool telemetryEvtl. wieder frei @codingjoe hat das vor 104 Tagen übernommen, und es ist kein Pull Request offen. Offenenhancement
codingjoe/threadmill#18 · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in codingjoe/threadmill
Ähnliche Issues
-
enhancement from-review
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
khuisman/mcp-gee-sweet#1010 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug confirmed issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
open-webui/open-webui#32333 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
ll7/robot_sf_ll7#10368 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
flop-labs/technocore-chat#982 ·
Maintainer antworten meist innerhalb von 1 Tag