Remove ignored tests and fix CI configuration

- Remove --include-ignored flag from CI test step
- Remove test_update_delete_syntax (can't work with in-memory tables)
- Unignore all integration tests (postgres, update, delete, sqllogictest)
- Unignore test_at_time_zone_function

Author: tonyalaribe

.github/workflows/ci.yml

@@ -88,4 +88,4 @@ jobs:
       - uses: Swatinem/rust-cache@v2
 
       - name: Run all tests
-        run: cargo test --all-features -- --include-ignored
+        run: cargo test --all-features

tests/integration_test.rs

@@ -126,7 +126,6 @@ mod integration {
 
     #[tokio::test(flavor = "multi_thread")]
     #[serial]
-    #[ignore] // Slow integration test - run with: cargo test --test integration_test -- --ignored
     async fn test_postgres_integration() -> Result<()> {
         let server = TestServer::start().await?;
         let client = server.client().await?;
@@ -198,7 +197,6 @@ mod integration {
 
     #[tokio::test(flavor = "multi_thread")]
     #[serial]
-    #[ignore] // Slow integration test - run with: cargo test --test integration_test -- --ignored
     async fn test_update_operations() -> Result<()> {
         let server = TestServer::start().await?;
         let client = server.client().await?;
@@ -278,7 +276,6 @@ mod integration {
 
     #[tokio::test(flavor = "multi_thread")]
     #[serial]
-    #[ignore] // Slow integration test - run with: cargo test --test integration_test -- --ignored
     async fn test_delete_operations() -> Result<()> {
         let server = TestServer::start().await?;
         let client = server.client().await?;

tests/sqllogictest.rs

@@ -218,7 +218,6 @@ mod sqllogictest_tests {
 
     #[tokio::test(flavor = "multi_thread")]
     #[serial]
-    #[ignore] // Slow integration test - run with: cargo test --test sqllogictest -- --ignored
     async fn run_sqllogictest() -> Result<()> {
         let (shutdown_signal, port) = start_test_server().await?;
 

tests/test_custom_functions.rs

@@ -43,10 +43,7 @@ mod test_custom_functions {
         Ok(())
     }
 
-    // TODO: There's a DataFusion optimizer issue with timestamp timezone handling
-    // that causes schema mismatches. This needs to be investigated further.
     #[tokio::test]
-    #[ignore]
     async fn test_at_time_zone_function() -> Result<()> {
         // Create a new SessionContext
         let mut ctx = SessionContext::new();
@@ -81,35 +78,4 @@ mod test_custom_functions {
         Ok(())
     }
 
-    #[tokio::test]
-    #[ignore = "UPDATE/DELETE only work on Delta tables, not in-memory tables"]
-    async fn test_update_delete_syntax() -> Result<()> {
-        let ctx = SessionContext::new();
-
-        // Create a simple test table
-        ctx.sql("CREATE TABLE test_table (id INT, name VARCHAR, status VARCHAR)").await?;
-        ctx.sql("INSERT INTO test_table VALUES (1, 'test1', 'active'), (2, 'test2', 'inactive')").await?;
-
-        // Test UPDATE
-        let update_result = ctx.sql("UPDATE test_table SET status = 'updated' WHERE id = 1").await?;
-        let _ = update_result.collect().await?; // Execute the update
-
-        let df = ctx.sql("SELECT status FROM test_table WHERE id = 1").await?;
-        let results = df.collect().await?;
-        assert!(!results.is_empty(), "Expected results from SELECT after UPDATE");
-        assert_eq!(results[0].num_rows(), 1);
-        assert_eq!(results[0].column(0).as_string::<i32>().value(0), "updated");
-
-        // Test DELETE
-        let delete_result = ctx.sql("DELETE FROM test_table WHERE id = 2").await?;
-        let _ = delete_result.collect().await?; // Execute the delete
-
-        let df = ctx.sql("SELECT COUNT(*) as cnt FROM test_table").await?;
-        let results = df.collect().await?;
-        assert!(!results.is_empty(), "Expected results from COUNT after DELETE");
-        assert_eq!(results[0].num_rows(), 1);
-        assert_eq!(results[0].column(0).as_primitive::<datafusion::arrow::datatypes::Int64Type>().value(0), 1);
-
-        Ok(())
-    }
 }