yegor256/takes

Refactor TkHtmlTest to improve structure and edge case coverage (relates to #1408)

Aberta

#1.435 aberto em 24 de jul. de 2025

Ver no GitHub
 (2 comentários) (0 reação) (0 responsável)Java (215 forks)github user discovery
enhancementhelp wanted

Métricas do repositório

Stars
 (875 estrelas)
Métricas de merge de PR
 (Mesclagem média 95d 2h) (22 fundiu PRs em 30d)

Description

Improve and Refactor TkHtmlTest for Maintainability and Coverage (related to #1408)

The current implementation of TkHtmlTest can be improved in the following ways:

  1. Duplication: Each test manually builds a nearly identical expected response using Joined(..., body). This can be extracted into a helper method or reused variable.

  2. Edge Cases Missing:

    • No tests for null or empty input.
    • No assertion that Content-Type and Content-Length headers are semantically correct (not just matched by string).

  3. Structure Validation:

    • Could add a test to verify that the body actually starts/ends with <html> tags to ensure semantic HTML is passed.

  4. Test Readability:

    • Introduce @DisplayName annotations or comment grouping to make intent clearer.

A can provide a PR if the maintainers agree.

Guia do colaborador

Direção de pesquisa
Inspecione a classe TkHtmlTest existente, entenda como ela constrói respostas esperadas com Joined e identifique duplicações. Procure casos de borda como entrada nula ou vazia e verifique se os cabeçalhos Content Type e Content Length estão semanticamente corretos. Considere adicionar anotações @DisplayName para esclarecer a intenção dos testes.
Pilha de tecnologia
java
Domain
backendtesting
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.
1-3 horas
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
GitJavaMavenJUnit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75