updated test coverage

Author: RPG-Alex

fuzz/fuzz_targets/sql_docs.rs

@@ -6,5 +6,67 @@ use std::str;
 use sql_docs::SqlDoc;
 
 fuzz_target!(|data: String| {
-    
+    let _ = SqlDoc::from_str(&data).build();
+
+    let _ = SqlDoc::from_str(&data)
+        .deny(&data)
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .flatten_multiline()
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .flatten_multiline_with("")
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .flatten_multiline_with(" ")
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .flatten_multiline_with("\n")
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .flatten_multiline_with(" | ")
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .preserve_multiline()
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .flatten_multiline()
+        .preserve_multiline()
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .preserve_multiline()
+        .flatten_multiline()
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .deny("nonexistent.sql")
+        .flatten_multiline()
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .deny("nonexistent.sql")
+        .flatten_multiline_with(" ")
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .deny(&data)
+        .flatten_multiline_with("\n")
+        .build();
+
+    let _ = SqlDoc::from_str(&data)
+        .deny("a.sql")
+        .deny("b.sql")
+        .flatten_multiline_with(" | ")
+        .preserve_multiline()
+        .flatten_multiline()
+        .build();
+
 });

src/comments.rs

@@ -521,7 +521,7 @@ CREATE TABLE posts (
     }
 
     fn multiline_comments_sql() -> &'static str {
-        r#"/* Users table stores user account information 
+        r"/* Users table stores user account information 
 multiline */
 CREATE TABLE users (
     /* Primary key 
@@ -556,7 +556,7 @@ CREATE TABLE posts (
     /* When the post was created 
     multiline */
     published_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP
-);"#
+);"
     }
 
     fn no_comments_sql() -> &'static str {

src/docs.rs

@@ -319,7 +319,7 @@ mod tests {
         assert_eq!(sql_doc_val_column.name(), "id");
     }
 
- fn single_line_comments_sql() -> &'static str {
+    fn single_line_comments_sql() -> &'static str {
         "-- Users table stores user account information
 CREATE TABLE users (
     -- Primary key
@@ -348,7 +348,7 @@ CREATE TABLE posts (
     }
 
     fn multiline_comments_sql() -> &'static str {
-        r#"/* Users table stores user account information 
+        r"/* Users table stores user account information 
 multiline */
 CREATE TABLE users (
     /* Primary key 
@@ -383,7 +383,7 @@ CREATE TABLE posts (
     /* When the post was created 
     multiline */
     published_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP
-);"#
+);"
     }
 
     fn no_comments_sql() -> &'static str {
@@ -435,7 +435,6 @@ CREATE TABLE posts (
 "
     }
 
-
     #[test]
     fn generate_docs_files() -> Result<(), Box<dyn std::error::Error>> {
         use crate::{ast::ParsedSqlFileSet, comments::Comments, files::SqlFileSet};
@@ -827,4 +826,27 @@ CREATE TABLE posts (
         assert_eq!(cols[1].name(), "username");
         assert_eq!(cols[1].doc(), Some("login name"));
     }
+    #[test]
+    fn test_from_sql_file_doc_into_vec_table_doc_preserves_contents_and_order() {
+        let t1 = TableDoc::new(
+            None,
+            "users".to_string(),
+            Some("users doc".to_string()),
+            vec![ColumnDoc::new("id".to_string(), Some("pk".to_string()))],
+            None,
+        );
+        let t2 = TableDoc::new(
+            Some("analytics".to_string()),
+            "events".to_string(),
+            None,
+            vec![ColumnDoc::new("payload".to_string(), None)],
+            None,
+        );
+
+        let sql_file_doc = SqlFileDoc::new(vec![t1.clone(), t2.clone()]);
+        let got: Vec<TableDoc> = Vec::from(sql_file_doc);
+
+        let expected = vec![t1, t2];
+        assert_eq!(got, expected);
+    }
 }

src/files.rs

@@ -352,4 +352,31 @@ mod tests {
         let bad_file_list = SqlFileSet::new(Path::new(invalid_dir), &[]);
         assert!(bad_file_list.is_err());
     }
+    #[test]
+    fn test_from_sql_files_list_into_vec_pathbuf_preserves_contents_and_order()
+    -> Result<(), Box<dyn std::error::Error>> {
+        let base = env::temp_dir().join("from_impl_sql_files_list_test");
+        let _ = fs::remove_dir_all(&base);
+        fs::create_dir_all(&base)?;
+        let file1 = base.join("one.sql");
+        let file2 = base.join("two.sql");
+        let noise = base.join("ignore.txt");
+        fs::File::create(&file1)?;
+        fs::File::create(&file2)?;
+        fs::File::create(&noise)?;
+        let sql_file_list = SqlFilesList::new(&base, &[])?;
+        let expected: Vec<PathBuf> = sql_file_list.sql_files().to_vec();
+        let got: Vec<PathBuf> = Vec::from(sql_file_list);
+        assert_eq!(got, expected);
+        let _ = fs::remove_dir_all(&base);
+        Ok(())
+    }
+    #[test]
+    fn test_sql_file_new_from_str_has_no_path_and_preserves_content() {
+        let sql = "SELECT * FROM users;".to_string();
+        let file = SqlFile::new_from_str(sql.clone());
+        assert!(file.path().is_none());
+        assert!(file.path_into_path_buf().is_none());
+        assert_eq!(file.content(), sql);
+    }
 }

src/sql_doc.rs

@@ -668,4 +668,18 @@ mod tests {
         let _ = fs::remove_dir_all(&base);
         Ok(())
     }
+    #[test]
+    fn test_sql_doc_from_str_builds_expected_builder() {
+        let content = "CREATE TABLE t(id INTEGER);";
+
+        let actual = SqlDoc::from_str(content);
+
+        let expected = SqlDocBuilder {
+            source: crate::sql_doc::SqlFileDocSource::FromString(content),
+            deny: vec![],
+            multiline_flat: MultiFlatten::NoFlat,
+        };
+
+        assert_eq!(actual, expected);
+    }
 }