diff options
| author | Teddy Wing | 2016-04-24 06:36:32 -0400 | 
|---|---|---|
| committer | Teddy Wing | 2016-04-24 06:36:32 -0400 | 
| commit | 66193af4e53799299c04ecf8715e44693f1bff8a (patch) | |
| tree | 33696e3319e10e1d483c1c13993dacf6ef8593e0 /src | |
| parent | 15759804bd3be99fc6374ee993038440267fdfb5 (diff) | |
| download | mutt-alias-auto-add-66193af4e53799299c04ecf8715e44693f1bff8a.tar.bz2 | |
Extract `alias_write_to_file` test function contents to helper
Create a new helper function that does the work of
`alias_write_to_file_must_write_given_alias_to_file`. We want to be able
to add another test of the `write_to_file` method that does the same
thing but for non-preexisting aliases. Making a helper method will allow
us to avoid duplicating work.
Diffstat (limited to 'src')
| -rw-r--r-- | src/tests.rs | 15 | 
1 files changed, 10 insertions, 5 deletions
| diff --git a/src/tests.rs b/src/tests.rs index 708121e..e4ec254 100644 --- a/src/tests.rs +++ b/src/tests.rs @@ -122,10 +122,7 @@ fn update_alias_id_increments_alias() {  } -#[test] -fn alias_write_to_file_must_write_given_alias_to_file() { -    let mut alias = update_alias_id_sample_alias(); - +fn alias_write_to_file_helper(alias: &mut Alias) -> String {      // Create a new test file      let test_file = "./testdata/write_to_file";      fs::copy("./testdata/aliases", test_file).expect("Alias file copy failed"); @@ -148,5 +145,13 @@ fn alias_write_to_file_must_write_given_alias_to_file() {      let file_contents: Vec<&str> = file_contents.split('\n').collect();      fs::remove_file(test_file).expect("Failed to delete test file"); -    assert_eq!(alias.to_string(), file_contents[file_contents.len() - 2]); +    file_contents[file_contents.len() - 2].to_string() +} + +#[test] +fn alias_write_to_file_must_write_given_alias_to_file() { +    let mut alias = update_alias_id_sample_alias(); +    let alias_line = alias_write_to_file_helper(&mut alias); + +    assert_eq!(alias.to_string(), alias_line);  } | 
