Rm comment

Author: owjs3901

crates/vespertide-query/src/sql.rs

@@ -1506,17 +1506,16 @@ mod tests {
         DatabaseBackend::MySql,
         &["ALTER TABLE `users` MODIFY COLUMN `age` varchar(50)"]
     )]
-    // NOTE: SQLite does not support MODIFY COLUMN; see sea-query sqlite backend.
-    // #[case::modify_column_type_sqlite(
-    //     "modify_column_type_sqlite",
-    //     MigrationAction::ModifyColumnType {
-    //         table: "users".into(),
-    //         column: "age".into(),
-    //         new_type: ColumnType::Complex(ComplexColumnType::Varchar { length: 50 }),
-    //     },
-    //     DatabaseBackend::Sqlite,
-    //     &["ALTER TABLE \"users\" MODIFY COLUMN \"age\" VARCHAR(50)"]
-    // )]
+    #[case::modify_column_type_sqlite(
+        "modify_column_type_sqlite",
+        MigrationAction::ModifyColumnType {
+            table: "users".into(),
+            column: "age".into(),
+            new_type: ColumnType::Complex(ComplexColumnType::Varchar { length: 50 }),
+        },
+        DatabaseBackend::Sqlite,
+        &[]
+    )]
     #[case::rename_table_action_postgres(
         "rename_table_action_postgres",
         MigrationAction::RenameTable {
@@ -2287,14 +2286,13 @@ mod tests {
             .to_tbl(Alias::new("b"))
             .to_col(Alias::new("id"))
             .to_owned())),DatabaseBackend::MySql, &["ALTER TABLE `a` ADD CONSTRAINT `fk` FOREIGN KEY (`c`) REFERENCES `b` (`id`)"])]
-    // NOTE: SQLite does not support modifying FK constraints on existing tables; sea-query panics for these.
-    // #[case::create_foreign_key_sqlite("create_foreign_key_sqlite", BuiltQuery::CreateForeignKey(Box::new(ForeignKey::create()
-    //         .name("fk")
-    //         .from_tbl(Alias::new("a"))
-    //         .from_col(Alias::new("c"))
-    //         .to_tbl(Alias::new("b"))
-    //         .to_col(Alias::new("id"))
-    //         .to_owned())),DatabaseBackend::Sqlite, &["ALTER TABLE \"a\" ADD CONSTRAINT \"fk\" FOREIGN KEY (\"c\") REFERENCES \"b\" (\"id\")"])]
+    #[case::create_foreign_key_sqlite("create_foreign_key_sqlite", BuiltQuery::CreateForeignKey(Box::new(ForeignKey::create()
+            .name("fk")
+            .from_tbl(Alias::new("a"))
+            .from_col(Alias::new("c"))
+            .to_tbl(Alias::new("b"))
+            .to_col(Alias::new("id"))
+            .to_owned())),DatabaseBackend::Sqlite, &[])]
     #[case::drop_foreign_key_postgres("drop_foreign_key_postgres", BuiltQuery::DropForeignKey(Box::new(ForeignKey::drop()
             .name("fk")
             .table(Alias::new("a"))
@@ -2303,10 +2301,10 @@ mod tests {
             .name("fk")
             .table(Alias::new("a"))
             .to_owned())),DatabaseBackend::MySql, &["ALTER TABLE `a` DROP FOREIGN KEY `fk`"])]
-    // #[case::drop_foreign_key_sqlite("drop_foreign_key_sqlite", BuiltQuery::DropForeignKey(Box::new(ForeignKey::drop()
-    //         .name("fk")
-    //         .table(Alias::new("a"))
-    //         .to_owned())),DatabaseBackend::Sqlite, &["ALTER TABLE \"a\" DROP CONSTRAINT \"fk\""])]
+    #[case::drop_foreign_key_sqlite("drop_foreign_key_sqlite", BuiltQuery::DropForeignKey(Box::new(ForeignKey::drop()
+            .name("fk")
+            .table(Alias::new("a"))
+            .to_owned())),DatabaseBackend::Sqlite, &[])]
     fn test_build_query(
         #[case] title: &str,
         #[case] q: BuiltQuery,