Add default issue

Author: owjs3901

Cargo.lock

@@ -3,9 +3,6 @@ use vespertide_core::MigrationPlan;
 use crate::error::QueryError;
 use crate::sql::{BuiltQuery, build_action_queries};
 
-#[cfg(test)]
-use crate::sql::DatabaseBackend;
-
 pub fn build_plan_queries(plan: &MigrationPlan) -> Result<Vec<BuiltQuery>, QueryError> {
     let mut queries: Vec<BuiltQuery> = Vec::new();
     for action in &plan.actions {
@@ -17,6 +14,7 @@ pub fn build_plan_queries(plan: &MigrationPlan) -> Result<Vec<BuiltQuery>, Query
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::sql::DatabaseBackend;
     use rstest::rstest;
     use vespertide_core::{
         ColumnDef, ColumnType, MigrationAction, MigrationPlan, SimpleColumnType,

crates/vespertide-query/src/builder.rs

@@ -3,5 +3,5 @@ source: crates/vespertide-query/src/sql.rs
 expression: "result.iter().map(|q| q.build(backend)).collect::<Vec<String>>().join(\"\\n\")"
 ---
 ALTER TABLE `users` ADD COLUMN `age` int
-UPDATE "users" SET "age" = 0
-ALTER TABLE "users" ALTER COLUMN "age" SET NOT NULL
+UPDATE `users` SET `age` = 0
+ALTER TABLE `users` MODIFY COLUMN `age` NOT NULL

crates/vespertide-query/src/snapshots/vespertide_query__sql__tests__build_migration_action@build_migration_action_add_column_with_backfill_mysql.snap

@@ -4,4 +4,4 @@ expression: "result.iter().map(|q| q.build(backend)).collect::<Vec<String>>().jo
 ---
 ALTER TABLE "users" ADD COLUMN "age" integer
 UPDATE "users" SET "age" = 0
-ALTER TABLE "users" ALTER COLUMN "age" SET NOT NULL
+-- SQLite: ALTER TABLE "users" requires table recreation to set NOT NULL on "age"

crates/vespertide-query/src/snapshots/vespertide_query__sql__tests__build_migration_action@build_migration_action_add_column_with_backfill_sqlite.snap

@@ -2,4 +2,4 @@
 source: crates/vespertide-query/src/sql.rs
 expression: "result.iter().map(|q| q.build(backend)).collect::<Vec<String>>().join(\"\\n\")"
 ---
-ALTER TABLE "users" ADD CONSTRAINT "chk_age" CHECK (age > 0)
+ALTER TABLE `users` ADD CONSTRAINT `chk_age` CHECK (age > 0)

crates/vespertide-query/src/snapshots/vespertide_query__sql__tests__build_migration_action@build_migration_action_add_constraint_check_named_mysql.snap

@@ -2,4 +2,4 @@
 source: crates/vespertide-query/src/sql.rs
 expression: "result.iter().map(|q| q.build(backend)).collect::<Vec<String>>().join(\"\\n\")"
 ---
-ALTER TABLE "posts" ADD CONSTRAINT "fk_user" FOREIGN KEY ("user_id") REFERENCES "users" ("id") ON DELETE CASCADE ON UPDATE RESTRICT
+ALTER TABLE `posts` ADD CONSTRAINT `fk_user` FOREIGN KEY (`user_id`) REFERENCES `users` (`id`) ON DELETE CASCADE ON UPDATE RESTRICT

crates/vespertide-query/src/snapshots/vespertide_query__sql__tests__build_migration_action@build_migration_action_add_constraint_check_unnamed_mysql.snap

@@ -2,4 +2,4 @@
 source: crates/vespertide-query/src/sql.rs
 expression: "result.iter().map(|q| q.build(backend)).collect::<Vec<String>>().join(\"\\n\")"
 ---
-ALTER TABLE "posts" ADD FOREIGN KEY ("user_id") REFERENCES "users" ("id")
+ALTER TABLE `posts` ADD FOREIGN KEY (`user_id`) REFERENCES `users` (`id`)