-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
difference between expected and current when using plotly (elementId) #174
Comments
same issue |
Just started testing my app, really like the test tool but facing the same issue |
For me it’s just to not test plotly elements at all. Standing by for a
better solution
…On Thu 21 Jun 2018 at 13:52, Jax89 ***@***.***> wrote:
Just started testing my app, really like the test tool but facing the same
issue
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#174 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA-Wdipq3c2MuW2W9alrmBZ2Xs2ySVPBks5t-4kBgaJpZM4SQnE0>
.
|
I am currently running a janky solution which involves exporting parts of the plotly object that don't depend on the element ID. in server.R: in test_script.R: Not sure how much this is really testing anything. |
Could you please try the PR to plotly via #install.packages("remotes")
remotes::install_github("ropenci/plotly#1365") ? You also need to fix the random seed with |
FYI: it's possible to call For an example of usage, see: https://github.com/rstudio/DT/blob/80981d3/R/shiny.R#L116-L124 Also, this is something that needs to be documented in the shinytest web site. |
Fixed in plotly. |
Hi
I'm testing a simpe shiny app with a plotly-figure. Since plotly always generates a new elementId for its plots, shinytest shows this as a difference between expected and current results. In my case (and I guess also in general) this should not count as a real difference though, since nothing has changed.
Is there a way to ignore this?
The text was updated successfully, but these errors were encountered: