JuliaLang/julia

Replace uses of tempname in CI by mktemp

Open

#43.597 aperta il 29 dic 2021

Vedi su GitHub
 (10 commenti) (0 reazioni) (0 assegnatari)Julia (5773 fork)batch import
cigood first issuehelp wanted

Metriche repository

Star
 (48.709 star)
Metriche merge PR
 (Merge medio 20g 6h) (157 PR mergiate in 30 g)

Descrizione

As of #38879 our temp path generation depends on the global RNG. That's probably not a bad idea in general, except that various parts of the test suite seed the global RNG, so this will cause reliable collisions in the test suite, so we'll get in trouble with tests stomping on each other. We should audit all uses of tempname in the test suite and replace them with mktemp unless they test the tempname function specifically.

Guida contributor

Direzione di ricerca
Trova tutti gli usi di `tempname` nella suite di test e sostituiscili con `mktemp` (o `tempname` con seed esplicito) per evitare collisioni dovute al RNG seedato, tranne dove si testa `tempname` stesso.
Tech stack
julia
Dominio
testing
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JuliaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70