Skip to content

Commit fa67138

Browse files
author
dimxy
committed
refactor (borngraced): nits parsing envs
1 parent 9cc04cf commit fa67138

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

mm2src/mm2_main/tests/integration_tests_common/mod.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,14 @@ pub fn test_mm_start_impl() {
2323
if let Ok(conf) = var("_MM2_TEST_CONF") {
2424
log!("test_mm_start] Starting the MarketMaker...");
2525
let conf: Json = json::from_str(&conf).unwrap();
26+
2627
#[cfg(not(target_arch = "wasm32"))]
2728
let filter = None;
2829
#[cfg(target_arch = "wasm32")]
29-
let filter = if let Ok(log_var) = var("RUST_LOG") {
30-
LogLevel::from_str(&log_var).ok(); // Actually filter is used for wasm only. For native RUST_LOG is parsed by env_logger directly, allowing setting multiple targets
31-
} else {
32-
None
33-
};
30+
let filter = var("RUST_LOG")
31+
.map(|log_var| LogLevel::from_str(&log_var).unwrap_or_default())
32+
.ok(); // Actually filter is used for wasm only. For native RUST_LOG is parsed by env_logger directly, allowing setting multiple targets
33+
3434
let params = LpMainParams::with_conf(conf).log_filter(filter);
3535
let ctx = block_on(lp_main(params, &|_ctx| (), "TEST".into(), "TEST".into())).unwrap();
3636
block_on(lp_run(ctx))

mm2src/mm2_test_helpers/src/for_tests.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1382,7 +1382,7 @@ impl MarketMakerIt {
13821382
.env("_MM2_TEST_CONF", try_s!(json::to_string(&conf)))
13831383
.env("MM2_UNBUFFERED_OUTPUT", "1")
13841384
.env("RUST_LOG", read_rust_log().as_str())
1385-
.envs(envs_fixed.iter().map(|(k, v)| (k, v)))
1385+
.envs(envs_fixed.into_iter())
13861386
.stdout(try_s!(log.try_clone()))
13871387
.stderr(log)
13881388
.spawn());

0 commit comments

Comments
 (0)