stretchr/testify

assert: NotNil accepts non-nillable types and doesn't fail

Ouverte

#570 ouverte le 7 mars 2018

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)Go (1 704 forks)batch import
help wanted

Métriques du dépôt

Stars
 (25 958 étoiles)
Métriques de merge PR
 (Merge moyen 6j 15h) (2 PRs mergées en 30 j)

Description

https://github.com/stretchr/testify/blob/b89eecf5ca5db6d3ba60b237ffe3df7bafb7662f/assert/assertions.go#L390-L410

isNil will always return false, so NotNil will always return true for int, string, struct, etc.

I would expect it to error with "invalid type".

Guide contributeur

Direction de recherche
Examinez la fonction 'isNil' dans assert/assertions.go et modifiez 'NotNil' pour rejeter les types non nullables.
Stack technique
go
Domaine
testing qa
Type d'issue
Bug
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.
Moins d'une heure
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
Gotestify
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70