influxdata/telegraf

Add tests for internal/snmp

Open

#10.531 aperta il 27 gen 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Go (4161 fork)batch import
area/snmphelp wantedsize/mtesting request

Metriche repository

Star
 (9892 star)
Metriche merge PR
 (Merge medio 2g 12h) (136 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina i test esistenti in plugin/inputs/snmp per i pattern e scrivi test unitari per ogni funzione non elencata in internal/snmp utilizzando le pratiche standard di test Go.
Tech stack
go
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75