Skip to content

Commit

Permalink
test: Add FK constraint preserves NOT NULL
Browse files Browse the repository at this point in the history
  • Loading branch information
andrew-farries committed Jan 19, 2024
1 parent 59a2b4c commit eb08d57
Showing 1 changed file with 76 additions and 0 deletions.
76 changes: 76 additions & 0 deletions pkg/migrations/op_set_fk_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -443,6 +443,82 @@ func TestSetForeignKey(t *testing.T) {
}, testutils.CheckViolationErrorCode)
},
},
{
name: "not null is preserved when adding a foreign key constraint",
migrations: []migrations.Migration{
{
Name: "01_add_tables",
Operations: migrations.Operations{
&migrations.OpCreateTable{
Name: "users",
Columns: []migrations.Column{
{
Name: "id",
Type: "serial",
Pk: true,
},
{
Name: "name",
Type: "text",
},
},
},
&migrations.OpCreateTable{
Name: "posts",
Columns: []migrations.Column{
{
Name: "id",
Type: "serial",
Pk: true,
},
{
Name: "title",
Type: "text",
Nullable: true,
},
{
Name: "user_id",
Type: "integer",
Nullable: false,
},
},
},
},
},
{
Name: "02_add_fk_constraint",
Operations: migrations.Operations{
&migrations.OpAlterColumn{
Table: "posts",
Column: "user_id",
References: &migrations.ForeignKeyReference{
Name: "fk_users_id",
Table: "users",
Column: "id",
},
Up: "(SELECT CASE WHEN EXISTS (SELECT 1 FROM users WHERE users.id = user_id) THEN user_id ELSE NULL END)",
Down: "user_id",
},
},
},
},
afterStart: func(t *testing.T, db *sql.DB) {
// Inserting a row that violates the NOT NULL constraint on `user_id` fails.
MustNotInsert(t, db, "public", "02_add_fk_constraint", "posts", map[string]string{
"id": "1",
"title": "post by alice",
}, testutils.NotNullViolationErrorCode)
},
afterRollback: func(t *testing.T, db *sql.DB) {
},
afterComplete: func(t *testing.T, db *sql.DB) {
// Inserting a row that violates the NOT NULL constraint on `user_id` fails.
MustNotInsert(t, db, "public", "02_add_fk_constraint", "posts", map[string]string{
"id": "1",
"title": "post by alice",
}, testutils.NotNullViolationErrorCode)
},
},
})
}

Expand Down

0 comments on commit eb08d57

Please sign in to comment.