[Feature]: New and replacement integration / regression tests
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Ambito
- testing-qa
Direzione di ricerca
Inizia esaminando i test di integrazione rimossi o non funzionanti tc_time_outside_window_test.py e td_leakage_between_runs_test.py, quindi leggi i commenti in PR 249. Chiarisci come sono configurati i frammenti di TriggerCandidate e le finestre di richiesta di TriggerDecision prima di ripristinare i test; il lavoro è completato quando i test di regressione sostitutivi vengono eseguiti correttamente, includendo qualsiasi copertura utile di TC-merging.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
Two of the three existing integtests in the trigger repo are broken. This has been true for a while, and I am going to remove them, for now.
However, the functionality that they once tested would still be good to have. So, I'm filing this Issue to remind us to come back to these tests someday.
- the
tc_time_outside_window_test.pytest attempts to validate the expected behavior of TriggerCandidate fragments when the TriggerDecision request window doesn't include the TC time. Unfortunately, at the moment, it is not clear to me how we expect TC fragments to be populated in various conditions, and I'm not sure of the current way (or ways) that we specify TC times versus other times. - the
td_leakage_between_runs_test.pytest attempted to validate the graceful stopping of TriggerDecision messages between the MLT and the DFO. There are comments in this this PR. The stumbling block to getting this integtest working again now is that it is not clear how we would configure the conditions that are likely to demonstrate the problem
In summary, it would be good to bring back these tests someday.
Independent of that, we can imagine other tests that would be useful. Such as ones that validate the behavior of TC merging.
Potential impact radius
Small/Isolated
Reason for change
Validating expected functionality is a good thing
Suggested implementations
No response
Testing suggestions
No response
Anything else?
No response
- Lingua principale
- C++
- Stelle
- 0
- Fork
- 8
- Merge medio
- 3h 50m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di DUNE-DAQ/trigger
-
enhancement
Difficoltà 3/5 Mezza giornata Idoneità per principianti 65/100
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
Tutte le issue di DUNE-DAQ/trigger
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
[BUG] Clipboard History: Buffer over-read in WideCharToMultiByte on double-click pasteForse già presa @softmgr l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
notepad-plus-plus/notepad-plus-plus#18519 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
justcallmekoko/ESP32Marauder#1635 ·
I maintainer di solito rispondono entro 1 giorno
-
3.7.6-dev commit 80481ea - files not centered when resolution differs and zoom = "pixel-to-pixel"Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
grizzlypeak3d/DJV#931 ·