trinodb/trino

Add test coverage for blackhole connector

Open

#18.482 aberto em 1 de ago. de 2023

Ver no GitHub
 (6 comments) (0 reactions) (1 assignee)Java (9.113 stars) (2.678 forks)batch import
good first issuetest

Description

During work on this PR https://github.com/trinodb/trino/pull/18016, noticed that test coverage is pretty small. I think we could refactor current Smoke test to be more consistent with BaseConnectorSmokeTest. And think about BaseConnectorTest

Guia do colaborador

Pilha de tecnologia
java
Domain
testing
Tipo Issue
test
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
needs maintainer response
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
JavaBasic understanding of Trino connectorsFamiliarity with test refactoring
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
45
Direção de pesquisa
Review the linked PR (https://github.com/trinodb/trino/pull/18016) to understand the current blackhole connector smoke test. Examine the existing BaseConnectorSmokeTest and BaseConnectorTest classes in the Trino repository to understand the expected patterns. Identify gaps in test coverage and propose a refactoring plan. Engage with maintainers on the issue to clarify the scope and preferred approach before coding.