Skip to content

Commit e63b553

Browse files
committed
atime test update
1 parent 074a46a commit e63b553

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

.ci/atime/tests.R

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -286,20 +286,18 @@ test.list <- atime::atime_test_list(
286286
Slow = "548410d23dd74b625e8ea9aeb1a5d2e9dddd2927", # Parent of the first commit in the PR (https://github.com/Rdatatable/data.table/commit/548410d23dd74b625e8ea9aeb1a5d2e9dddd2927)
287287
Fast = "c0b32a60466bed0e63420ec105bc75c34590865e"), # Commit in the PR (https://github.com/Rdatatable/data.table/pull/7144/commits) that uses a much faster implementation
288288

289-
"tables() !recursive refactor in #2606" = atime::atime_test(
289+
"tables(recursive=FALSE) performance unchanged in #7141" = atime::atime_test(
290+
N = as.integer(10^seq(1, 4)),
290291
setup = {
291292
test_env <- new.env()
292-
for (i in 1:N) {
293-
assign(paste0("dt_perf_test", i), data.table(a=1), envir = test_env)
294-
assign(paste0("vec_perf_test", i), 1, envir = test_env)
293+
for (i in seq_len(N)) {
294+
assign(paste0("dt_", i), data.table::data.table(a = 1), envir = test_env)
295+
assign(paste0("vec_", i), 1:10, envir = test_env)
295296
}
296297
},
297-
expr = {
298-
data.table::tables(env = test_env, silent = TRUE, index = TRUE);
299-
NULL
300-
},
301-
before = "7c59daaed1836db57747d92494b1ce96612bbf80", # Parent of the first commit in the PR (https://github.com/Rdatatable/data.table/commit/7c59daaed1836db57747d92494b1ce96612bbf80)
302-
after = "6fee38c89200e10dcc10a6f2057ab784f9a011e7"), # Commit in the PR (https://github.com/Rdatatable/data.table/pull/7141/commits)
298+
expr = data.table::tables(env = test_env, recursive = FALSE, silent = TRUE),
303299

300+
Slow = "7c59daaed1836db57747d92494b1ce96612bbf80",
301+
Fast = "6fee38c89200e10dcc10a6f2057ab784f9a011e7"),
304302
tests=extra.test.list)
305303
# nolint end: undesirable_operator_linter.

0 commit comments

Comments
 (0)