plotly / plotly/plotly.R

Run `testthat::snapshot_review('plotly-symbol/')` to review changes

Open
#2,151 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
R
Stars
2.7k
Forks
641
PR merge metrics
No merged PRs in 30d

Description

https://github.com/plotly/plotly.R/runs/6295712575?check_suite_focus=true#step:12:1691

Failure (test-plotly-symbol.R:58:3): Can specify a scale manually
Snapshot of path to 'plotly-symbol/plotly-symbol-symbol-manual.svg' has changed
Run testthat::snapshot_review('plotly-symbol/') to review changes
Backtrace:

 1. plotly expect_traces(p, 2, "symbol-manual")
      at test-plotly-symbol.R:58:2
 2. plotly:::expect_doppelganger_built(...)
      at test-plotly-symbol.R:4:2
 3. plotly:::expect_doppelganger(p, name, ...)
      at tests/testthat/helper-vdiffr.R:35:2
 4. testthat::expect_snapshot_file(path = path, name = file, cran = FALSE)
      at tests/testthat/helper-vdiffr.R:30:2

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with test-plotly-symbol.R:58 and run testthat::snapshot_review('plotly-symbol/') as requested. Inspect the changed plotly-symbol/plotly-symbol-symbol-manual.svg snapshot, then rerun the relevant test to confirm the snapshot review is resolved.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
data-visualization, testing
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.