morelinq/MoreLINQ

Revise tests to avoid using `Eumerable.Range`

Open

#1.044 aberto em 16 de nov. de 2023

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)C# (406 forks)batch import
Component-Testsgood first issuehelp wanted

Métricas do repositório

Stars
 (3.495 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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:

Guia do colaborador

Direção de pesquisa
Identifique cada arquivo de teste listado no problema que usa Enumerable.Range e substitua o por um gerador de sequência diferente (por exemplo, Enumerable.Range(0, n).Select(x => x) ou um array simples) para garantir que o teste execute o caminho puro de IEnumerable.
Pilha de tecnologia
csharp
Domain
testing
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#LINQunit testing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65