apache / apache/datafusion-sqlparser-rs

Seemingly pointless test

未关闭
#1,807 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Rust
星标
3.5k
派生
772
平均合并
4 天 9 小时
30 天内合并 PR
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 摘要。