morelinq/MoreLINQ

Revise tests to avoid using `Eumerable.Range`

Open

#1044 aperta il 16 nov 2023

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)C# (406 fork)batch import
Component-Testsgood first issuehelp wanted

Metriche repository

Star
 (3495 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Starting with .NET 8, Eumerable.Range returns an instance of a private RangeIterator that implements IList<T> (see also dotnet/runtime#88249). A lot of the unit tests potentially went with the assumption that Eumerable.Range returns a pure sequence. There are some operators in MoreLINQ that are optimised for sources that are collections and/or lists so tests using Eumerable.Range may be testing those optimised paths rather than the actual intended path iterating the source as an IEnumerable<>. Those tests need to be revised to avoid Enumerable.Range altogether.

Potential candidates that need revision:

Guida contributor

Direzione di ricerca
Identifica ogni file di test elencato nel problema che utilizza Enumerable.Range e sostituiscilo con un generatore di sequenze diverso (ad esempio Enumerable.Range(0, n).Select(x => x) o un semplice array) per garantire che il test percorra il percorso puro di IEnumerable.
Tech stack
csharp
Dominio
testing
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
C#LINQunit testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65