New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove residual NamedConstraint gating #2199
Conversation
5da5746
to
50233b0
Compare
295f998
to
96f5c12
Compare
3bef1ff
to
cbc229d
Compare
In particular: - Foreign key constraint names were not rendered in introspection - Default constraint names were not diffed in migrations
cbc229d
to
de76a9d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Posthumous approval.
use barrel::types; | ||
use enumflags2::BitFlags; | ||
use expect_test::expect; | ||
use introspection_engine_tests::test_api::*; | ||
use introspection_engine_tests::TestResult; | ||
use test_macros::test_connector; | ||
|
||
#[test_connector(exclude(Mysql, Mssql))] | ||
#[test_connector(exclude(Mysql, Mssql, Sqlite))] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -131,7 +131,10 @@ async fn a_one_to_one_relation(api: &TestApi) -> TestResult { | |||
t.add_column("id", types::integer().increments(true)); | |||
t.add_column("user_id", types::integer().nullable(true)); | |||
|
|||
t.add_foreign_key(&["user_id"], "User", &["id"]); | |||
t.add_constraint( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ngggh this API from Barrel (I wrote). We should maybe just stop using it...
In particular: