Skip to content

Try even harder to suppress pak's output#2681

Merged
jennybc merged 1 commit intomainfrom
suppress-test-noise
Mar 11, 2026
Merged

Try even harder to suppress pak's output#2681
jennybc merged 1 commit intomainfrom
suppress-test-noise

Conversation

@jennybc
Copy link
Member

@jennybc jennybc commented Mar 11, 2026

Somehow the nested use of callr (first layer is parallel testthat workers, second layer is inside pak) gives rise to some very difficult to suppress output. This is just about reducing noise in the testthat report.

Somehow the nested use of callr (first layer is parallel testthat workers, second layer is inside pak) gives rise to some very difficult to suppress output. This is just about reducing noise in the testthat report.
@jennybc
Copy link
Member Author

jennybc commented Mar 11, 2026

Before this PR:

> test_check("devtools")
  Starting 2 test processes.
  callr_message:
  callr_message: → Will download 1 package with unknown size.
  callr_message: ℹ No downloads are needed
  callr_message: ✔ : [1.9s]
  callr_message:
  callr_message: → Will download 1 package with unknown size.
  callr_message: ℹ No downloads are needed
  callr_message: ✔ : [323ms]
  callr_message:
  callr_message: → Will download 1 package with unknown size.
  callr_message: ℹ No downloads are needed
  callr_message: ✔ : [297ms]
  callr_message:
  callr_message: → Will download 1 package with unknown size.
  callr_message: ℹ No downloads are needed
  callr_message: ✔ : [234ms]
  callr_message:
  callr_message: → Will download 1 package with unknown size.
  callr_message: ℹ No downloads are needed
  callr_message: ✔ : [295ms]
  callr_message:
  callr_message: → Will download 1 package with unknown size.
  callr_message: ℹ No downloads are needed
  callr_message: ✔ : [271ms]
  callr_message:
  callr_message: → Will download 1 package with unknown size.
  callr_message: ℹ No downloads are needed
  callr_message: ✔ : [380ms]
  [ FAIL 0 | WARN 0 | SKIP 7 | PASS 133 ]

After this PR:

> test_check("devtools")
  Starting 2 test processes.
  [ FAIL 0 | WARN 0 | SKIP 7 | PASS 133 ]

@jennybc jennybc merged commit cc625e3 into main Mar 11, 2026
14 checks passed
@jennybc jennybc deleted the suppress-test-noise branch March 11, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant