apache / apache/datafusion-sqlparser-rs

Seemingly pointless test

オープン
#1,807 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Rust
スター
3.5k
フォーク
772
平均マージ
4日 9時間
マージ済み PR(30日)
17

説明

While implementing the support for [`INHERITS`](https://www.postgresql.org/docs/current/ddl-inherit.html), I happened to find [this test](https://github.com/jmhain/sqlparser-rs/blob/7723ea56c5119c7d1a15233c18eb2aaf48b60dc0/tests/sqlparser_postgres.rs#L505-L515) which contains in its name the word `inherit` and I suppose may reference the [CREATE ROLE INHERIT](https://www.postgresql.org/docs/current/sql-createrole.html), but it not present in the actual test.

If this test is actually pointless, it may be best to remove it. Do let me know how to proceed.

```rust
#[test]
fn parse_create_table_with_inherit() {
let sql = "\
CREATE TABLE bazaar.settings (\
settings_id UUID PRIMARY KEY DEFAULT uuid_generate_v4() NOT NULL, \
user_id UUID UNIQUE, \
value TEXT[], \
use_metric BOOLEAN DEFAULT true\
)";
pg().verified_stmt(sql);
}

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

tests/sqlparser_postgres.rs の 505-515 行から始め、テスト名とそれが解析する SQL を比較します。特に、issue にある PostgreSQL INHERITS および CREATE ROLE INHERIT の参照を確認してください。テストが意図した動作をカバーしているか確認します。完了とは、テストが一貫した形で削除または改名され、関連する Rust テストがパスすることを意味します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
rust
領域
testing-qa
issue の種類
リファクタリング
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。