apache / apache/datafusion-sqlparser-rs

Seemingly pointless test

Ouverte
#1,807 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Rust
Étoiles
3.5k
Forks
772
Merge moyen
4 j 9 h
PR mergées (30 j)
17

Description

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);
}

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par les lignes 505-515 de tests/sqlparser_postgres.rs et comparez le nom du test avec le SQL qu’il analyse, en particulier les références PostgreSQL INHERITS et CREATE ROLE INHERIT dans l’issue. Vérifiez que le test couvre bien le comportement attendu ; le travail est terminé lorsque le test est supprimé ou renommé de manière cohérente et que les tests Rust concernés réussissent.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
rust
Domaine
testing-qa
Type d'issue
Refactorisation
Difficulté
2/5
Temps estimé
1-3 heures
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.