Skip to content

feat(csharp): add E2E tests CI workflow#34

Merged
msrathore-db merged 12 commits intomainfrom
e2e-workflow
Nov 17, 2025
Merged

feat(csharp): add E2E tests CI workflow#34
msrathore-db merged 12 commits intomainfrom
e2e-workflow

Conversation

@msrathore-db
Copy link
Collaborator

@msrathore-db msrathore-db commented Nov 14, 2025

Summary

  • Add GitHub Actions workflow for E2E testing with Databricks
  • Uses the PECO-testing workspace
  • Uses direct token authentication
  • Only runs on PRs from the repo itself (not forks)
  • Test script filters for E2E tests

Changes

  • .github/workflows/e2e-tests.yml - CI workflow configuration
  • ci/scripts/csharp_test_databricks_e2e.sh - E2E test execution script

Test Plan

  • Verified workflow steps locally with test credentials
  • All E2E tests passed successfully
  • Build and test scripts execute correctly

Required Secrets

The workflow requires the following secrets to be configured in the GitHub repository:

  • DATABRICKS_HOST
  • TEST_PECO_WAREHOUSE_HTTP_PATH
  • DATABRICKS_TOKEN

🤖 Generated with Claude Code

Add GitHub Actions workflow for E2E testing with Databricks:
- New workflow file for running E2E tests on push/PR
- Uses direct token authentication (no OIDC)
- Only runs on PRs from the repo itself (not forks)
- Test script filters for E2E tests

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@msrathore-db msrathore-db marked this pull request as draft November 14, 2025 09:46
msrathore-db and others added 4 commits November 14, 2025 15:26
…Actions

Fixed authentication failures in GitHub Actions by explicitly passing
DATABRICKS_TEST_CONFIG_FILE to the test step. Also expanded test filter
to include all E2E test suites that require Databricks credentials.

Changes:
- Add explicit env block in test step to ensure DATABRICKS_TEST_CONFIG_FILE is passed
- Expand test filter to include StatementTests, ClientTests, DriverTests, and value tests

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
…onfig

The test framework expects JSON property names (token, auth_type, catalog)
which it converts to ADBC parameters internally. The workflow was using
ADBC parameter names directly (adbc.spark.access_token), causing authentication
failures. Updated to match the working local test config format.
@msrathore-db msrathore-db marked this pull request as ready for review November 17, 2025 08:31
@msrathore-db msrathore-db removed the request for review from eric-wang-1990 November 17, 2025 19:10
@msrathore-db msrathore-db merged commit 7b47adf into main Nov 17, 2025
10 checks passed
@msrathore-db msrathore-db deleted the e2e-workflow branch November 17, 2025 19:10
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.

2 participants