Skip to content

Commit d5e374f

Browse files
committed
as_logger_config.list no longer ignores propagate
1 parent e6ae949 commit d5e374f

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

R/logger_config.R

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,8 @@ as_logger_config.list <- function(x){
9191
threshold = x$threshold,
9292
filters = x$filters,
9393
appenders = x$appenders,
94-
exception_handler = x$exception_handler
94+
exception_handler = x$exception_handler,
95+
propagate = x$propagate
9596
)
9697
}
9798

tests/testthat/test_logger_config.R

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,9 @@ test_that("logger_config works as expected", {
1515
expect_true(inherits(tl$appenders[[1]], "Appender"))
1616
expect_false(is_virgin_Logger(tl))
1717

18+
expect_silent(tl$fatal("test")) # propagate
19+
expect_equal(tl$last_event$level, 400) # force level filter
20+
1821
tl <- get_logger("test")$config(logger_config())
1922
expect_true(is_virgin_Logger(tl))
2023
})

tests/testthat/test_parallel.R

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,11 +29,11 @@ for (strategy in c(
2929
lg <- get_logger("par_root/par_child")
3030

3131
x <- future::future(lr$info("root_logger"))
32-
x <- future::value(x)
32+
expect_silent(x <- future::value(x))
3333
expect_match(readLines(tf)[[1]], "root_logger")
3434

3535
y <- future::future(lg$info("child_logger"))
36-
y <- future::value(y)
36+
expect_silent(y <- future::value(y))
3737
expect_match(readLines(tf)[[2]], "child_logger")
3838

3939
future.apply::future_lapply(

0 commit comments

Comments
 (0)