greg/diskonaut
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(tests): turn off parallelism when in test mode (#43)

Browse Source 
* Added conditional compilation to turn off/on parallelism

* style(name): clarify constant name

Co-authored-by: Aram Drevekenin <aram@poor.dev>
This commit is contained in:
redzic
2020-06-23 08:58:41 -05:00
committed by GitHub
parent adf3e1ba9a
commit e94f9b942d
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/main.rs
View File

@@ -8,7 +8,7 @@ mod state;
mod ui; mod ui;
use ::failure; use ::failure;
use ::jwalk::Parallelism::RayonDefaultPool; use ::jwalk::Parallelism::{RayonDefaultPool, Serial};
use ::jwalk::WalkDir; use ::jwalk::WalkDir;
use ::std::env; use ::std::env;
use ::std::io; use ::std::io;
@@ -38,6 +38,10 @@ const SHOULD_SHOW_LOADING_ANIMATION: bool = false;
const SHOULD_HANDLE_WIN_CHANGE: bool = true; const SHOULD_HANDLE_WIN_CHANGE: bool = true;
#[cfg(test)] #[cfg(test)]
const SHOULD_HANDLE_WIN_CHANGE: bool = false; const SHOULD_HANDLE_WIN_CHANGE: bool = false;
#[cfg(not(test))]
const SHOULD_SCAN_HD_FILES_IN_MULTIPLE_THREADS: bool = true;
#[cfg(test)]
const SHOULD_SCAN_HD_FILES_IN_MULTIPLE_THREADS: bool = false;
#[derive(StructOpt, Debug)] #[derive(StructOpt, Debug)]
#[structopt(name = "diskonaut")] #[structopt(name = "diskonaut")]
@@ -144,7 +148,11 @@ pub fn start<B>(
let loaded = loaded.clone(); let loaded = loaded.clone();
move || { move || {
'scanning: for entry in WalkDir::new(&path) 'scanning: for entry in WalkDir::new(&path)
.parallelism(RayonDefaultPool) .parallelism(if SHOULD_SCAN_HD_FILES_IN_MULTIPLE_THREADS {
RayonDefaultPool
} else {
Serial
})
.skip_hidden(false) .skip_hidden(false)
.follow_links(false) .follow_links(false)
.into_iter() .into_iter()