Skip to content

Conversation

@calvarjorge
Copy link
Contributor

  • Run playground integration tests in the CI (playwright based)

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
@calvarjorge calvarjorge changed the base branch from jorge.calvar/add_integration_tests_locally to main February 6, 2026 13:03
Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
@calvarjorge calvarjorge changed the base branch from main to jorge.calvar/add_integration_tests_locally February 6, 2026 14:25
Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
@calvarjorge calvarjorge changed the base branch from jorge.calvar/add_integration_tests_locally to main February 6, 2026 16:41
Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
@calvarjorge calvarjorge changed the base branch from main to jorge.calvar/add_integration_tests_locally February 6, 2026 16:47
@calvarjorge calvarjorge changed the base branch from jorge.calvar/add_integration_tests_locally to main February 6, 2026 16:53
Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
@calvarjorge calvarjorge changed the base branch from main to jorge.calvar/add_integration_tests_locally February 6, 2026 17:09
Copy link
Member

@pkosiec pkosiec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👌

@MarioCadenas MarioCadenas merged commit 095c048 into jorge.calvar/add_integration_tests_locally Feb 9, 2026
5 checks passed
@MarioCadenas MarioCadenas deleted the jorge.calvar/add_dev_playground_integration_to_ci branch February 9, 2026 09:22
calvarjorge added a commit that referenced this pull request Feb 9, 2026
* test: add Playwright integration tests for dev-playground

Add comprehensive frontend integration tests using Playwright to verify:
- Page navigation and routing
- Chart and data visualization rendering
- SSE streaming and reconnection
- Telemetry and SQL helpers functionality
- Console error detection

Tests use mocked API responses to verify UI behavior without backend dependencies.
Includes test utilities, mock data fixtures, and Playwright configuration.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* test: apply PR feedback - remove unnecessary tests and reorganize utils

- Remove navigation.spec.ts (tests router, not AppKit)
- Remove 3 tests from smoke.spec.ts (test Dev Playground, not AppKit)
- Move test utilities from fixtures/ to utils/

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* refactor: simplify test navigation by using goto waitUntil option

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* test: update playwright version

* ci: add playground integration tests job (#88)

* ci: add playground integration tests job

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* chore: trigger CI

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* ci: skip ServiceContext initialization in CI

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* feat: allow passing a WorkspaceClient to createApp for e2e testing

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* chore: trigger CI

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* refactor: move e2e env vars from playwright config to CI workflow

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* docs: regenerate API docs for createApp client param

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

---------

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>

* chore: update CONTRIBUTING.md and playwright config to use DATABRICKS_APP_PORT

---------

Signed-off-by: Jorge Calvar <jorge.calvar@databricks.com>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants