ok/jj
1
0
Fork 0
forked from mirrors/jj
Code Pull requests Activity

tests: use tempfile::TempDir instead of env::temp_dir() + random number

Browse source 
Otherwise /tmp/test-<random> file would be left.
This commit is contained in:
Yuya Nishihara 2022-09-07 13:16:35 +09:00
parent 3b835df66e
commit 31e90bdede
1 changed files with 7 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
lib/src/lock.rs
View file

@ -72,16 +72,17 @@ impl Drop for FileLock {
#[cfg(test)]
mod tests {
use std::cmp::max;
use std::{env, thread};
use std::thread;
use byteorder::{LittleEndian, ReadBytesExt, WriteBytesExt};
use super::*;
use crate::testutils;
#[test]
fn lock_basic() {
let number: u32 = rand::random();
let lock_path = env::temp_dir().join(format!("test-{}.lock", number));
let temp_dir = testutils::new_temp_dir();
let lock_path = temp_dir.path().join("test.lock");
assert!(!lock_path.exists());
{
let _lock = FileLock::lock(lock_path.clone());
@ -92,9 +93,9 @@ mod tests {
#[test]
fn lock_concurrent() {
let number: u32 = rand::random();
let data_path = env::temp_dir().join(format!("test-{}", number));
let lock_path = env::temp_dir().join(format!("test-{}.lock", number));
let temp_dir = testutils::new_temp_dir();
let data_path = temp_dir.path().join("test");
let lock_path = temp_dir.path().join("test.lock");
let mut data_file = OpenOptions::new()
.create(true)
.write(true)