sindresorhus/type-fest

ReadonlyDeep: add test for function or method returning an object?

Fermée

#826 ouverte le 7 mars 2024

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)TypeScript (471 forks)batch import
help wanted

Métriques du dépôt

Stars
 (12 328 étoiles)
Métriques de merge PR
 (Merge moyen 3j 5h) (11 PRs mergées en 30 j)

Description

Reading through the ReadonlyDeep type and its tests, I can't tell if ReadonlyDeep<() => Map>() would return a Map or a ReadonlyDeep<Map>. The tests just don't cover that sort of case.

Guide contributeur

Direction de recherche
Examinez le fichier de test existant test d/readonly deep.ts pour comprendre le modèle de test. Passez en revue la définition du type ReadonlyDeep dans source/readonly deep.d.ts pour déterminer comment il gère les types de retour de fonction. L'objectif est d'ajouter un cas de test pour `ReadonlyDeep<() => Map>` et de vérifier le comportement attendu en vérifiant si le résultat est un Map ou ReadonlyDeep<Map>. Reportez vous aux tests existants pour la structure et les assertions.
Stack technique
typescript
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-3 heures
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
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70