dotnet/runtime

Unit tests are failing if IPv6 is not configured

Open

#23 798 ouverte le 10 oct. 2017

Voir sur GitHub
 (13 commentaires) (0 réactions) (0 assignés)C# (5 445 forks)batch import
area-System.Nethelp wantedtest-bug

Métriques du dépôt

Stars
 (17 886 stars)
Métriques de merge PR
 (Merge moyen 12j 11h) (661 PRs mergées en 30 j)

Description

I bump to this while running test on FreeBSD, however this issue is not OS specific. Some tests make silent assumption about IPv6 and they fail if system is not configured to meet that. This is unpleasant for anybody from community to run tests and get good pass.

The easy repro for Linux is: sysctl -w net.ipv6.conf.all.disable_ipv6=1 ./build-tests.sh

The expected behavior would be that tests needing IPv6 would be skipped. Right now, some tests even hang for very long time.

Guide contributeur

Direction de recherche
Le problème demande de sauter les tests dépendants d'IPv6 lorsque IPv6 n'est pas configuré. Le contributeur doit identifier les fichiers de test spécifiques qui reposent sur IPv6 (probablement dans les suites de tests réseau) et ajouter un saut conditionnel en utilisant des vérifications d'environnement (par exemple `Socket.OSSupportsIPv6`). Consultez les commentaires du problème pour toute suggestion de fichiers ou d'approches. L'objectif est de garantir que les tests ne échouent pas ou ne se bloquent pas, et soient sautés de manière élégante. Un mainteneur a mentionné l'utilisation d'un attribut `[ConditionalFact]` ou d'un modèle similaire—explorez la logique de saut existante dans le projet de test.
Stack technique
csharp
Domaine
backendtesting
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.
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
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70