diff options
| author | Mason Reed <mason@vector35.com> | 2025-02-26 20:56:18 -0500 |
|---|---|---|
| committer | Mason Reed <mason@vector35.com> | 2025-02-26 21:05:34 -0500 |
| commit | dc8015f6695ee069279dfbd681000e8b1fcd30e5 (patch) | |
| tree | c1e387c68a249ff90999bbdcc4fa90372516d88c /rust/tests/main_thread.rs | |
| parent | 4608523101738492124069765324141966372229 (diff) | |
Remove module level `Session` initialization in Rust unit tests
This was preventing the enterprise license checkout from dropping, also the initialization story is much better than it was when we added the rstest fixtures, we can get away with initialization in parallel more broadly.
Diffstat (limited to 'rust/tests/main_thread.rs')
| -rw-r--r-- | rust/tests/main_thread.rs | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/rust/tests/main_thread.rs b/rust/tests/main_thread.rs index f74414d5..a42157a9 100644 --- a/rust/tests/main_thread.rs +++ b/rust/tests/main_thread.rs @@ -1,22 +1,16 @@ use binaryninja::headless::Session; -use rstest::*; // TODO: Add a test for MainThreadHandler -#[fixture] -#[once] -fn session() -> Session { - Session::new().expect("Failed to initialize session") -} - -#[rstest] -fn test_not_main_thread(_session: &Session) { +#[test] +fn test_not_main_thread() { // We should never be the main thread. assert!(!binaryninja::is_main_thread()) } -#[rstest] -fn test_main_thread_different(_session: &Session) { +#[test] +fn test_main_thread_different() { + let _session = Session::new().expect("Failed to initialize session"); let calling_thread = std::thread::current(); binaryninja::main_thread::execute_on_main_thread_and_wait(move || { let main_thread = std::thread::current(); |
