Skip to content

Commit

Permalink
Improve the error message of fortify.default() (#4963)
Browse files Browse the repository at this point in the history
  • Loading branch information
yutannihilation authored Aug 25, 2022
1 parent a383fdb commit a58b48c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion R/fortify.r
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ fortify.grouped_df <- function(model, data, ...) {
}
#' @export
fortify.default <- function(model, data, ...) {
msg <- "{.arg data} must be a {.cls data.frame}, or an object coercible by `fortify()`, not {obj_desc(model)}."
msg <- glue("{{.arg data}} must be a {{.cls data.frame}}, or an object coercible by `fortify()`, not {obj_desc(model)}.")
if (inherits(model, "uneval")) {
msg <- c(
msg,
Expand Down
2 changes: 1 addition & 1 deletion tests/testthat/_snaps/fortify.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# fortify.default proves a helpful error with class uneval

`data` must be a <data.frame>, or an object coercible by `fortify()`, not an S3 object with class {.cls uneval}.
`data` must be a <data.frame>, or an object coercible by `fortify()`, not an S3 object with class <uneval>.
i Did you accidentally pass `aes()` to the `data` argument?

0 comments on commit a58b48c

Please sign in to comment.