Skip to content

Commit a5325d0

Browse files
sudosteveweiznich
authored andcommitted
Fix test to catch foreign key issue in --diff-schema generation
1 parent ffedce2 commit a5325d0

File tree

10 files changed

+36
-36
lines changed

10 files changed

+36
-36
lines changed
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
CREATE TABLE `users`(
2-
`id` INTEGER NOT NULL PRIMARY KEY,
2+
`user_id` INTEGER NOT NULL PRIMARY KEY,
33
`name` TEXT NOT NULL
44
);

diesel_cli/tests/generate_migrations/diff_add_table_with_fk/mysql/schema_out.rs/expected.snap

+6-6
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,21 @@ description: "Test: diff_add_table_with_fk"
66
// @generated automatically by Diesel CLI.
77

88
diesel::table! {
9-
posts (id) {
10-
id -> Integer,
9+
posts (post_id) {
10+
post_id -> Integer,
1111
title -> Text,
1212
body -> Nullable<Text>,
13-
user_id -> Integer,
13+
foreign_key_user_id -> Integer,
1414
}
1515
}
1616

1717
diesel::table! {
18-
users (id) {
19-
id -> Integer,
18+
users (user_id) {
19+
user_id -> Integer,
2020
name -> Text,
2121
}
2222
}
2323

24-
diesel::joinable!(posts -> users (user_id));
24+
diesel::joinable!(posts -> users (foreign_key_user_id));
2525

2626
diesel::allow_tables_to_appear_in_same_query!(posts, users,);

diesel_cli/tests/generate_migrations/diff_add_table_with_fk/mysql/up.sql/expected.snap

+3-3
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@ description: "Test: diff_add_table_with_fk"
66
-- Your SQL goes here
77

88
CREATE TABLE `posts`(
9-
`id` INTEGER NOT NULL PRIMARY KEY,
9+
`post_id` INTEGER NOT NULL PRIMARY KEY,
1010
`title` TEXT NOT NULL,
1111
`body` TEXT,
12-
`user_id` INTEGER NOT NULL,
13-
FOREIGN KEY (`user_id`) REFERENCES `users`(`id`)
12+
`foreign_key_user_id` INTEGER NOT NULL,
13+
FOREIGN KEY (`foreign_key_user_id`) REFERENCES `users`(`user_id`)
1414
);
1515

1616

Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
CREATE TABLE "users"(
2-
"id" INTEGER NOT NULL PRIMARY KEY,
2+
"user_id" INTEGER NOT NULL PRIMARY KEY,
33
"name" TEXT NOT NULL
44
);

diesel_cli/tests/generate_migrations/diff_add_table_with_fk/postgres/schema_out.rs/expected.snap

+6-6
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,21 @@ snapshot_kind: text
66
// @generated automatically by Diesel CLI.
77

88
diesel::table! {
9-
posts (id) {
10-
id -> Int4,
9+
posts (post_id) {
10+
post_id -> Int4,
1111
title -> Text,
1212
body -> Nullable<Text>,
13-
user_id -> Int4,
13+
foreign_key_user_id -> Int4,
1414
}
1515
}
1616

1717
diesel::table! {
18-
users (id) {
19-
id -> Int4,
18+
users (user_id) {
19+
user_id -> Int4,
2020
name -> Text,
2121
}
2222
}
2323

24-
diesel::joinable!(posts -> users (user_id));
24+
diesel::joinable!(posts -> users (foreign_key_user_id));
2525

2626
diesel::allow_tables_to_appear_in_same_query!(posts, users,);

diesel_cli/tests/generate_migrations/diff_add_table_with_fk/postgres/up.sql/expected.snap

+3-3
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@ description: "Test: diff_add_table_with_fk"
66
-- Your SQL goes here
77

88
CREATE TABLE "posts"(
9-
"id" INTEGER NOT NULL PRIMARY KEY,
9+
"post_id" INTEGER NOT NULL PRIMARY KEY,
1010
"title" TEXT NOT NULL,
1111
"body" TEXT,
12-
"user_id" INTEGER NOT NULL,
13-
FOREIGN KEY ("user_id") REFERENCES "users"("id")
12+
"foreign_key_user_id" INTEGER NOT NULL,
13+
FOREIGN KEY ("foreign_key_user_id") REFERENCES "users"("user_id")
1414
);
1515

1616

Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
diesel::table! {
2-
users {
3-
id -> Integer,
2+
users(user_id) {
3+
user_id -> Integer,
44
name -> Text,
55
}
66
}
77

88
diesel::table! {
9-
posts {
10-
id -> Integer,
9+
posts(post_id) {
10+
post_id -> Integer,
1111
title -> Text,
1212
body -> Nullable<Text>,
13-
user_id -> Integer,
13+
foreign_key_user_id -> Integer,
1414
}
1515
}
1616

17-
diesel::joinable!(posts -> users (user_id));
17+
diesel::joinable!(posts -> users (foreign_key_user_id));
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
CREATE TABLE `users`(
2-
`id` INTEGER NOT NULL PRIMARY KEY,
2+
`user_id` INTEGER NOT NULL PRIMARY KEY,
33
`name` TEXT NOT NULL
44
);

diesel_cli/tests/generate_migrations/diff_add_table_with_fk/sqlite/schema_out.rs/expected.snap

+6-6
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,21 @@ snapshot_kind: text
66
// @generated automatically by Diesel CLI.
77

88
diesel::table! {
9-
posts (id) {
10-
id -> Integer,
9+
posts (post_id) {
10+
post_id -> Integer,
1111
title -> Text,
1212
body -> Nullable<Text>,
13-
user_id -> Integer,
13+
foreign_key_user_id -> Integer,
1414
}
1515
}
1616

1717
diesel::table! {
18-
users (id) {
19-
id -> Integer,
18+
users (user_id) {
19+
user_id -> Integer,
2020
name -> Text,
2121
}
2222
}
2323

24-
diesel::joinable!(posts -> users (user_id));
24+
diesel::joinable!(posts -> users (foreign_key_user_id));
2525

2626
diesel::allow_tables_to_appear_in_same_query!(posts, users,);

diesel_cli/tests/generate_migrations/diff_add_table_with_fk/sqlite/up.sql/expected.snap

+3-3
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,11 @@ description: "Test: diff_add_table_with_fk"
66
-- Your SQL goes here
77

88
CREATE TABLE `posts`(
9-
`id` INTEGER NOT NULL PRIMARY KEY,
9+
`post_id` INTEGER NOT NULL PRIMARY KEY,
1010
`title` TEXT NOT NULL,
1111
`body` TEXT,
12-
`user_id` INTEGER NOT NULL,
13-
FOREIGN KEY (`user_id`) REFERENCES `users`(`id`)
12+
`foreign_key_user_id` INTEGER NOT NULL,
13+
FOREIGN KEY (`foreign_key_user_id`) REFERENCES `users`(`user_id`)
1414
);
1515

1616

0 commit comments

Comments
 (0)