Skip to content

Use path.package in package resolution tests - #49

Merged
t-kalinowski merged 3 commits into
mainfrom
use-pathpackage-in-tests
Jun 4, 2026
Merged

t-kalinowski merged 3 commits into
mainfrom
use-pathpackage-in-tests

Conversation

@t-kalinowski

Copy link
Copy Markdown
Member

Summary

  • Replace .libPaths()[[1]] heuristics in the test fixtures with direct path.package() checks.
  • Drop the temporary IR_EXPECT_CACHE_DIR plumbing from the CLI tests.
  • Keep the assertions focused on the package path actually reported by R, which makes the tests simpler and less brittle.

Internal changes

  • Updated the script, Quarto, and reticulate fixtures to compare path.package() output against the resolved library path.
  • Simplified the inline CLI expression test to use the same direct package-path check.
  • Removed the extra environment variable setup that was only supporting the old heuristic.

@t-kalinowski
t-kalinowski marked this pull request as ready for review June 4, 2026 17:38
@t-kalinowski
t-kalinowski merged commit 39a25b6 into main Jun 4, 2026
3 of 4 checks passed
@t-kalinowski
t-kalinowski deleted the use-pathpackage-in-tests branch July 7, 2026 12:48
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