trinodb/trino

Add test coverage for blackhole connector

Open

Aperta il 1 ago 2023

Vedi su GitHub
 (6 commenti) (0 reazioni) (1 assegnatario)Java (9113 star) (2678 fork)batch import
good first issuetest

Descrizione

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

Guida contributor

Tech stack
java
Dominio
testing
Tipo issue
test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
needs maintainer response
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
mostly clear
Prerequisiti
JavaBasic understanding of Trino connectorsFamiliarity with test refactoring
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
45
Direzione di ricerca
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.