diff options
author | Ben Morrison <ben@gbmor.dev> | 2020-05-28 12:51:07 -0400 |
---|---|---|
committer | Ben Morrison <ben@gbmor.dev> | 2020-05-28 12:51:07 -0400 |
commit | 87bf08a5cfa640d92d8b9b1abcecf269d97c8773 (patch) | |
tree | e196bcf2910ae2f77a0d66750a30a89f6604eb52 | |
parent | 3578c803914042fc8c05aa54ad510f7ec2438d69 (diff) | |
download | clinte-87bf08a5cfa640d92d8b9b1abcecf269d97c8773.tar.gz |
expanded testing
-rw-r--r-- | src/db.rs | 6 | ||||
-rw-r--r-- | src/error.rs | 3 | ||||
-rw-r--r-- | src/posts.rs | 48 |
3 files changed, 54 insertions, 3 deletions
diff --git a/src/db.rs b/src/db.rs index 02cd776..20ad5a1 100644 --- a/src/db.rs +++ b/src/db.rs @@ -22,7 +22,7 @@ pub struct Post { #[derive(Debug, Deserialize, Serialize)] pub struct Posts { - pub posts: Vec<Post>, + posts: Vec<Post>, } #[derive(Debug)] @@ -90,6 +90,10 @@ impl Posts { "Couldn't write data to clinte.json", ); } + + pub fn posts(&self) -> Vec<Post> { + self.posts.clone() + } } #[cfg(test)] diff --git a/src/error.rs b/src/error.rs index aa21021..ee4511c 100644 --- a/src/error.rs +++ b/src/error.rs @@ -14,7 +14,8 @@ where if *conf::DEBUG { log::error!("--> {:?}", err); } - std::process::exit(1); + //std::process::exit(1); + panic!("{:?}", err); } } } diff --git a/src/posts.rs b/src/posts.rs index b286469..c38c84f 100644 --- a/src/posts.rs +++ b/src/posts.rs @@ -63,7 +63,7 @@ pub fn display() -> error::Result<()> { let all = db::Posts::get_all(db::PATH); let mut postvec = Vec::new(); - all.posts.iter().enumerate().for_each(|(id, post)| { + all.posts().iter().enumerate().for_each(|(id, post)| { let newpost = format!( "{}. {} -> by {}\n{}\n\n", id + 1, @@ -97,10 +97,15 @@ pub fn update_handler(id: usize) -> error::Result<()> { id_num_in -= 1; + #[cfg(not(test))] let user = &*user::NAME; + let mut all = db::Posts::get_all(db::PATH); let post = all.get(id_num_in); + #[cfg(test)] + let user = &post.author; + if *user != post.author { println!(); println!("Users don't match. Can't update post!"); @@ -108,6 +113,10 @@ pub fn update_handler(id: usize) -> error::Result<()> { std::process::exit(1); } + #[cfg(test)] + let new_title = String::from("TEST_TITLE"); + + #[cfg(not(test))] let mut new_title = String::new(); println!("Updating post {}", id_num_in); @@ -115,9 +124,16 @@ pub fn update_handler(id: usize) -> error::Result<()> { println!("Current Title: {}", post.title); println!(); println!("Enter new title:"); + + #[cfg(not(test))] io::stdin().read_line(&mut new_title)?; + #[cfg(test)] + let body_raw = String::from("TEST_BODY"); + + #[cfg(not(test))] let body_raw = str_to_utf8(&ed::call(&post.body)); + let body = if body_raw.len() > 500 { &body_raw[..500] } else { @@ -171,3 +187,33 @@ pub fn delete_handler(id: usize) -> error::Result<()> { Ok(()) } + +#[cfg(test)] +mod tests { + use super::*; + + use std::fs; + + #[test] + fn test_str_to_utf8() { + let lhs = "foobar"; + let rhs = str_to_utf8(lhs); + assert_eq!(lhs, rhs); + } + + #[test] + fn display_doesnt_explode() { + assert!(display().is_ok()); + } + + #[test] + fn test_update_handler() { + fs::copy(db::PATH, "clinte_bak.json").unwrap(); + update_handler(1).unwrap(); + let all = db::Posts::get_all(db::PATH); + let post = all.get(0); + assert_eq!(post.title, "TEST_TITLE"); + assert_eq!(post.body, "TEST_BODY"); + fs::rename("clinte_bak.json", db::PATH).unwrap(); + } +} |