trinodb/trino

Add test coverage for blackhole connector

Open

#18,482 opened on 2023年8月1日

GitHub で見る
 (6 comments) (0 reactions) (1 assignee)Java (9,113 stars) (2,678 forks)batch import
good first issuetest

説明

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

コントリビューターガイド

技術スタック
java
領域
testing
Issue 種別
test
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
needs maintainer response
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
JavaBasic understanding of Trino connectorsFamiliarity with test refactoring
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
45
調査方針
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.
Add test coverage for blackhole connector · trinodb/trino#18482 | Good First Issue