abcxyz / abcxyz/jvs

Write UT for public methods if it is possible

Aberta
#108 0 comentários 0 reações 0 responsáveis Ver no GitHub
cleanup good first issue
Linguagem predominante
Go
Estrelas
8
Forks
1
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

### TL;DR

Write UT for public methods instead of private methods if it is possible

https://github.com/abcxyz/jvs/blob/23cdda185a8fc277ba798bfaf3e34543458f1934/pkg/jvscrypto/cert_action_api_test.go#L306 and
https://github.com/abcxyz/jvs/blob/23cdda185a8fc277ba798bfaf3e34543458f1934/pkg/jvscrypto/key_hosting_test.go#L124

### Detailed design

_No response_

### Alternatives considered

_No response_

### Additional information

_No response_

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Look at the two linked test files: pkg/jvscrypto/cert_action_api_test.go and pkg/jvscrypto/key_hosting_test.go. Identify the public methods being tested indirectly via private methods. Refactor the tests to directly test the public methods instead. Run the existing tests to ensure they still pass after the change.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Domínio
testing
Tipo de issue
Refatoração
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
50/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.