Wrong `snapshot_accept()` suggestion if period in test file name

[moodymudskipper]

I have a test file called "test-data.frame.R" (notice there are 2 periods in full name)

When running tests with a new output I get the suggestion to run snapshot_accept("data.frame")

When I run it however I get "No snapshots to update"

This is due to testthat:::snapshot_meta where we find :

files <- ifelse(tools::file_ext(files) == "", paste0(files, ".md"))

There "frame" is returned by tools::file_ext() so no suffix is added.

A solution might be to have :

files <- ifelse(tools::file_ext(files) != "md", paste0(files, ".md")

Another solution would be to manage to have the suggestion be snapshot_accept("data.frame.md")

[d-morrison]

I ran into the same issue; here's a reprex: https://github.com/d-morrison/testthat.suffix.issue

[d-morrison]

snapshot_accept() (with no arguments) is a possible workaround (but it accepts changes for all snapshots).