Skip to content

Commit

Permalink
update mobile ui test
Browse files Browse the repository at this point in the history
  • Loading branch information
@jtpio
jtpio committed May 10, 2024
1 parent b189c83 commit ad6bec2
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions ui-tests/test/mobile.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,8 @@ test.describe('Mobile', () => {
// wait for the kernel status animations to be finished
await waitForKernelReady(notebook);

// force switching back to command mode to avoid capturing the cursor in the screenshot
await notebook.evaluate(async () => {
await window.jupyterapp.commands.execute('notebook:enter-command-mode');
});
// click to make the blue border around the cell disappear
await page.click('.jp-WindowedPanel-outer');

Check failure on line 42 in ui-tests/test/mobile.spec.ts

View workflow job for this annotation

GitHub Actions / ui-tests (firefox)

[firefox] › test/mobile.spec.ts:24:7 › Mobile › The layout should be more compact on the notebook page 

  1) [firefox] › test/mobile.spec.ts:24:7 › Mobile › The layout should be more compact on the notebook page 
    Error: proxy.click: Target closed
    =========================== logs ===========================
    waiting for locator('.jp-WindowedPanel-outer')
    ============================================================

      40 |
      41 |     // click to make the blue border around the cell disappear
    > 42 |     await page.click('.jp-WindowedPanel-outer');
         |                ^
      43 |
      44 |     // TODO: remove
      45 |     if (browserName === 'firefox') {

        at /home/runner/work/notebook/notebook/ui-tests/test/mobile.spec.ts:42:16

// TODO: remove
if (browserName === 'firefox') {
Expand Down

0 comments on commit ad6bec2

Please sign in to comment.