trinodb/trino

Migrate parser tests to use ParserAssert

Open

#7,520 opened on 2021年4月7日

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

説明

Tests in https://github.com/trinodb/trino/blob/master/core/trino-parser/src/test/java/io/trino/sql/parser/TestSqlParser.java that currently rely on assertExpression and assertStatement need to be updated to use ParserAssert.

Examples: https://github.com/trinodb/trino/blob/master/core/trino-parser/src/test/java/io/trino/sql/parser/TestSqlParser.java#L234 https://github.com/trinodb/trino/blob/master/core/trino-parser/src/test/java/io/trino/sql/parser/TestSqlParser.java#L1247

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

技術スタック
java
領域
backendtesting
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
blocked
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
JavaJUnit test experience
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
Examine the provided examples in TestSqlParser.java at lines 234 and 1247 to understand the current usage of assertExpression and assertStatement. Find the ParserAssert class in the same module to understand its API. Then systematically replace all occurrences of those assertion methods with equivalent ParserAssert calls, ensuring tests still pass.