morelinq/MoreLINQ

Revise tests to avoid using `Eumerable.Range`

Open

#1 044 ouverte le 16 nov. 2023

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)C# (406 forks)batch import
Component-Testsgood first issuehelp wanted

Métriques du dépôt

Stars
 (3 495 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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:

Guide contributeur

Direction de recherche
Identifiez chaque fichier de test listé dans le problème qui utilise Enumerable.Range et remplacez le par un générateur de séquence différent (par exemple Enumerable.Range(0, n).Select(x => x) ou un tableau simple) pour garantir que le test emprunte le chemin pur IEnumerable.
Stack technique
csharp
Domaine
testing
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#LINQunit testing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65