influxdata/telegraf

Add tests for internal/snmp

Open

#10 531 ouverte le 27 janv. 2022

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Go (4 161 forks)batch import
area/snmphelp wantedsize/mtesting request

Métriques du dépôt

Stars
 (9 892 stars)
Métriques de merge PR
 (Merge moyen 2j 12h) (136 PRs mergées en 30 j)

Description

Feature Request

Opening a feature request kicks off a discussion.

Proposal:

Add tests for internal/snmp package:

  • ClearCache
  • GetIndex
  • LoadMibsFromPath #10551
  • NewWrapper
  • SetAgent
  • SnmpTranslateCall
  • TrapLookup #10430

Some of these are already covered by tests in plugin/inputs/snmp, but best is to test them at the correct place.

Current behavior:

Changes in code do change behaviour of some plugins.

Desired behavior:

Tests prevent changes in behaviour to the end-users.

Guide contributeur

Direction de recherche
Examinez les tests existants dans plugin/inputs/snmp pour les modèles et écrivez des tests unitaires pour chaque fonction non répertoriée dans internal/snmp en utilisant les pratiques de test Go standard.
Stack technique
go
Domaine
backend
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.
Une demi journée
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
Go
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75