refactor: update Excel import/export functionality #330
Annotations
10 errors, 2 warnings, and 1 notice
|
E2E tests:
e2e/merge-cells.test.ts#L41
10) [firefox] › e2e/merge-cells.test.ts:32:5 › unmerge cells ─────────────────────────────────────
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
- waiting for getByTestId('toolbar-merge-cell')
39 |
40 | // Merge cells
> 41 | await page.getByTestId('toolbar-merge-cell').click();
| ^
42 | await sleep(50);
43 |
44 | // Unmerge by clicking again
at /home/runner/work/excel/excel/e2e/merge-cells.test.ts:41:48
|
|
E2E tests:
e2e/merge-cells.test.ts#L29
9) [firefox] › e2e/merge-cells.test.ts:24:5 › merge cell select dropdown exists ──────────────────
Error: expect(locator).toBeVisible() failed
Locator: getByTestId('toolbar-merge-cell-select')
Expected: visible
Timeout: 10000ms
Error: element(s) not found
Call log:
- Expect "toBeVisible" with timeout 10000ms
- waiting for getByTestId('toolbar-merge-cell-select')
27 | // Check if merge cell select exists
28 | const mergeSelect = page.getByTestId('toolbar-merge-cell-select');
> 29 | await expect(mergeSelect).toBeVisible();
| ^
30 | });
31 |
32 | test('unmerge cells', async ({ page }) => {
at /home/runner/work/excel/excel/e2e/merge-cells.test.ts:29:29
|
|
E2E tests:
e2e/merge-cells.test.ts#L14
8) [firefox] › e2e/merge-cells.test.ts:4:5 › merge cells button is clickable ─────────────────────
Error: expect(locator).toBeVisible() failed
Locator: getByTestId('toolbar-merge-cell')
Expected: visible
Timeout: 10000ms
Error: element(s) not found
Call log:
- Expect "toBeVisible" with timeout 10000ms
- waiting for getByTestId('toolbar-merge-cell')
12 | // Click merge cells button
13 | const mergeButton = page.getByTestId('toolbar-merge-cell');
> 14 | await expect(mergeButton).toBeVisible();
| ^
15 | await mergeButton.click();
16 | await sleep(50);
17 |
at /home/runner/work/excel/excel/e2e/merge-cells.test.ts:14:29
|
|
E2E tests:
e2e/input.test.ts#L11
7) [firefox] › e2e/input.test.ts:4:5 › input text ────────────────────────────────────────────────
Error: expect(locator).toHaveValue(expected) failed
Locator: getByTestId('formula-bar-name-input')
Expected: "A1"
Timeout: 10000ms
Error: element(s) not found
Call log:
- Expect "toHaveValue" with timeout 10000ms
- waiting for getByTestId('formula-bar-name-input')
9 | await page.keyboard.press('ArrowUp');
10 |
> 11 | await expect(page.getByTestId('formula-bar-name-input')).toHaveValue('A1');
| ^
12 | await expect(page.getByTestId('formula-editor-trigger')).toHaveText(
13 | 'Hello World',
14 | );
at /home/runner/work/excel/excel/e2e/input.test.ts:11:60
|
|
E2E tests:
e2e/formatting.test.ts#L73
6) [firefox] › e2e/formatting.test.ts:66:5 › toggle wrap text ────────────────────────────────────
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
- waiting for getByTestId('toolbar-wrap-text')
71 |
72 | // Toggle wrap text
> 73 | await page.getByTestId('toolbar-wrap-text').click();
| ^
74 | await sleep(50);
75 |
76 | await expect(page.getByTestId('formula-bar-name-input')).toHaveValue('A1');
at /home/runner/work/excel/excel/e2e/formatting.test.ts:73:47
|
|
E2E tests:
e2e/formatting.test.ts#L60
5) [firefox] › e2e/formatting.test.ts:53:5 › change font size ────────────────────────────────────
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
- waiting for getByTestId('toolbar-font-size')
58 |
59 | // Click font size dropdown
> 60 | await page.getByTestId('toolbar-font-size').click();
| ^
61 | await sleep(100);
62 |
63 | await expect(page.getByTestId('formula-bar-name-input')).toHaveValue('A1');
at /home/runner/work/excel/excel/e2e/formatting.test.ts:60:47
|
|
E2E tests:
e2e/formatting.test.ts#L44
4) [firefox] › e2e/formatting.test.ts:37:5 › apply strike formatting ─────────────────────────────
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
- waiting for getByTestId('toolbar-strike')
42 |
43 | // Apply strike
> 44 | await page.getByTestId('toolbar-strike').click();
| ^
45 | await sleep(50);
46 |
47 | await expect(page.getByTestId('formula-bar-name-input')).toHaveValue('A1');
at /home/runner/work/excel/excel/e2e/formatting.test.ts:44:44
|
|
E2E tests:
e2e/formatting.test.ts#L28
3) [firefox] › e2e/formatting.test.ts:21:5 › apply italic formatting ─────────────────────────────
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
- waiting for getByTestId('toolbar-italic')
26 |
27 | // Apply italic
> 28 | await page.getByTestId('toolbar-italic').click();
| ^
29 | await sleep(50);
30 |
31 | await expect(page.getByTestId('formula-bar-name-input')).toHaveValue('A1');
at /home/runner/work/excel/excel/e2e/formatting.test.ts:28:44
|
|
E2E tests:
e2e/formatting.test.ts#L11
2) [firefox] › e2e/formatting.test.ts:4:5 › apply bold formatting ────────────────────────────────
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
- waiting for getByTestId('toolbar-bold')
9 |
10 | // Apply bold
> 11 | await page.getByTestId('toolbar-bold').click();
| ^
12 | await sleep(50);
13 |
14 | // Verify cell is selected and has content
at /home/runner/work/excel/excel/e2e/formatting.test.ts:11:42
|
|
E2E tests:
e2e/context-menu.test.ts#L103
1) [firefox] › e2e/context-menu.test.ts:96:5 › context menu paste option ─────────────────────────
TimeoutError: locator.click: Timeout 10000ms exceeded.
Call log:
- waiting for getByTestId('toolbar-copy')
101 | await page.keyboard.press('Enter');
102 | await page.keyboard.press('ArrowUp');
> 103 | await page.getByTestId('toolbar-copy').click();
| ^
104 | await sleep(100);
105 |
106 | // Move to another cell
at /home/runner/work/excel/excel/e2e/context-menu.test.ts:103:42
|
|
Complete job
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/upload-artifact@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Lint:
scripts/jest.setup.js#L129
scripts/jest.setup.js:129:10: Catch parameter 'e' is caught but never used.
|
|
E2E tests
193 flaky
[firefox] › e2e/context-menu.test.ts:96:5 › context menu paste option ──────────────────────────
[firefox] › e2e/formatting.test.ts:4:5 › apply bold formatting ─────────────────────────────────
[firefox] › e2e/formatting.test.ts:21:5 › apply italic formatting ──────────────────────────────
[firefox] › e2e/formatting.test.ts:37:5 › apply strike formatting ──────────────────────────────
[firefox] › e2e/formatting.test.ts:53:5 › change font size ─────────────────────────────────────
[firefox] › e2e/formatting.test.ts:66:5 › toggle wrap text ─────────────────────────────────────
[firefox] › e2e/input.test.ts:4:5 › input text ─────────────────────────────────────────────────
[firefox] › e2e/merge-cells.test.ts:4:5 › merge cells button is clickable ──────────────────────
[firefox] › e2e/merge-cells.test.ts:24:5 › merge cell select dropdown exists ───────────────────
[firefox] › e2e/merge-cells.test.ts:32:5 › unmerge cells ───────────────────────────────────────
[firefox] › e2e/number-format.test.ts:4:5 › number format dropdown is visible ──────────────────
[firefox] › e2e/number-format.test.ts:16:5 › apply number format to cell with number ───────────
[firefox] › e2e/number-format.test.ts:36:5 › number format with percentage ─────────────────────
[firefox] › e2e/number-format.test.ts:52:5 › number format with currency ───────────────────────
[firefox] › e2e/number-format.test.ts:67:5 › number format with large number ───────────────────
[firefox] › e2e/performance.test.ts:12:5 › has title ───────────────────────────────────────────
[firefox] › e2e/scrolling.test.ts:4:5 › scrollbars are visible ─────────────────────────────────
[firefox] › e2e/scrolling.test.ts:16:5 › navigate far down and scrollbar updates ───────────────
[firefox] › e2e/scrolling.test.ts:35:5 › navigate far right and scrollbar updates ──────────────
[firefox] › e2e/sheets.test.ts:4:5 › add new sheet ─────────────────────────────────────────────
[firefox] › e2e/sheets.test.ts:16:5 › verify default sheet is present ──────────────────────────
[firefox] › e2e/sheets.test.ts:28:5 › input data and add new sheet ─────────────────────────────
[firefox] › e2e/sheets.test.ts:53:5 › switch between sheets maintains data ─────────────────────
[firefox] › e2e/sheets.test.ts:82:5 › sheet bar list is visible ────────────────────────────────
[firefox] › e2e/undo-redo.test.ts:144:7 › undo and redo › toolbar buttons disabled state ───────
[safari] › e2e/context-menu.test.ts:25:5 › context menu copy option ───────��
|
background
wait
wait-all
cancel
parallel
Loading