apache / apache/datafusion-sqlparser-rs
Seemingly pointless test
- 主要言語
- 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