clippy fixes

Author: lovasoa

tests/basic/mod.rs

@@ -1,11 +1,10 @@
 use actix_web::{
     body::MessageBody,
-    http::{self, StatusCode},
+    http::{self},
     test,
 };
-use sqlpage::webserver::http::main_handler;
 
-use crate::common::{get_request_to, req_path};
+use crate::common::req_path;
 
 #[actix_web::test]
 async fn test_index_ok() {
@@ -82,4 +81,4 @@ async fn test_spaces_in_file_names() {
     let body = test::read_body(resp).await;
     let body_str = String::from_utf8(body.to_vec()).unwrap();
     assert!(body_str.contains("It works !"), "{body_str}");
-} 
+}

tests/common/mod.rs

@@ -1,4 +1,4 @@
-use std::{collections::HashMap, path::PathBuf, time::Duration};
+use std::time::Duration;
 
 use actix_web::{
     http::header::ContentType,
@@ -7,9 +7,7 @@ use actix_web::{
 };
 use sqlpage::{
     app_config::{test_database_url, AppConfig},
-    webserver::{
-        http::{form_config, main_handler, payload_config},
-    },
+    webserver::http::{form_config, main_handler, payload_config},
     AppState,
 };
 
@@ -101,4 +99,4 @@ pub fn init_log() {
         .parse_default_env()
         .is_test(true)
         .try_init();
-} 
+}

tests/components/mod.rs

@@ -1,7 +1,4 @@
-use actix_web::{
-    http::StatusCode,
-    test,
-};
+use actix_web::{http::StatusCode, test};
 use sqlpage::webserver::http::main_handler;
 
 use crate::common::get_request_to;
@@ -25,4 +22,4 @@ async fn test_overwrite_variable() -> actix_web::Result<()> {
         "{body_str}\nexpected to contain: It works !"
     );
     Ok(())
-} 
+}

tests/core/mod.rs

@@ -1,11 +1,8 @@
 use actix_web::{
-    http::{self, StatusCode},
+    http::StatusCode,
     test,
 };
-use sqlpage::{
-    webserver,
-    AppState,
-};
+use sqlpage::{webserver, AppState};
 use sqlx::Executor as _;
 
 use crate::common::{make_app_data_from_config, req_path, req_path_with_app_data, test_config};
@@ -53,7 +50,8 @@ async fn test_routing_with_db_fs() {
     config.site_prefix = "/prefix/".to_string();
     let state = AppState::init(&config).await.unwrap();
 
-    let create_table_sql = sqlpage::filesystem::DbFsQueries::get_create_table_sql(state.db.connection.any_kind());
+    let create_table_sql =
+        sqlpage::filesystem::DbFsQueries::get_create_table_sql(state.db.connection.any_kind());
     state
         .db
         .connection
@@ -144,11 +142,9 @@ async fn test_hidden_files() {
     assert_eq!(resp.status(), StatusCode::FORBIDDEN);
     let srv_resp = actix_web::test::TestRequest::default().to_srv_response(resp);
     let body = test::read_body(srv_resp).await;
-    assert!(
-        String::from_utf8_lossy(&body)
-            .to_lowercase()
-            .contains("forbidden"),
-    );
+    assert!(String::from_utf8_lossy(&body)
+        .to_lowercase()
+        .contains("forbidden"),);
 }
 
 #[actix_web::test]
@@ -196,4 +192,4 @@ async fn make_app_data_for_official_website() -> actix_web::web::Data<AppState>
         .await
         .unwrap();
     app_state
-} 
+}

tests/data_formats/mod.rs

@@ -79,4 +79,4 @@ async fn test_json_columns() {
         !body_html_escaped.contains("{"),
         "the json should have been parsed, not returned as a string, in: {body_html_escaped}"
     );
-} 
+}

tests/errors/mod.rs

@@ -1,8 +1,4 @@
-use actix_web::{
-    http::StatusCode,
-    test,
-};
-use sqlpage::webserver::http::main_handler;
+use actix_web::{http::StatusCode, test};
 
 use crate::common::req_path;
 
@@ -17,9 +13,7 @@ async fn test_privileged_paths_are_not_accessible() {
     assert_eq!(resp.status(), StatusCode::FORBIDDEN);
     let srv_resp = actix_web::test::TestRequest::default().to_srv_response(resp);
     let body = test::read_body(srv_resp).await;
-    assert!(
-        String::from_utf8_lossy(&body)
-            .to_lowercase()
-            .contains("forbidden"),
-    );
-} 
+    assert!(String::from_utf8_lossy(&body)
+        .to_lowercase()
+        .contains("forbidden"),);
+}

tests/mod.rs

@@ -7,13 +7,3 @@ mod errors;
 mod requests;
 mod transactions;
 mod uploads;
-
-pub use basic::*;
-pub use common::*;
-pub use components::*;
-pub use core::*;
-pub use data_formats::*;
-pub use errors::*;
-pub use requests::*;
-pub use transactions::*;
-pub use uploads::*; 

tests/requests/mod.rs

@@ -1,7 +1,4 @@
-use actix_web::{
-    http::StatusCode,
-    test,
-};
+use actix_web::{http::StatusCode, test};
 use sqlpage::webserver::http::main_handler;
 
 use crate::common::get_request_to;
@@ -101,4 +98,4 @@ async fn test_large_form_field_roundtrip() -> actix_web::Result<()> {
         "{body_str}\nexpected to contain long string submitted"
     );
     Ok(())
-} 
+}

tests/transactions/mod.rs

@@ -1,7 +1,4 @@
-use actix_web::{
-    http::StatusCode,
-    test,
-};
+use actix_web::{http::StatusCode, test};
 use sqlpage::webserver::http::main_handler;
 
 use crate::common::{get_request_to_with_data, make_app_data};
@@ -97,4 +94,4 @@ async fn test_failed_copy_followed_by_query() -> actix_web::Result<()> {
         );
     }
     Ok(())
-} 
+}

tests/uploads/mod.rs

@@ -1,7 +1,4 @@
-use actix_web::{
-    http::StatusCode,
-    test,
-};
+use actix_web::{http::StatusCode, test};
 use sqlpage::webserver::http::main_handler;
 
 use crate::common::get_request_to;
@@ -165,4 +162,4 @@ async fn test_csv_upload() -> actix_web::Result<()> {
         "{body_str}\nexpected to contain: Ophir is 29 years old"
     );
     Ok(())
-} 
+}