clean up the tests

molpopgen · Nov 13, 2024 · 19039a7 · 19039a7
1 parent 3e9e39a
commit 19039a7
Showing 1 changed file with 4 additions and 14 deletions.
diff --git a/demes/src/ffi.rs b/demes/src/ffi.rs
@@ -1798,7 +1798,8 @@ fn test_load_from_nonexistent_file() {
     let code = unsafe {
         demes_graph_load_from_file(c_filename.as_ptr(), &mut error, &mut graph_from_file)
     };
-    assert_eq!(code, 1);
+    assert_ne!(code, 0);
+    assert!(graph_from_file.is_null());
     assert!(matches!(error.error, Some(ErrorDetails::BoxedError(_))))
 }
 
@@ -1808,19 +1809,8 @@ fn test_load_from_null_file() {
     let mut error = FFIError::default();
     let code =
         unsafe { demes_graph_load_from_file(std::ptr::null(), &mut error, &mut graph_from_file) };
-    assert_eq!(code, 1);
-    assert!(matches!(
-        error.error,
-        Some(ErrorDetails::UnexpectedNullPointer)
-    ))
-}
-
-#[test]
-fn test_load_from_null_file_all_other_ptr_null() {
-    let mut error = FFIError::default();
-    let code =
-        unsafe { demes_graph_load_from_file(std::ptr::null(), &mut error, std::ptr::null_mut()) };
-    assert_eq!(code, 1);
+    assert_ne!(code, 0);
+    assert!(graph_from_file.is_null());
     assert!(matches!(
         error.error,
         Some(ErrorDetails::UnexpectedNullPointer)