google / google/patrick

API for handling `skip()` of some cases?

Open
#19 2 comments 1 reaction 0 assignees View on GitHub
Dominant language
R
Stars
145
Forks
16
PR merge metrics
No merged PRs in 30d

Description

See https://github.com/r-lib/lintr/pull/2504. I didn't like having to run `skip_if()` multiple times in every test case. It made more sense to me to exclude certain cases up front.

The downside is that the final {testthat} report doesn't list out which tests were skipped, though.

It would be nice if {patrick} could offer some way to conditionally subset `.cases`, but _also_ report which cases were skipped. Maybe just a `subset=` argument _a la_ `lm()` would work?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.