Issue #649: Update snapshots to fix failing CI checks

[nikosbosse]

Description

This PR closes #649

CI tests were failing because snapshots changed. This PR updates the relevant snapshots to fix the CI failures.

Checklist

• My PR is based on a package issue and I have explicitly linked it.
• I have included the target issue or issues in the PR title as follows: issue-number: PR title
• I have tested my changes locally.
• I have added or updated unit tests where necessary.
• I have updated the documentation if required.
• I have built the package locally and run rebuilt docs using roxygen2.
• My code follows the established coding standards and I have run lintr::lint_package() to check for style issues introduced by my changes.
• I have added a news item linked to this PR.
• I have reviewed CI checks for this PR and addressed them as far as I am able.

[seabbs]

It doesn't look like this has helped. Have you made sure all your local dependencies are up to date (and in particular ggplot2?)

[nikosbosse]

All packages are updated - there was an order in which they were failing (first Mac then Windows). So maybe this just reflects the time it needs for the CRAN servers to update? Sill confusing that it would be different on Ubuntu-3.6 and the other Ubuntu systems

[seabbs]

yeah maybe and you are sure you have the same version of ggplot2 as these others? I assume one or other of them is not actually up to date

[nikosbosse]

ok it seems to be essentially that.

my local machine has ggplot2 3.5.

some CI machines download 3.5, others download 3.4.4

Cran has the following:

How did you know it was ggplot2 that caused this?
Should we just merge this and wait for a few days until tests resolve?

[seabbs]

Because I knew there was a new release and its a classic for breaking image-based snapshots. It sounds like we should hold this PR until tomorrow or the next day and then things should have cleared up.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.73%. Comparing base (23b708e) to head (104d3bd).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #650   +/-   ##
=======================================
  Coverage   87.73%   87.73%           
=======================================
  Files          21       21           
  Lines        1786     1786           
=======================================
  Hits         1567     1567           
  Misses        219      219           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nikosbosse]

The more you know! gh actions caches info. And it is possible to delete the caches.

Reference