Skip to content

Commit

Permalink
remove ErrorKing::Other from tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jamesadevine committed Mar 31, 2023
1 parent 535e633 commit acf7714
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions tests/fs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,6 @@ fn set_current_dir_fails_if_node_is_a_file<T: FileSystem>(fs: &T, parent: &Path)
let result = fs.set_current_dir(path);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
}

fn is_dir_returns_true_if_node_is_dir<T: FileSystem>(fs: &T, parent: &Path) {
Expand Down Expand Up @@ -290,7 +289,6 @@ fn remove_dir_fails_if_node_is_a_file<T: FileSystem>(fs: &T, parent: &Path) {
let result = fs.remove_dir(&path);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
assert!(fs.is_file(&path));
}

Expand All @@ -304,7 +302,6 @@ fn remove_dir_fails_if_dir_is_not_empty<T: FileSystem>(fs: &T, parent: &Path) {
let result = fs.remove_dir(&path);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
assert!(fs.is_dir(&path));
assert!(fs.is_file(&child));
}
Expand Down Expand Up @@ -332,7 +329,6 @@ fn remove_dir_all_fails_if_node_is_a_file<T: FileSystem>(fs: &T, parent: &Path)
let result = fs.remove_dir_all(&path);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
assert!(fs.is_file(&path));
}

Expand Down Expand Up @@ -457,7 +453,7 @@ fn read_dir_fails_if_node_is_a_file<T: FileSystem>(fs: &T, parent: &Path) {
assert!(result.is_err());
match result {
Ok(_) => panic!("should be an err"),
Err(err) => assert_eq!(err.kind(), ErrorKind::Other),
Err(_) => (),
}
}

Expand Down Expand Up @@ -506,7 +502,6 @@ fn write_file_fails_if_node_is_a_directory<T: FileSystem>(fs: &T, parent: &Path)
let result = fs.write_file(&path, "test contents");

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
}

fn overwrite_file_overwrites_contents_of_existing_file<T: FileSystem>(fs: &T, parent: &Path) {
Expand Down Expand Up @@ -551,7 +546,6 @@ fn overwrite_file_fails_if_node_is_a_directory<T: FileSystem>(fs: &T, parent: &P
let result = fs.overwrite_file(&path, "test contents");

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
}

fn read_file_returns_contents_as_bytes<T: FileSystem>(fs: &T, parent: &Path) {
Expand Down Expand Up @@ -678,7 +672,6 @@ fn remove_file_fails_if_node_is_a_directory<T: FileSystem>(fs: &T, parent: &Path
let result = fs.remove_file(&path);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
}

fn copy_file_copies_a_file<T: FileSystem>(fs: &T, parent: &Path) {
Expand Down Expand Up @@ -761,7 +754,6 @@ fn copy_file_fails_if_destination_node_is_directory<T: FileSystem>(fs: &T, paren
let result = fs.copy_file(&from, &to);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
}

fn rename_renames_a_file<T: FileSystem>(fs: &T, parent: &Path) {
Expand Down Expand Up @@ -887,12 +879,10 @@ fn rename_fails_if_original_and_destination_are_different_types<T: FileSystem>(
let result = fs.rename(&file, &dir);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);

let result = fs.rename(&dir, &file);

assert!(result.is_err());
assert_eq!(result.unwrap_err().kind(), ErrorKind::Other);
}

fn rename_fails_if_destination_directory_is_not_empty<T: FileSystem>(fs: &T, parent: &Path) {
Expand Down

0 comments on commit acf7714

Please sign in to comment.