influxdata/telegraf

Add tests for internal/snmp

Open

#10.531 aberto em 27 de jan. de 2022

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Go (4.161 forks)batch import
area/snmphelp wantedsize/mtesting request

Métricas do repositório

Stars
 (9.892 stars)
Métricas de merge de PR
 (Mesclagem média 2d 12h) (136 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
Examine os testes existentes em plugin/inputs/snmp para padrões e escreva testes unitários para cada função não listada em internal/snmp usando práticas padrão de teste Go.
Pilha de tecnologia
go
Domain
backend
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Go
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75