Merge pull request #178 from snshn/tests-code-refactor

Group all tests into either passing or failing groups

src/tests/cli.rs

@@ -1,9 +1,3 @@
-use assert_cmd::prelude::*;
-use std::env;
-use std::io::Write;
-use std::process::Command;
-use tempfile::NamedTempFile;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -11,8 +5,16 @@ use tempfile::NamedTempFile;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use assert_cmd::prelude::*;
+    use std::env;
+    use std::io::Write;
+    use std::process::Command;
+    use tempfile::NamedTempFile;
+
     #[test]
-fn passing_print_version() -> Result<(), Box<dyn std::error::Error>> {
+    fn print_version() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd.arg("-V").output().unwrap();
 
@@ -32,7 +34,7 @@ fn passing_print_version() -> Result<(), Box<dyn std::error::Error>> {
     }
 
     #[test]
-fn passing_bad_input_empty_target() -> Result<(), Box<dyn std::error::Error>> {
+    fn bad_input_empty_target() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd.arg("").output().unwrap();
 
@@ -52,7 +54,7 @@ fn passing_bad_input_empty_target() -> Result<(), Box<dyn std::error::Error>> {
     }
 
     #[test]
-fn passing_bad_input_data_url() -> Result<(), Box<dyn std::error::Error>> {
+    fn bad_input_data_url() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd.arg("data:,Hello%2C%20World!").output().unwrap();
 
@@ -72,7 +74,7 @@ fn passing_bad_input_data_url() -> Result<(), Box<dyn std::error::Error>> {
     }
 
     #[test]
-fn passing_isolate_data_url() -> Result<(), Box<dyn std::error::Error>> {
+    fn isolate_data_url() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd
             .arg("-M")
@@ -99,7 +101,7 @@ fn passing_isolate_data_url() -> Result<(), Box<dyn std::error::Error>> {
     }
 
     #[test]
-fn passing_remove_css_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
+    fn remove_css_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd
             .arg("-M")
@@ -127,7 +129,7 @@ fn passing_remove_css_from_data_url() -> Result<(), Box<dyn std::error::Error>>
     }
 
     #[test]
-fn passing_remove_frames_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
+    fn remove_frames_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd
             .arg("-M")
@@ -154,7 +156,7 @@ fn passing_remove_frames_from_data_url() -> Result<(), Box<dyn std::error::Error
     }
 
     #[test]
-fn passing_remove_images_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
+    fn remove_images_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd
             .arg("-M")
@@ -190,7 +192,7 @@ Hi\
     }
 
     #[test]
-fn passing_remove_js_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
+    fn remove_js_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd
             .arg("-M")
@@ -220,7 +222,7 @@ fn passing_remove_js_from_data_url() -> Result<(), Box<dyn std::error::Error>> {
     }
 
     #[test]
-fn passing_local_file_target_input() -> Result<(), Box<dyn std::error::Error>> {
+    fn local_file_target_input() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let cwd_normalized: String =
             str!(env::current_dir().unwrap().to_str().unwrap()).replace("\\", "/");
@@ -273,8 +275,7 @@ fn passing_local_file_target_input() -> Result<(), Box<dyn std::error::Error>> {
     }
 
     #[test]
-fn passing_local_file_target_input_absolute_target_path() -> Result<(), Box<dyn std::error::Error>>
-{
+    fn local_file_target_input_absolute_target_path() -> Result<(), Box<dyn std::error::Error>> {
         let cwd = env::current_dir().unwrap();
         let cwd_normalized: String =
             str!(env::current_dir().unwrap().to_str().unwrap()).replace("\\", "/");
@@ -335,7 +336,7 @@ fn passing_local_file_target_input_absolute_target_path() -> Result<(), Box<dyn
     }
 
     #[test]
-fn passing_local_file_url_target_input() -> Result<(), Box<dyn std::error::Error>> {
+    fn local_file_url_target_input() -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let cwd_normalized: String =
             str!(env::current_dir().unwrap().to_str().unwrap()).replace("\\", "/");
@@ -405,7 +406,7 @@ fn passing_local_file_url_target_input() -> Result<(), Box<dyn std::error::Error
     }
 
     #[test]
-fn passing_security_disallow_local_assets_within_data_url_targets(
+    fn security_disallow_local_assets_within_data_url_targets(
     ) -> Result<(), Box<dyn std::error::Error>> {
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let out = cmd
@@ -430,8 +431,8 @@ fn passing_security_disallow_local_assets_within_data_url_targets(
     }
 
     #[test]
-fn passing_embed_file_url_local_asset_within_style_attribute(
-) -> Result<(), Box<dyn std::error::Error>> {
+    fn embed_file_url_local_asset_within_style_attribute() -> Result<(), Box<dyn std::error::Error>>
+    {
         let file_url_prefix: &str = if cfg!(windows) { "file:///" } else { "file://" };
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let mut file_svg = NamedTempFile::new()?;
@@ -476,7 +477,7 @@ fn passing_embed_file_url_local_asset_within_style_attribute(
     }
 
     #[test]
-fn passing_css_import_string() -> Result<(), Box<dyn std::error::Error>> {
+    fn css_import_string() -> Result<(), Box<dyn std::error::Error>> {
         let file_url_prefix: &str = if cfg!(windows) { "file:///" } else { "file://" };
         let mut cmd = Command::cargo_bin(env!("CARGO_PKG_NAME"))?;
         let mut file_css = NamedTempFile::new()?;
@@ -527,3 +528,4 @@ fn passing_css_import_string() -> Result<(), Box<dyn std::error::Error>> {
 
         Ok(())
     }
+}

src/tests/css/embed_css.rs

@@ -1,8 +1,3 @@
-use reqwest::blocking::Client;
-use std::collections::HashMap;
-
-use crate::css;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -10,8 +5,14 @@ use crate::css;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::css;
+    use reqwest::blocking::Client;
+    use std::collections::HashMap;
+
     #[test]
-fn passing_empty_input() {
+    fn empty_input() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -22,7 +23,7 @@ fn passing_empty_input() {
     }
 
     #[test]
-fn passing_style_exclude_unquoted_images() {
+    fn style_exclude_unquoted_images() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -58,7 +59,7 @@ height: calc(100vh - 10pt)",
     }
 
     #[test]
-fn passing_style_exclude_single_quoted_images() {
+    fn style_exclude_single_quoted_images() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -86,7 +87,7 @@ height: calc(100vh - 10pt)",
     }
 
     #[test]
-fn passing_style_block() {
+    fn style_block() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -105,7 +106,7 @@ html > body {}";
     }
 
     #[test]
-fn passing_attribute_selectors() {
+    fn attribute_selectors() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -146,7 +147,7 @@ fn passing_attribute_selectors() {
     }
 
     #[test]
-fn passing_import_string() {
+    fn import_string() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -179,7 +180,7 @@ fn passing_import_string() {
     }
 
     #[test]
-fn passing_hash_urls() {
+    fn hash_urls() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -208,7 +209,7 @@ body {\n    \
     }
 
     #[test]
-fn passing_transform_percentages_and_degrees() {
+    fn transform_percentages_and_degrees() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -235,7 +236,7 @@ div {\n    \
     }
 
     #[test]
-fn passing_unusual_indents() {
+    fn unusual_indents() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -264,7 +265,7 @@ fn passing_unusual_indents() {
     }
 
     #[test]
-fn passing_exclude_fonts() {
+    fn exclude_fonts() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -315,3 +316,4 @@ div {\n    \
             CSS_OUT
         );
     }
+}

src/tests/css/enquote.rs

@@ -1,5 +1,3 @@
-use crate::css;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,18 +5,22 @@ use crate::css;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::css;
+
     #[test]
-fn passing_empty_input_single_quotes() {
+    fn empty_input_single_quotes() {
         assert_eq!(css::enquote(str!(""), false), "''");
     }
 
     #[test]
-fn passing_empty_input_double_quotes() {
+    fn empty_input_double_quotes() {
         assert_eq!(css::enquote(str!(""), true), "\"\"");
     }
 
     #[test]
-fn passing_apostrophes_single_quotes() {
+    fn apostrophes_single_quotes() {
         assert_eq!(
             css::enquote(str!("It's a lovely day, don't you think?"), false),
             "'It\\'s a lovely day, don\\'t you think?'"
@@ -26,7 +28,7 @@ fn passing_apostrophes_single_quotes() {
     }
 
     #[test]
-fn passing_apostrophes_double_quotes() {
+    fn apostrophes_double_quotes() {
         assert_eq!(
             css::enquote(str!("It's a lovely day, don't you think?"), true),
             "\"It's a lovely day, don't you think?\""
@@ -34,7 +36,7 @@ fn passing_apostrophes_double_quotes() {
     }
 
     #[test]
-fn passing_feet_and_inches_single_quotes() {
+    fn feet_and_inches_single_quotes() {
         assert_eq!(
             css::enquote(str!("5'2\", 6'5\""), false),
             "'5\\'2\", 6\\'5\"'"
@@ -42,9 +44,10 @@ fn passing_feet_and_inches_single_quotes() {
     }
 
     #[test]
-fn passing_feet_and_inches_double_quotes() {
+    fn feet_and_inches_double_quotes() {
         assert_eq!(
             css::enquote(str!("5'2\", 6'5\""), true),
             "\"5'2\\\", 6'5\\\"\""
         );
     }
+}

src/tests/html/get_node_name.rs

@@ -1,6 +1,3 @@
-use crate::html;
-use html5ever::rcdom::{Handle, NodeData};
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -8,6 +5,11 @@ use html5ever::rcdom::{Handle, NodeData};
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::html;
+    use html5ever::rcdom::{Handle, NodeData};
+
     #[test]
     fn get_node_name() {
         let html = "<!doctype html><html><HEAD></HEAD><body><div><P></P></div></body></html>";
@@ -47,3 +49,4 @@ fn get_node_name() {
 
         assert_eq!(count, 7);
     }
+}

src/tests/html/is_icon.rs

@@ -1,5 +1,3 @@
-use crate::html;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,30 +5,35 @@ use crate::html;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::html;
+
     #[test]
-fn passing_icon() {
+    fn icon() {
         assert!(html::is_icon("icon"));
     }
 
     #[test]
-fn passing_shortcut_icon_capitalized() {
+    fn shortcut_icon_capitalized() {
         assert!(html::is_icon("Shortcut Icon"));
     }
 
     #[test]
-fn passing_icon_uppercase() {
+    fn icon_uppercase() {
         assert!(html::is_icon("ICON"));
     }
 
     #[test]
-fn passing_mask_icon() {
+    fn mask_icon() {
         assert!(html::is_icon("mask-icon"));
     }
 
     #[test]
-fn passing_fluid_icon() {
+    fn fluid_icon() {
         assert!(html::is_icon("fluid-icon"));
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -39,12 +42,17 @@ fn passing_fluid_icon() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::html;
+
     #[test]
-fn failing_stylesheet() {
+    fn stylesheet() {
         assert!(!html::is_icon("stylesheet"));
     }
 
     #[test]
-fn failing_empty_string() {
+    fn empty_string() {
         assert!(!html::is_icon(""));
     }
+}

src/tests/html/stringify_document.rs

@@ -1,5 +1,3 @@
-use crate::html;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,8 +5,12 @@ use crate::html;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::html;
+
     #[test]
-fn passing_div_as_root_element() {
+    fn div_as_root_element() {
         let html = "<div><script src=\"some.js\"></script></div>";
         let dom = html::html_to_dom(&html);
 
@@ -32,7 +34,7 @@ fn passing_div_as_root_element() {
     }
 
     #[test]
-fn passing_full_page_with_no_html_head_or_body() {
+    fn full_page_with_no_html_head_or_body() {
         let html = "<title>Isolated document</title>\
                     <link rel=\"something\" href=\"some.css\" />\
                     <meta http-equiv=\"Content-Security-Policy\" content=\"default-src https:\">\
@@ -71,7 +73,7 @@ fn passing_full_page_with_no_html_head_or_body() {
     }
 
     #[test]
-fn passing_doctype_and_the_rest_no_html_head_or_body() {
+    fn doctype_and_the_rest_no_html_head_or_body() {
         let html = "<!doctype html>\
                     <title>Unstyled document</title>\
                     <link rel=\"stylesheet\" href=\"main.css\"/>\
@@ -106,7 +108,7 @@ fn passing_doctype_and_the_rest_no_html_head_or_body() {
     }
 
     #[test]
-fn passing_doctype_and_the_rest_no_html_head_or_body_forbid_frames() {
+    fn doctype_and_the_rest_no_html_head_or_body_forbid_frames() {
         let html = "<!doctype html>\
                     <title>Frameless document</title>\
                     <link rel=\"something\"/>\
@@ -141,7 +143,7 @@ fn passing_doctype_and_the_rest_no_html_head_or_body_forbid_frames() {
     }
 
     #[test]
-fn passing_doctype_and_the_rest_all_forbidden() {
+    fn doctype_and_the_rest_all_forbidden() {
         let html = "<!doctype html>\
                     <title>no-frame no-css no-js no-image isolated document</title>\
                     <meta http-equiv=\"Content-Security-Policy\" content=\"default-src https:\">\
@@ -186,3 +188,4 @@ fn passing_doctype_and_the_rest_all_forbidden() {
                 </html>"
         );
     }
+}

src/tests/html/walk_and_embed_assets.rs

@@ -1,8 +1,3 @@
-use crate::html;
-use html5ever::serialize::{serialize, SerializeOpts};
-use reqwest::blocking::Client;
-use std::collections::HashMap;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -10,8 +5,15 @@ use std::collections::HashMap;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::html;
+    use html5ever::serialize::{serialize, SerializeOpts};
+    use reqwest::blocking::Client;
+    use std::collections::HashMap;
+
     #[test]
-fn passing_basic() {
+    fn basic() {
         let cache = &mut HashMap::new();
 
         let html = "<div><P></P></div>";
@@ -50,7 +52,7 @@ fn passing_basic() {
     }
 
     #[test]
-fn passing_ensure_no_recursive_iframe() {
+    fn ensure_no_recursive_iframe() {
         let html = "<div><P></P><iframe src=\"\"></iframe></div>";
         let dom = html::html_to_dom(&html);
         let url = "http://localhost";
@@ -88,7 +90,7 @@ fn passing_ensure_no_recursive_iframe() {
     }
 
     #[test]
-fn passing_ensure_no_recursive_frame() {
+    fn ensure_no_recursive_frame() {
         let html = "<frameset><frame src=\"\"></frameset>";
         let dom = html::html_to_dom(&html);
         let url = "http://localhost";
@@ -126,7 +128,7 @@ fn passing_ensure_no_recursive_frame() {
     }
 
     #[test]
-fn passing_no_css() {
+    fn no_css() {
         let html = "<link rel=\"stylesheet\" href=\"main.css\">\
                     <style>html{background-color: #000;}</style>\
                     <div style=\"display: none;\"></div>";
@@ -173,7 +175,7 @@ fn passing_no_css() {
     }
 
     #[test]
-fn passing_no_images() {
+    fn no_images() {
         let html = "<link rel=\"icon\" href=\"favicon.ico\">\
                     <div><img src=\"http://localhost/assets/mono_lisa.png\" /></div>";
         let dom = html::html_to_dom(&html);
@@ -224,8 +226,9 @@ fn passing_no_images() {
     }
 
     #[test]
-fn passing_no_body_background_images() {
-    let html = "<body background=\"no/such/image.png\" background=\"no/such/image2.png\"></body>";
+    fn no_body_background_images() {
+        let html =
+            "<body background=\"no/such/image.png\" background=\"no/such/image2.png\"></body>";
         let dom = html::html_to_dom(&html);
         let url = "http://localhost";
         let cache = &mut HashMap::new();
@@ -262,7 +265,7 @@ fn passing_no_body_background_images() {
     }
 
     #[test]
-fn passing_no_frames() {
+    fn no_frames() {
         let html = "<frameset><frame src=\"http://trackbook.com\"></frameset>";
         let dom = html::html_to_dom(&html);
         let url = "http://localhost";
@@ -299,7 +302,7 @@ fn passing_no_frames() {
     }
 
     #[test]
-fn passing_no_iframes() {
+    fn no_iframes() {
         let html = "<iframe src=\"http://trackbook.com\"></iframe>";
         let dom = html::html_to_dom(&html);
         let url = "http://localhost";
@@ -336,7 +339,7 @@ fn passing_no_iframes() {
     }
 
     #[test]
-fn passing_no_js() {
+    fn no_js() {
         let html = "<div onClick=\"void(0)\">\
                     <script src=\"http://localhost/assets/some.js\"></script>\
                     <script>alert(1)</script>\
@@ -378,7 +381,7 @@ fn passing_no_js() {
     }
 
     #[test]
-fn passing_with_no_integrity() {
+    fn with_no_integrity() {
         let html = "<title>No integrity</title>\
                     <link integrity=\"sha384-...\" rel=\"something\"/>\
                     <script integrity=\"sha384-...\" src=\"some.js\"></script>";
@@ -417,3 +420,4 @@ fn passing_with_no_integrity() {
             </html>"
         );
     }
+}

src/tests/js/attr_is_event_handler.rs

@@ -1,5 +1,3 @@
-use crate::js;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,20 +5,25 @@ use crate::js;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::js;
+
     #[test]
-fn passing_onblur_camelcase() {
+    fn onblur_camelcase() {
         assert!(js::attr_is_event_handler("onBlur"));
     }
 
     #[test]
-fn passing_onclick_lowercase() {
+    fn onclick_lowercase() {
         assert!(js::attr_is_event_handler("onclick"));
     }
 
     #[test]
-fn passing_onclick_camelcase() {
+    fn onclick_camelcase() {
         assert!(js::attr_is_event_handler("onClick"));
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -29,17 +32,22 @@ fn passing_onclick_camelcase() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::js;
+
     #[test]
-fn failing_href() {
+    fn href() {
         assert!(!js::attr_is_event_handler("href"));
     }
 
     #[test]
-fn failing_empty_string() {
+    fn empty_string() {
         assert!(!js::attr_is_event_handler(""));
     }
 
     #[test]
-fn failing_class() {
+    fn class() {
         assert!(!js::attr_is_event_handler("class"));
     }
+}

src/tests/utils/clean_url.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,8 +5,12 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_removes_fragment() {
+    fn removes_fragment() {
         assert_eq!(
             utils::clean_url("https://somewhere.com/font.eot#iefix"),
             "https://somewhere.com/font.eot"
@@ -16,7 +18,7 @@ fn passing_removes_fragment() {
     }
 
     #[test]
-fn passing_removes_empty_fragment() {
+    fn removes_empty_fragment() {
         assert_eq!(
             utils::clean_url("https://somewhere.com/font.eot#"),
             "https://somewhere.com/font.eot"
@@ -24,7 +26,7 @@ fn passing_removes_empty_fragment() {
     }
 
     #[test]
-fn passing_removes_empty_query_and_empty_fragment() {
+    fn removes_empty_query_and_empty_fragment() {
         assert_eq!(
             utils::clean_url("https://somewhere.com/font.eot?#"),
             "https://somewhere.com/font.eot"
@@ -32,7 +34,7 @@ fn passing_removes_empty_query_and_empty_fragment() {
     }
 
     #[test]
-fn passing_removes_empty_query_amp_and_empty_fragment() {
+    fn removes_empty_query_amp_and_empty_fragment() {
         assert_eq!(
             utils::clean_url("https://somewhere.com/font.eot?a=b&#"),
             "https://somewhere.com/font.eot?a=b"
@@ -40,9 +42,10 @@ fn passing_removes_empty_query_amp_and_empty_fragment() {
     }
 
     #[test]
-fn passing_keeps_credentials() {
+    fn keeps_credentials() {
         assert_eq!(
             utils::clean_url("https://cookie:monster@gibson.internet/"),
             "https://cookie:monster@gibson.internet/"
         );
     }
+}

src/tests/utils/data_to_data_url.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,8 +5,12 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_encode_string_with_specific_media_type() {
+    fn encode_string_with_specific_media_type() {
         let mime = "application/javascript";
         let data = "var word = 'hello';\nalert(word);\n";
         let data_url = utils::data_to_data_url(mime, data.as_bytes(), "", "");
@@ -20,9 +22,10 @@ fn passing_encode_string_with_specific_media_type() {
     }
 
     #[test]
-fn passing_encode_append_fragment() {
+    fn encode_append_fragment() {
         let data = "<svg></svg>\n";
         let data_url = utils::data_to_data_url("text/css", data.as_bytes(), "", "fragment");
 
         assert_eq!(&data_url, "data:text/css;base64,PHN2Zz48L3N2Zz4K#fragment");
     }
+}

src/tests/utils/data_url_to_data.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,8 +5,12 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_parse_text_html_base64() {
+    fn parse_text_html_base64() {
         let (media_type, data) = utils::data_url_to_data("data:text/html;base64,V29yayBleHBhbmRzIHNvIGFzIHRvIGZpbGwgdGhlIHRpbWUgYXZhaWxhYmxlIGZvciBpdHMgY29tcGxldGlvbg==");
 
         assert_eq!(media_type, "text/html");
@@ -19,7 +21,7 @@ fn passing_parse_text_html_base64() {
     }
 
     #[test]
-fn passing_parse_text_html_utf8() {
+    fn parse_text_html_utf8() {
         let (media_type, data) = utils::data_url_to_data(
             "data:text/html;utf8,Work expands so as to fill the time available for its completion",
         );
@@ -32,7 +34,7 @@ fn passing_parse_text_html_utf8() {
     }
 
     #[test]
-fn passing_parse_text_html_plaintext() {
+    fn parse_text_html_plaintext() {
         let (media_type, data) = utils::data_url_to_data(
             "data:text/html,Work expands so as to fill the time available for its completion",
         );
@@ -45,7 +47,7 @@ fn passing_parse_text_html_plaintext() {
     }
 
     #[test]
-fn passing_parse_text_html_charset_utf_8_between_two_whitespaces() {
+    fn parse_text_html_charset_utf_8_between_two_whitespaces() {
         let (media_type, data) = utils::data_url_to_data(" data:text/html;charset=utf-8,Work expands so as to fill the time available for its completion ");
 
         assert_eq!(media_type, "text/html");
@@ -56,15 +58,16 @@ fn passing_parse_text_html_charset_utf_8_between_two_whitespaces() {
     }
 
     #[test]
-fn passing_parse_text_css_url_encoded() {
-    let (media_type, data) = utils::data_url_to_data("data:text/css,div{background-color:%23000}");
+    fn parse_text_css_url_encoded() {
+        let (media_type, data) =
+            utils::data_url_to_data("data:text/css,div{background-color:%23000}");
 
         assert_eq!(media_type, "text/css");
         assert_eq!(String::from_utf8_lossy(&data), "div{background-color:#000}");
     }
 
     #[test]
-fn passing_parse_no_media_type_base64() {
+    fn parse_no_media_type_base64() {
         let (media_type, data) = utils::data_url_to_data("data:;base64,dGVzdA==");
 
         assert_eq!(media_type, "");
@@ -72,12 +75,13 @@ fn passing_parse_no_media_type_base64() {
     }
 
     #[test]
-fn passing_parse_no_media_type_no_encoding() {
+    fn parse_no_media_type_no_encoding() {
         let (media_type, data) = utils::data_url_to_data("data:;,test%20test");
 
         assert_eq!(media_type, "");
         assert_eq!(String::from_utf8_lossy(&data), "test test");
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -86,10 +90,15 @@ fn passing_parse_no_media_type_no_encoding() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+
     #[test]
-fn failing_just_word_data() {
+    fn just_word_data() {
         let (media_type, data) = utils::data_url_to_data("data");
 
         assert_eq!(media_type, "");
         assert_eq!(String::from_utf8_lossy(&data), "");
     }
+}

src/tests/utils/detect_media_type.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,23 +5,27 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_image_gif87() {
+    fn image_gif87() {
         assert_eq!(utils::detect_media_type(b"GIF87a", ""), "image/gif");
     }
 
     #[test]
-fn passing_image_gif89() {
+    fn image_gif89() {
         assert_eq!(utils::detect_media_type(b"GIF89a", ""), "image/gif");
     }
 
     #[test]
-fn passing_image_jpeg() {
+    fn image_jpeg() {
         assert_eq!(utils::detect_media_type(b"\xFF\xD8\xFF", ""), "image/jpeg");
     }
 
     #[test]
-fn passing_image_png() {
+    fn image_png() {
         assert_eq!(
             utils::detect_media_type(b"\x89PNG\x0D\x0A\x1A\x0A", ""),
             "image/png"
@@ -31,12 +33,12 @@ fn passing_image_png() {
     }
 
     #[test]
-fn passing_image_svg() {
+    fn image_svg() {
         assert_eq!(utils::detect_media_type(b"<svg ", ""), "image/svg+xml");
     }
 
     #[test]
-fn passing_image_webp() {
+    fn image_webp() {
         assert_eq!(
             utils::detect_media_type(b"RIFF....WEBPVP8 ", ""),
             "image/webp"
@@ -44,7 +46,7 @@ fn passing_image_webp() {
     }
 
     #[test]
-fn passing_image_icon() {
+    fn image_icon() {
         assert_eq!(
             utils::detect_media_type(b"\x00\x00\x01\x00", ""),
             "image/x-icon"
@@ -52,7 +54,7 @@ fn passing_image_icon() {
     }
 
     #[test]
-fn passing_image_svg_filename() {
+    fn image_svg_filename() {
         assert_eq!(
             utils::detect_media_type(b"<?xml ", "local-file.svg"),
             "image/svg+xml"
@@ -60,7 +62,7 @@ fn passing_image_svg_filename() {
     }
 
     #[test]
-fn passing_image_svg_url_uppercase() {
+    fn image_svg_url_uppercase() {
         assert_eq!(
             utils::detect_media_type(b"", "https://some-site.com/images/local-file.SVG"),
             "image/svg+xml"
@@ -68,27 +70,27 @@ fn passing_image_svg_url_uppercase() {
     }
 
     #[test]
-fn passing_audio_mpeg() {
+    fn audio_mpeg() {
         assert_eq!(utils::detect_media_type(b"ID3", ""), "audio/mpeg");
     }
 
     #[test]
-fn passing_audio_mpeg_2() {
+    fn audio_mpeg_2() {
         assert_eq!(utils::detect_media_type(b"\xFF\x0E", ""), "audio/mpeg");
     }
 
     #[test]
-fn passing_audio_mpeg_3() {
+    fn audio_mpeg_3() {
         assert_eq!(utils::detect_media_type(b"\xFF\x0F", ""), "audio/mpeg");
     }
 
     #[test]
-fn passing_audio_ogg() {
+    fn audio_ogg() {
         assert_eq!(utils::detect_media_type(b"OggS", ""), "audio/ogg");
     }
 
     #[test]
-fn passing_audio_wav() {
+    fn audio_wav() {
         assert_eq!(
             utils::detect_media_type(b"RIFF....WAVEfmt ", ""),
             "audio/wav"
@@ -96,12 +98,12 @@ fn passing_audio_wav() {
     }
 
     #[test]
-fn passing_audio_flac() {
+    fn audio_flac() {
         assert_eq!(utils::detect_media_type(b"fLaC", ""), "audio/x-flac");
     }
 
     #[test]
-fn passing_video_avi() {
+    fn video_avi() {
         assert_eq!(
             utils::detect_media_type(b"RIFF....AVI LIST", ""),
             "video/avi"
@@ -109,12 +111,12 @@ fn passing_video_avi() {
     }
 
     #[test]
-fn passing_video_mp4() {
+    fn video_mp4() {
         assert_eq!(utils::detect_media_type(b"....ftyp", ""), "video/mp4");
     }
 
     #[test]
-fn passing_video_mpeg() {
+    fn video_mpeg() {
         assert_eq!(
             utils::detect_media_type(b"\x00\x00\x01\x0B", ""),
             "video/mpeg"
@@ -122,17 +124,18 @@ fn passing_video_mpeg() {
     }
 
     #[test]
-fn passing_video_quicktime() {
+    fn video_quicktime() {
         assert_eq!(utils::detect_media_type(b"....moov", ""), "video/quicktime");
     }
 
     #[test]
-fn passing_video_webm() {
+    fn video_webm() {
         assert_eq!(
             utils::detect_media_type(b"\x1A\x45\xDF\xA3", ""),
             "video/webm"
         );
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -141,7 +144,12 @@ fn passing_video_webm() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+
     #[test]
-fn failing_unknown_media_type() {
+    fn unknown_media_type() {
         assert_eq!(utils::detect_media_type(b"abcdef0123456789", ""), "");
     }
+}

src/tests/utils/file_url_to_fs_path.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,8 +5,12 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_remove_protocl_and_fragment() {
+    fn remove_protocl_and_fragment() {
         if cfg!(windows) {
             assert_eq!(
                 utils::file_url_to_fs_path("file:///C:/documents/some-path/some-file.svg#fragment"),
@@ -23,7 +25,7 @@ fn passing_remove_protocl_and_fragment() {
     }
 
     #[test]
-fn passing_decodes_urls() {
+    fn decodes_urls() {
         if cfg!(windows) {
             assert_eq!(
                 utils::file_url_to_fs_path("file:///C:/Documents%20and%20Settings/some-file.html"),
@@ -36,3 +38,4 @@ fn passing_decodes_urls() {
             );
         }
     }
+}

src/tests/utils/get_url_fragment.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,8 +5,12 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_data_url() {
+    fn data_url() {
         assert_eq!(
             utils::get_url_fragment(
                 "data:image/svg+xml;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h#test"
@@ -18,6 +20,7 @@ fn passing_data_url() {
     }
 
     #[test]
-fn passing_https_empty() {
+    fn https_empty() {
         assert_eq!(utils::get_url_fragment("https://kernel.org#"), "");
     }
+}

src/tests/utils/is_data_url.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,19 +5,24 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_data_url_text_html() {
+    fn data_url_text_html() {
         assert!(utils::is_data_url(
             "data:text/html;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h"
         ));
     }
 
     #[test]
-fn passing_data_url_no_media_type() {
+    fn data_url_no_media_type() {
         assert!(utils::is_data_url(
             "data:;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h"
         ));
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -28,17 +31,22 @@ fn passing_data_url_no_media_type() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+
     #[test]
-fn failing_https_url() {
+    fn https_url() {
         assert!(!utils::is_data_url("https://kernel.org"));
     }
 
     #[test]
-fn failing_no_protocol_url() {
+    fn no_protocol_url() {
         assert!(!utils::is_data_url("//kernel.org"));
     }
 
     #[test]
-fn failing_empty_string() {
+    fn empty_string() {
         assert!(!utils::is_data_url(""));
     }
+}

src/tests/utils/is_file_url.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,33 +5,38 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_unix_file_url() {
+    fn unix_file_url() {
         assert!(utils::is_file_url(
             "file:///home/user/Websites/my-website/index.html"
         ));
     }
 
     #[test]
-fn passing_windows_file_url() {
+    fn windows_file_url() {
         assert!(utils::is_file_url(
             "file:///C:/Documents%20and%20Settings/user/Websites/my-website/assets/images/logo.png"
         ));
     }
 
     #[test]
-fn passing_unix_url_with_backslashes() {
+    fn unix_url_with_backslashes() {
         assert!(utils::is_file_url(
             "file:\\\\\\home\\user\\Websites\\my-website\\index.html"
         ));
     }
 
     #[test]
-fn passing_windows_file_url_with_backslashes() {
+    fn windows_file_url_with_backslashes() {
         assert!(utils::is_file_url(
             "file:\\\\\\C:\\Documents%20and%20Settings\\user\\Websites\\my-website\\assets\\images\\logo.png"
         ));
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -42,34 +45,39 @@ fn passing_windows_file_url_with_backslashes() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+
     #[test]
-fn failing_url_with_no_protocl() {
+    fn url_with_no_protocl() {
         assert!(!utils::is_file_url("//kernel.org"));
     }
 
     #[test]
-fn failing_dot_slash_filename() {
+    fn dot_slash_filename() {
         assert!(!utils::is_file_url("./index.html"));
     }
 
     #[test]
-fn failing_just_filename() {
+    fn just_filename() {
         assert!(!utils::is_file_url("some-local-page.htm"));
     }
 
     #[test]
-fn failing_https_ip_port_url() {
+    fn https_ip_port_url() {
         assert!(!utils::is_file_url("https://1.2.3.4:80/www/index.html"));
     }
 
     #[test]
-fn failing_data_url() {
+    fn data_url() {
         assert!(!utils::is_file_url(
             "data:text/html;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h"
         ));
     }
 
     #[test]
-fn failing_just_word_file() {
+    fn just_word_file() {
         assert!(!utils::is_file_url("file"));
     }
+}

src/tests/utils/is_http_url.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,20 +5,25 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_http_url() {
+    fn http_url() {
         assert!(utils::is_http_url("http://kernel.org"));
     }
 
     #[test]
-fn passing_https_url() {
+    fn https_url() {
         assert!(utils::is_http_url("https://www.rust-lang.org/"));
     }
 
     #[test]
-fn passing_http_url_with_backslashes() {
+    fn http_url_with_backslashes() {
         assert!(utils::is_http_url("http:\\\\freebsd.org\\"));
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -29,29 +32,34 @@ fn passing_http_url_with_backslashes() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+
     #[test]
-fn failing_url_with_no_protocol() {
+    fn url_with_no_protocol() {
         assert!(!utils::is_http_url("//kernel.org"));
     }
 
     #[test]
-fn failing_dot_slash_filename() {
+    fn dot_slash_filename() {
         assert!(!utils::is_http_url("./index.html"));
     }
 
     #[test]
-fn failing_just_filename() {
+    fn just_filename() {
         assert!(!utils::is_http_url("some-local-page.htm"));
     }
 
     #[test]
-fn failing_https_ip_port_url() {
+    fn https_ip_port_url() {
         assert!(!utils::is_http_url("ftp://1.2.3.4/www/index.html"));
     }
 
     #[test]
-fn failing_data_url() {
+    fn data_url() {
         assert!(!utils::is_http_url(
             "data:text/html;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h"
         ));
     }
+}

src/tests/utils/resolve_url.rs

@@ -1,7 +1,3 @@
-use url::ParseError;
-
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -9,9 +5,15 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+    use url::ParseError;
+
     #[test]
-fn passing_from_https_to_level_up_relative() -> Result<(), ParseError> {
-    let resolved_url = utils::resolve_url("https://www.kernel.org", "../category/signatures.html")?;
+    fn from_https_to_level_up_relative() -> Result<(), ParseError> {
+        let resolved_url =
+            utils::resolve_url("https://www.kernel.org", "../category/signatures.html")?;
 
         assert_eq!(
             resolved_url.as_str(),
@@ -22,7 +24,7 @@ fn passing_from_https_to_level_up_relative() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_just_filename_to_full_https_url() -> Result<(), ParseError> {
+    fn from_just_filename_to_full_https_url() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "saved_page.htm",
             "https://www.kernel.org/category/signatures.html",
@@ -37,7 +39,7 @@ fn passing_from_just_filename_to_full_https_url() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_https_url_to_url_with_no_protocol() -> Result<(), ParseError> {
+    fn from_https_url_to_url_with_no_protocol() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "https://www.kernel.org",
             "//www.kernel.org/theme/images/logos/tux.png",
@@ -52,8 +54,8 @@ fn passing_from_https_url_to_url_with_no_protocol() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_https_url_to_url_with_no_protocol_and_on_different_hostname(
-) -> Result<(), ParseError> {
+    fn from_https_url_to_url_with_no_protocol_and_on_different_hostname() -> Result<(), ParseError>
+    {
         let resolved_url = utils::resolve_url(
             "https://www.kernel.org",
             "//another-host.org/theme/images/logos/tux.png",
@@ -68,7 +70,7 @@ fn passing_from_https_url_to_url_with_no_protocol_and_on_different_hostname(
     }
 
     #[test]
-fn passing_from_https_url_to_relative_root_path() -> Result<(), ParseError> {
+    fn from_https_url_to_relative_root_path() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "https://www.kernel.org/category/signatures.html",
             "/theme/images/logos/tux.png",
@@ -83,7 +85,7 @@ fn passing_from_https_url_to_relative_root_path() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_https_to_just_filename() -> Result<(), ParseError> {
+    fn from_https_to_just_filename() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "https://www.w3schools.com/html/html_iframe.asp",
             "default.asp",
@@ -98,7 +100,7 @@ fn passing_from_https_to_just_filename() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_data_url_to_https() -> Result<(), ParseError> {
+    fn from_data_url_to_https() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "data:text/html;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h",
             "https://www.kernel.org/category/signatures.html",
@@ -113,7 +115,7 @@ fn passing_from_data_url_to_https() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_data_url_to_data_url() -> Result<(), ParseError> {
+    fn from_data_url_to_data_url() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "data:text/html;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h",
             "data:text/html;base64,PGEgaHJlZj0iaW5kZXguaHRtbCI+SG9tZTwvYT4K",
@@ -128,7 +130,7 @@ fn passing_from_data_url_to_data_url() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_file_url_to_relative_path() -> Result<(), ParseError> {
+    fn from_file_url_to_relative_path() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "file:///home/user/Websites/my-website/index.html",
             "assets/images/logo.png",
@@ -144,7 +146,7 @@ fn passing_from_file_url_to_relative_path() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_from_file_url_to_relative_path_with_backslashes() -> Result<(), ParseError> {
+    fn from_file_url_to_relative_path_with_backslashes() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "file:\\\\\\home\\user\\Websites\\my-website\\index.html",
             "assets\\images\\logo.png",
@@ -160,7 +162,7 @@ fn passing_from_file_url_to_relative_path_with_backslashes() -> Result<(), Parse
     }
 
     #[test]
-fn passing_from_data_url_to_file_url() -> Result<(), ParseError> {
+    fn from_data_url_to_file_url() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "data:text/html;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h",
             "file:///etc/passwd",
@@ -173,7 +175,7 @@ fn passing_from_data_url_to_file_url() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_preserve_fragment() -> Result<(), ParseError> {
+    fn preserve_fragment() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "http://doesnt-matter.local/",
             "css/fonts/fontmarvelous.svg#fontmarvelous",
@@ -189,7 +191,7 @@ fn passing_preserve_fragment() -> Result<(), ParseError> {
     }
 
     #[test]
-fn passing_resolve_from_file_url_to_file_url() -> Result<(), ParseError> {
+    fn resolve_from_file_url_to_file_url() -> Result<(), ParseError> {
         let resolved_url = if cfg!(windows) {
             utils::resolve_url("file:///c:/index.html", "file:///c:/image.png").unwrap_or(str!())
         } else {
@@ -207,6 +209,7 @@ fn passing_resolve_from_file_url_to_file_url() -> Result<(), ParseError> {
 
         Ok(())
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -215,8 +218,13 @@ fn passing_resolve_from_file_url_to_file_url() -> Result<(), ParseError> {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+    use url::ParseError;
+
     #[test]
-fn failing_from_data_url_to_url_with_no_protocol() -> Result<(), ParseError> {
+    fn from_data_url_to_url_with_no_protocol() -> Result<(), ParseError> {
         let resolved_url = utils::resolve_url(
             "data:text/html;base64,V2VsY29tZSBUbyBUaGUgUGFydHksIDxiPlBhbDwvYj4h",
             "//www.w3schools.com/html/html_iframe.asp",
@@ -227,3 +235,4 @@ fn failing_from_data_url_to_url_with_no_protocol() -> Result<(), ParseError> {
 
         Ok(())
     }
+}

src/tests/utils/retrieve_asset.rs

@@ -1,8 +1,3 @@
-use crate::utils;
-use reqwest::blocking::Client;
-use std::collections::HashMap;
-use std::env;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -10,8 +5,15 @@ use std::env;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+    use reqwest::blocking::Client;
+    use std::collections::HashMap;
+    use std::env;
+
     #[test]
-fn passing_read_data_url() {
+    fn read_data_url() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -37,7 +39,7 @@ fn passing_read_data_url() {
     }
 
     #[test]
-fn passing_read_local_file_with_file_url_parent() {
+    fn read_local_file_with_file_url_parent() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -71,6 +73,7 @@ fn passing_read_local_file_with_file_url_parent() {
             )
         );
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -79,8 +82,14 @@ fn passing_read_local_file_with_file_url_parent() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+    use reqwest::blocking::Client;
+    use std::collections::HashMap;
+
     #[test]
-fn failing_read_local_file_with_data_url_parent() {
+    fn read_local_file_with_data_url_parent() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -102,7 +111,7 @@ fn failing_read_local_file_with_data_url_parent() {
     }
 
     #[test]
-fn failing_read_local_file_with_https_parent() {
+    fn read_local_file_with_https_parent() {
         let cache = &mut HashMap::new();
         let client = Client::new();
 
@@ -122,3 +131,4 @@ fn failing_read_local_file_with_https_parent() {
             }
         }
     }
+}

src/tests/utils/url_has_protocol.rs

@@ -1,5 +1,3 @@
-use crate::utils;
-
 //  ██████╗  █████╗ ███████╗███████╗██╗███╗   ██╗ ██████╗
 //  ██╔══██╗██╔══██╗██╔════╝██╔════╝██║████╗  ██║██╔════╝
 //  ██████╔╝███████║███████╗███████╗██║██╔██╗ ██║██║  ███╗
@@ -7,51 +5,56 @@ use crate::utils;
 //  ██║     ██║  ██║███████║███████║██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚══════╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod passing {
+    use crate::utils;
+
     #[test]
-fn passing_mailto() {
+    fn mailto() {
         assert!(utils::url_has_protocol(
             "mailto:somebody@somewhere.com?subject=hello"
         ));
     }
 
     #[test]
-fn passing_tel() {
+    fn tel() {
         assert!(utils::url_has_protocol("tel:5551234567"));
     }
 
     #[test]
-fn passing_ftp_no_slashes() {
+    fn ftp_no_slashes() {
         assert!(utils::url_has_protocol("ftp:some-ftp-server.com"));
     }
 
     #[test]
-fn passing_ftp_with_credentials() {
+    fn ftp_with_credentials() {
         assert!(utils::url_has_protocol(
             "ftp://user:password@some-ftp-server.com"
         ));
     }
 
     #[test]
-fn passing_javascript() {
+    fn javascript() {
         assert!(utils::url_has_protocol("javascript:void(0)"));
     }
 
     #[test]
-fn passing_http() {
+    fn http() {
         assert!(utils::url_has_protocol("http://news.ycombinator.com"));
     }
 
     #[test]
-fn passing_https() {
+    fn https() {
         assert!(utils::url_has_protocol("https://github.com"));
     }
 
     #[test]
-fn passing_mailto_uppercase() {
+    fn mailto_uppercase() {
         assert!(utils::url_has_protocol(
             "MAILTO:somebody@somewhere.com?subject=hello"
         ));
     }
+}
 
 //  ███████╗ █████╗ ██╗██╗     ██╗███╗   ██╗ ██████╗
 //  ██╔════╝██╔══██╗██║██║     ██║████╗  ██║██╔════╝
@@ -60,24 +63,29 @@ fn passing_mailto_uppercase() {
 //  ██║     ██║  ██║██║███████╗██║██║ ╚████║╚██████╔╝
 //  ╚═╝     ╚═╝  ╚═╝╚═╝╚══════╝╚═╝╚═╝  ╚═══╝ ╚═════╝
 
+#[cfg(test)]
+mod failing {
+    use crate::utils;
+
     #[test]
-fn failing_url_with_no_protocol() {
+    fn url_with_no_protocol() {
         assert!(!utils::url_has_protocol(
             "//some-hostname.com/some-file.html"
         ));
     }
 
     #[test]
-fn failing_relative_path() {
+    fn relative_path() {
         assert!(!utils::url_has_protocol("some-hostname.com/some-file.html"));
     }
 
     #[test]
-fn failing_relative_to_root_path() {
+    fn relative_to_root_path() {
         assert!(!utils::url_has_protocol("/some-file.html"));
     }
 
     #[test]
-fn failing_empty_string() {
+    fn empty_string() {
         assert!(!utils::url_has_protocol(""));
     }
+}