refactor(editor): Replace N8nPopover with N8nPopoverReka (and rename) for consistency and improved functionality across components #47802
Annotations
8 errors, 3 warnings, and 1 notice
|
Run Tests
Process completed with exit code 1.
|
|
Run Tests:
packages/testing/playwright/pages/SettingsUsersPage.ts#L70
3) [multi-main:ui] › tests/ui/settings/users/users.spec.ts:40:6 › Users Settings › should delete user and transfer their data
TimeoutError: locator.click: Timeout 20000ms exceeded.
Call log:
- waiting for getByTestId('action-delete').filter({ visible: true })
- locator resolved to <li tabindex="-1" role="menuitem" data-el-collection-item="" data-test-id="action-delete" class="el-dropdown-menu__item">…</li>
- attempting click action
2 × waiting for element to be visible, enabled and stable
- element is not stable
- retrying click action
- waiting 20ms
2 × waiting for element to be visible, enabled and stable
- element is not stable
- retrying click action
- waiting 100ms
- waiting for element to be visible, enabled and stable
- element is not stable
37 × retrying click action
- waiting 500ms
- waiting for element to be visible, enabled and stable
- element is not visible
- retrying click action
- waiting 500ms
at pages/SettingsUsersPage.ts:70
68 | async clickDeleteUser(email: string) {
69 | await this.openActions(email);
> 70 | await this.page.getByTestId('action-delete').filter({ visible: true }).click();
| ^
71 | }
72 | }
73 |
at SettingsUsersPage.clickDeleteUser (/home/runner/_work/n8n/n8n/packages/testing/playwright/pages/SettingsUsersPage.ts:70:74)
at /home/runner/_work/n8n/n8n/packages/testing/playwright/tests/ui/settings/users/users.spec.ts:49:3
|
|
Run Tests:
packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts#L34
2) [multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:26:6 › External Webhook Triggering › should surface workflow configuration errors to the caller
Retry #2 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toContain(expected) // indexOf
Expected substring: "Unused Respond to Webhook node"
Received string: "{\"code\":404,\"message\":\"The requested webhook \\\"GET test-webhook-UZxFifinAyNR\\\" is not registered.\",\"hint\":\"The workflow must be active for a production URL to run successfully. You can activate the workflow using the toggle in the top-right of the editor. Note that unlike test URL calls, production URL calls aren't shown on the canvas (only in the executions list)\",\"stacktrace\":\"ResponseError: The requested webhook \\\"GET test-webhook-UZxFifinAyNR\\\" is not registered.\\n at LiveWebhooks.findWebhook (/usr/local/lib/node_modules/n8n/src/webhooks/live-webhooks.ts:184:10)\\n at processTicksAndRejections (node:internal/process/task_queues:105:5)\\n at LiveWebhooks.executeWebhook (/usr/local/lib/node_modules/n8n/src/webhooks/live-webhooks.ts:83:19)\\n at WebhookRequestHandler.handleRequest (/usr/local/lib/node_modules/n8n/src/webhooks/webhook-request-handler.ts:66:21)\\n at /usr/local/lib/node_modules/n8n/src/webhooks/webhook-request-handler.ts:257:3\"}"
32 |
33 | expect(webhookResponse.ok()).toBe(false);
> 34 | expect(await webhookResponse.text()).toContain('Unused Respond to Webhook node');
| ^
35 | });
36 | });
37 |
at /home/runner/_work/n8n/n8n/packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts:34:40
|
|
Run Tests:
packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts#L34
2) [multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:26:6 › External Webhook Triggering › should surface workflow configuration errors to the caller
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toContain(expected) // indexOf
Expected substring: "Unused Respond to Webhook node"
Received string: "{\"code\":404,\"message\":\"The requested webhook \\\"GET test-webhook-cxRiXwVgvTrO\\\" is not registered.\",\"hint\":\"The workflow must be active for a production URL to run successfully. You can activate the workflow using the toggle in the top-right of the editor. Note that unlike test URL calls, production URL calls aren't shown on the canvas (only in the executions list)\",\"stacktrace\":\"ResponseError: The requested webhook \\\"GET test-webhook-cxRiXwVgvTrO\\\" is not registered.\\n at LiveWebhooks.findWebhook (/usr/local/lib/node_modules/n8n/src/webhooks/live-webhooks.ts:184:10)\\n at processTicksAndRejections (node:internal/process/task_queues:105:5)\\n at LiveWebhooks.executeWebhook (/usr/local/lib/node_modules/n8n/src/webhooks/live-webhooks.ts:83:19)\\n at WebhookRequestHandler.handleRequest (/usr/local/lib/node_modules/n8n/src/webhooks/webhook-request-handler.ts:66:21)\\n at /usr/local/lib/node_modules/n8n/src/webhooks/webhook-request-handler.ts:257:3\"}"
32 |
33 | expect(webhookResponse.ok()).toBe(false);
> 34 | expect(await webhookResponse.text()).toContain('Unused Respond to Webhook node');
| ^
35 | });
36 | });
37 |
at /home/runner/_work/n8n/n8n/packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts:34:40
|
|
Run Tests:
packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts#L34
2) [multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:26:6 › External Webhook Triggering › should surface workflow configuration errors to the caller
Error: expect(received).toContain(expected) // indexOf
Expected substring: "Unused Respond to Webhook node"
Received string: "{\"code\":404,\"message\":\"The requested webhook \\\"GET test-webhook-ofWpEZmp97Dn\\\" is not registered.\",\"hint\":\"The workflow must be active for a production URL to run successfully. You can activate the workflow using the toggle in the top-right of the editor. Note that unlike test URL calls, production URL calls aren't shown on the canvas (only in the executions list)\",\"stacktrace\":\"ResponseError: The requested webhook \\\"GET test-webhook-ofWpEZmp97Dn\\\" is not registered.\\n at LiveWebhooks.findWebhook (/usr/local/lib/node_modules/n8n/src/webhooks/live-webhooks.ts:184:10)\\n at processTicksAndRejections (node:internal/process/task_queues:105:5)\\n at LiveWebhooks.executeWebhook (/usr/local/lib/node_modules/n8n/src/webhooks/live-webhooks.ts:83:19)\\n at WebhookRequestHandler.handleRequest (/usr/local/lib/node_modules/n8n/src/webhooks/webhook-request-handler.ts:66:21)\\n at /usr/local/lib/node_modules/n8n/src/webhooks/webhook-request-handler.ts:257:3\"}"
32 |
33 | expect(webhookResponse.ok()).toBe(false);
> 34 | expect(await webhookResponse.text()).toContain('Unused Respond to Webhook node');
| ^
35 | });
36 | });
37 |
at /home/runner/_work/n8n/n8n/packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts:34:40
|
|
Run Tests:
packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts#L17
1) [multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:4:6 › External Webhook Triggering › should create workflow via API, activate it, trigger webhook externally, and verify execution
Retry #2 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
Expected: true
Received: false
15 | });
16 |
> 17 | expect(webhookResponse.ok()).toBe(true);
| ^
18 |
19 | const execution = await api.workflows.waitForExecution(workflowId, 5000);
20 | expect(execution.status).toBe('success');
at /home/runner/_work/n8n/n8n/packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts:17:32
|
|
Run Tests:
packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts#L17
1) [multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:4:6 › External Webhook Triggering › should create workflow via API, activate it, trigger webhook externally, and verify execution
Retry #1 ───────────────────────────────────────────────────────────────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
Expected: true
Received: false
15 | });
16 |
> 17 | expect(webhookResponse.ok()).toBe(true);
| ^
18 |
19 | const execution = await api.workflows.waitForExecution(workflowId, 5000);
20 | expect(execution.status).toBe('success');
at /home/runner/_work/n8n/n8n/packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts:17:32
|
|
Run Tests:
packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts#L17
1) [multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:4:6 › External Webhook Triggering › should create workflow via API, activate it, trigger webhook externally, and verify execution
Error: expect(received).toBe(expected) // Object.is equality
Expected: true
Received: false
15 | });
16 |
> 17 | expect(webhookResponse.ok()).toBe(true);
| ^
18 |
19 | const execution = await api.workflows.waitForExecution(workflowId, 5000);
20 | expect(execution.status).toBe('success');
at /home/runner/_work/n8n/n8n/packages/testing/playwright/tests/ui/triggers/webhook-external.spec.ts:17:32
|
|
Post Setup Environment
Skipping sticky disk commit due to previous step failures
|
|
Post Setup Environment
- Step: unknown (failed)
|
|
Post Setup Environment
Found 2 failed/cancelled steps in previous workflow steps
|
|
Run Tests
2 failed
[multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:4:6 › External Webhook Triggering › should create workflow via API, activate it, trigger webhook externally, and verify execution
[multi-main:ui] › tests/ui/triggers/webhook-external.spec.ts:26:6 › External Webhook Triggering › should surface workflow configuration errors to the caller
1 flaky
[multi-main:ui] › tests/ui/settings/users/users.spec.ts:40:6 › Users Settings › should delete user and transfer their data
14 skipped
53 passed (4.8m)
|
Loading