Skip to content

Adding Accept and Reject All buttons for bulk code changes. #38

Adding Accept and Reject All buttons for bulk code changes.

Adding Accept and Reject All buttons for bulk code changes. #38

Triggered via pull request October 29, 2025 02:35
Status Failure
Total duration 3m 55s
Artifacts 1

tests.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

6 errors and 1 notice
ui-tests
Process completed with exit code 1.
tests/split-cell-diff.spec.ts:50:7 › Cell Diff Extension › should show diff with action buttons: ui-tests/tests/split-cell-diff.spec.ts#L18
2) tests/split-cell-diff.spec.ts:50:7 › Cell Diff Extension › should show diff with action buttons Error: locator.evaluate: Test timeout of 60000ms exceeded. 16 | ) { 17 | await page.notebook.createNew(); > 18 | await page.notebook.setCell(0, 'code', originalSource); | ^ 19 | 20 | await page.evaluate( 21 | async ({ originalSource, newSource, command }) => { at NotebookHelper.setCell (/home/runner/work/jupyterlab-diff/jupyterlab-diff/ui-tests/node_modules/@jupyterlab/galata/src/helpers/notebook.ts:1269:18) at setupCellWithDiff (/home/runner/work/jupyterlab-diff/jupyterlab-diff/ui-tests/tests/split-cell-diff.spec.ts:18:3) at /home/runner/work/jupyterlab-diff/jupyterlab-diff/ui-tests/tests/split-cell-diff.spec.ts:54:24
tests/split-cell-diff.spec.ts:50:7 › Cell Diff Extension › should show diff with action buttons: ui-tests/tests/split-cell-diff.spec.ts#L0
2) tests/split-cell-diff.spec.ts:50:7 › Cell Diff Extension › should show diff with action buttons Test timeout of 60000ms exceeded.
tests/unified-cell-diff.spec.ts:63:7 › Unified Cell Diff Extension › should accept all changes when the accept button is clicked: ui-tests/tests/unified-cell-diff.spec.ts#L74
1) tests/unified-cell-diff.spec.ts:63:7 › Unified Cell Diff Extension › should accept all changes when the accept button is clicked Retry #2 ─────────────────────────────────────────────────────────────────────────────────────── Error: expect(locator).not.toBeVisible() failed Locator: getByText('Accept All') Expected: not visible Received: visible Timeout: 5000ms Call log: - Expect "not toBeVisible" with timeout 5000ms - waiting for getByText('Accept All') 9 × locator resolved to <button class="jp-merge-accept-button" title="Accept all changes in this notebook">Accept All</button> - unexpected value "visible" 72 | await acceptButton.click(); 73 | > 74 | await expect(acceptButton).not.toBeVisible(); | ^ 75 | 76 | const cellContent = await getCellContent(page); 77 | expect(cellContent).toBe(newSource); at /home/runner/work/jupyterlab-diff/jupyterlab-diff/ui-tests/tests/unified-cell-diff.spec.ts:74:36
tests/unified-cell-diff.spec.ts:63:7 › Unified Cell Diff Extension › should accept all changes when the accept button is clicked: ui-tests/tests/unified-cell-diff.spec.ts#L74
1) tests/unified-cell-diff.spec.ts:63:7 › Unified Cell Diff Extension › should accept all changes when the accept button is clicked Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: expect(locator).not.toBeVisible() failed Locator: getByText('Accept All') Expected: not visible Received: visible Timeout: 5000ms Call log: - Expect "not toBeVisible" with timeout 5000ms - waiting for getByText('Accept All') 9 × locator resolved to <button class="jp-merge-accept-button" title="Accept all changes in this notebook">Accept All</button> - unexpected value "visible" 72 | await acceptButton.click(); 73 | > 74 | await expect(acceptButton).not.toBeVisible(); | ^ 75 | 76 | const cellContent = await getCellContent(page); 77 | expect(cellContent).toBe(newSource); at /home/runner/work/jupyterlab-diff/jupyterlab-diff/ui-tests/tests/unified-cell-diff.spec.ts:74:36
tests/unified-cell-diff.spec.ts:63:7 › Unified Cell Diff Extension › should accept all changes when the accept button is clicked: ui-tests/tests/unified-cell-diff.spec.ts#L74
1) tests/unified-cell-diff.spec.ts:63:7 › Unified Cell Diff Extension › should accept all changes when the accept button is clicked Error: expect(locator).not.toBeVisible() failed Locator: getByText('Accept All') Expected: not visible Received: visible Timeout: 5000ms Call log: - Expect "not toBeVisible" with timeout 5000ms - waiting for getByText('Accept All') 9 × locator resolved to <button class="jp-merge-accept-button" title="Accept all changes in this notebook">Accept All</button> - unexpected value "visible" 72 | await acceptButton.click(); 73 | > 74 | await expect(acceptButton).not.toBeVisible(); | ^ 75 | 76 | const cellContent = await getCellContent(page); 77 | expect(cellContent).toBe(newSource); at /home/runner/work/jupyterlab-diff/jupyterlab-diff/ui-tests/tests/unified-cell-diff.spec.ts:74:36
🎭 Playwright Run Summary
1 failed tests/unified-cell-diff.spec.ts:63:7 › Unified Cell Diff Extension › should accept all changes when the accept button is clicked 1 flaky tests/split-cell-diff.spec.ts:50:7 › Cell Diff Extension › should show diff with action buttons 8 passed (1.7m)

Artifacts

Produced during runtime
Name Size Digest
jupyterlab-diff-ui-tests-report-18895287838
10.1 MB
sha256:9c21ec1f09c675638b9192a36bfc21cd393fbb4bcd953b75fc666c4a39637eb1