offline-transactions interruption oracle fails when a random sample misses a landing
Maintainer antworten meist innerhalb von 1 Tag
Ein zugehöriger Pull Request wurde bereits gemerged.
- #2144 von @KyleAMathews — gemerged
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- Ein halber Tag
- Anfängerfreundlichkeit
- 62/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- typescript
- Bereich
- testing
Rechercherichtung
Start with packages/offline-transactions/tests/interruption-settlement-oracle.property.test.ts, where the failing assertion compares the landings reached by a random sample with the full landing list. Run the test several times to reproduce the intermittent failure. Done means the coverage check no longer depends on random sampling (either fixed named histories per landing, or moved to the fixed-seed campaign), and the random law check still runs and passes repeatedly.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Summary
packages/offline-transactions/tests/interruption-settlement-oracle.property.test.ts fails at random. The failing check asserts that the random histories cover every interruption landing. It is not a behavior check. When a run draws no history that reaches a refused landing, the test fails.
Failure
The test "keeps settlement and ownership for an interruption at any microtask turn" fails with:
AssertionError: interruption landings: expected [ 'dispose issued', …(5) ] to deeply equal [ 'dispose issued', …(7) ]
- "dispose refused",
- "leadership refused",
It failed CI on #2128, which does not change offline-transactions. CI ran it because it tests the pull request merged with main. On main (cd92156d6), 1 of 3 local runs failed the same way.
Cause
The check compares the landings that one random sample reached with the full set of landings. A sample of config.runs histories does not always reach every landing, especially the refused ones. Any run that misses one fails, even when every history it does run satisfies the law.
Suggested fix
- Prove coverage with fixed, named histories (one per landing), not with a random sample.
- Or move the coverage check to the fixed-seed campaign only, where the sample is deterministic.
- Keep the random campaign for the settlement and ownership law itself.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 3.9k
- Forks
- 272
- Ø Merge
- 1 T. 48 Min.
- Gemergte PRs (30 T.)
- 238
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine 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 TanStack/db
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 12/100
Maintainer antworten meist innerhalb von 1 Tag
-
Ordered, limited live query over a string key commits `loading` first (and `orderBy().findOne()` since 0.11.1)Evtl. vergeben @KyleAMathews hat das heute übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 18/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 18/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 18/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 8/100
Maintainer antworten meist innerhalb von 1 Tag
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
AvaProtocol/ava-sdk-js#276 ·
Maintainer antworten meist innerhalb von 1 Tag
-
useFragment does not require `variables` when the fragment's variables type has required fieldsOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
apollographql/apollo-client#13493 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Custom hosts written as trailing-dot FQDNs (`host.local.`) aren't detected as local backendsEvtl. vergeben @roboomp hat das heute übernommen. Offenbug prio:p2 providers triaged
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
can1357/oh-my-pi#15798 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
code-yeongyu/oh-my-openagent#10172 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
PDFCraftTool/pdfcraft#189 ·