Skip to content

update to the new jaspfile testing structure#411

Open
FBartos wants to merge 4 commits intojasp-stats:masterfrom
FBartos:update-jaspTools
Open

update to the new jaspfile testing structure#411
FBartos wants to merge 4 commits intojasp-stats:masterfrom
FBartos:update-jaspTools

Conversation

@FBartos
Copy link
Collaborator

@FBartos FBartos commented Feb 27, 2026

jasp files are now in

tests\testthat\jaspfiles\
- verified
- library
- other

tests are to be re-generated via
makeTestsFromExamples() based on the latest version of jaspTools

@FBartos FBartos marked this pull request as ready for review February 27, 2026 10:54
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the test infrastructure to align with jaspTools 0.20.1, reorganizing .jasp example files from examples/ into tests/testthat/jaspfiles/ subdirectories (library/, other/, verified/). Tests have been regenerated using makeTestsFromExamples() to reflect the new file locations.

Changes:

  • Reorganized .jasp test files into structured subdirectories under tests/testthat/jaspfiles/
  • Updated test file paths from test_path("..", "..", "examples", "...") to test_path("jaspfiles", "library", "...")
  • Removed synthetic test file test-petpeese.R (PetPeese is now tested via library examples)
  • Removed plot snapshots for regeneration (deleted 6 SVG files under _snaps/petpeese/)
  • Added jaspTools 0.20.1 to renv.lock with expanded package metadata
  • Minor whitespace cleanup in R/bayesianbinomialmetaanalysis.R

Reviewed changes

Copilot reviewed 17 out of 67 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/testthat/test-library-Precognition.R Updated path from examples/ to jaspfiles/library/
tests/testthat/jaspfiles/library/Precognition.jasp Binary .jasp file added to new location
tests/testthat/test-petpeese.R Removed synthetic test (replaced by library-based tests)
tests/testthat/_snaps/petpeese/*.svg Removed 6 plot snapshots for regeneration
renv.lock Added jaspTools 0.20.1, expanded metadata for jaspBase/jaspGraphs/jaspSem, updated renv 1.1.6→1.1.7
R/bayesianbinomialmetaanalysis.R Removed blank line (whitespace cleanup)

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.

2 participants