fix

Author: sbdchd

crates/squawk_linter/src/analyze.rs

@@ -134,7 +134,7 @@ pub fn possibly_slow_stmt(stmt: &ast::Stmt) -> bool {
         | ast::Stmt::DropType(_)
         | ast::Stmt::DropUser(_)
         | ast::Stmt::DropUserMapping(_)
-        | ast::Stmt::DropView(_) 
+        | ast::Stmt::DropView(_)
         // non-Alter, Create, Drop statements
         | ast::Stmt::Cluster(_)
         | ast::Stmt::CommentOn(_)
@@ -204,12 +204,11 @@ mod tests {
         assert!(possibly_slow_stmt(&stmts));
     }
 
-
     #[test]
     fn select() {
         let sql = "select 1;";
         let file = SourceFile::parse(sql);
         let stmts = file.tree().stmts().next().unwrap();
-        assert_eq!(possibly_slow_stmt(&stmts), false);
+        assert!(!possibly_slow_stmt(&stmts));
     }
 }

crates/squawk_linter/src/lib.rs

@@ -405,7 +405,7 @@ impl Linter {
             ban_truncate_cascade(self, file);
         }
         if self.rules.contains(&Rule::RequireTimeoutSettings) {
-            require_timeout_settings(self, &file);
+            require_timeout_settings(self, file);
         }
         // xtask:new-rule:rule-call
 

crates/squawk_linter/src/rules/mod.rs

@@ -25,8 +25,8 @@ pub(crate) mod renaming_column;
 pub(crate) mod renaming_table;
 pub(crate) mod require_concurrent_index_creation;
 pub(crate) mod require_concurrent_index_deletion;
-pub(crate) mod transaction_nesting;
 pub(crate) mod require_timeout_settings;
+pub(crate) mod transaction_nesting;
 // xtask:new-rule:mod-decl
 
 pub(crate) use adding_field_with_default::adding_field_with_default;
@@ -56,6 +56,6 @@ pub(crate) use renaming_column::renaming_column;
 pub(crate) use renaming_table::renaming_table;
 pub(crate) use require_concurrent_index_creation::require_concurrent_index_creation;
 pub(crate) use require_concurrent_index_deletion::require_concurrent_index_deletion;
-pub(crate) use transaction_nesting::transaction_nesting;
 pub(crate) use require_timeout_settings::require_timeout_settings;
+pub(crate) use transaction_nesting::transaction_nesting;
 // xtask:new-rule:export