add a way to skip some tests on some databases

Author: lovasoa

src/webserver/database/mod.rs

@@ -50,3 +50,9 @@ pub fn highlight_sql_error(
     }
     anyhow::Error::new(db_err).context(msg)
 }
+
+impl std::fmt::Display for Database {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(f, "{:?}", self.connection.any_kind())
+    }
+}

tests/index.rs

@@ -133,6 +133,11 @@ async fn test_files() {
         if test_file_path.extension().unwrap_or_default() != "sql" {
             continue;
         }
+        if test_file_path_string.contains(&format!("no{}", app_data.db.to_string().to_lowercase()))
+        {
+            // skipping because the test does not support the database
+            continue;
+        }
         let req_str = format!("/{}?x=1", test_file_path_string);
         let resp = req_path_with_app_data(&req_str, app_data.clone())
             .await

tests/sql_test_files/README.md

@@ -6,6 +6,10 @@ Files with names starting with `it_works` should all
 return a page that contains the text "It works !" and does not contain the
 text "error" (case insensitive) when executed.
 
+If a file name contains `nosqlite`, `nomssql`, `nopostgres` or `nomysql`, then
+the test will be ignored when running against the corresponding database. 
+This allows using syntax that is not supported on all databases in some tests.
+
 ## `error_` files
 
 Files with names starting with `error` should all return a page that contains

tests/sql_test_files/it_works_temp_table_accessible_in_run_sql.sql renamed to tests/sql_test_files/it_works_temp_table_accessible_in_run_sql_nomssql.sql

@@ -1,3 +1,4 @@
+-- Doesnt work on mssql because it does not support "create temporary table"
 create temporary table temp_t(x text);
 insert into temp_t(x) values ('It works !');
 select 'dynamic' as component, sqlpage.run_sql('tests/select_temp_t.sql') AS properties;