Skip to content

messed up workflow... changes should be on development branch - #75

Merged
ohagen merged 2 commits into
developmentfrom
master
Aug 13, 2025
Merged

ohagen merged 2 commits into
developmentfrom
master

Conversation

@ohagen

@ohagen ohagen commented Aug 12, 2025

Copy link
Copy Markdown
Member

No description provided.

ohagen added 2 commits August 12, 2025 17:20
fix #72
with_mock() (also local_mock())officially deprecated since testthat’s 3rd edition. tests switched to local_mocked_bindings()
krlmlr.github.io
+3
testthat.r-lib.org
+3
mpn.metworx.com
+3
.

These newer functions are smarter—they let you temporarily override bindings within your package’s namespace and co‑work with byte‑compiled R, with fewer internal gymnastics
x.com
+10
testthat.r-lib.org
+10
cran.rstudio.com
+10
.
Fix the testthat mocking target for base functions. local_mocked_bindings must point to the base namespace environment, not baseenv(), to find bindings like file.exists and dir.create.
Remove the skip added because mocking didn’t work.
Optionally set the testthat edition in DESCRIPTION for consistency.

test-input_creation.R
Point local_mocked_bindings to base::.BaseNamespaceEnv and remove the temporary skip.

DESCRIPTION
Declare testthat edition for stable behavior across environments.
@ohagen
ohagen merged commit 7ab07ac into development Aug 13, 2025
3 of 5 checks passed

This branch was previously deployed

1 inactive deployment
github-pages — 88dd7ce8 Deployed Aug 12, 2025 by ohagen via deploy #21
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