Skip to content

Commit

Permalink
e2e test: fix reprex case so as not to depend on imgur (#5971)
Browse files Browse the repository at this point in the history
No longer depend on image upload for basic viewer test of reprex

### QA Notes

All tests should pass

@:viewer
  • Loading branch information
testlabauto authored Jan 13, 2025
1 parent 505eb17 commit d625154
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions test/e2e/tests/viewer/viewer.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ test.describe('Viewer', { tag: [tags.VIEWER] }, () => {
logger.log('Sending code to console');
await app.workbench.console.executeCode('R', rReprexScript, '>');

const rnorm = app.workbench.viewer.getViewerLocator('code.sourceCode').filter({ hasText: 'x <- rnorm(100)' });
const rnorm = app.workbench.viewer.getViewerLocator('code.sourceCode').filter({ hasText: 'rbinom' });

await rnorm.waitFor({ state: 'attached' });

Expand All @@ -90,8 +90,4 @@ modelsummary(m1)`;
const rReactableScript = `library(reactable)
mtcars |> reactable::reactable()`;


const rReprexScript = `reprex::reprex({
x <- rnorm(100)
plot(x, sin(x))
})`;
const rReprexScript = `reprex::reprex(rbinom(3, size = 10, prob = 0.5), comment = "#;-)")`;

0 comments on commit d625154

Please sign in to comment.