Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Introduce manual SEA test scripts for Exec Phase#589

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
varun-edachali-dbx merged 37 commits intosea-migrationfromsea-test-scripts
Jun 27, 2025

Conversation

@varun-edachali-dbx
Copy link
Contributor

@varun-edachali-dbxvarun-edachali-dbx commentedJun 9, 2025•
edited
Loading

What type of PR is this?

  • Other - test scripts for eventual feature

Description

For the SEA implementation, before we introduce e2e tests, we introduce manual testing scripts which allow us to execute queries against a real warehouse as an implementation sanity check.

The tests are structured as follows:

sea_connector_test.py│├─ test_sea_session.py - tests basic session opening and closure │├─ test_sea_sync_query.py - tests synchronous query execution │  ├─ test_sea_sync_query_with_cloud_fetch()    [use_cloud_fetch=True]│  └─ test_sea_sync_query_without_cloud_fetch() [use_cloud_fetch=False]│├─ test_sea_async_query.py - tests async query execution │  ├─ test_sea_async_query_with_cloud_fetch()    [use_cloud_fetch=True]│  └─ test_sea_async_query_without_cloud_fetch() [use_cloud_fetch=False]│└─ test_sea_metadata.py - tests metadata commands

This allows us to experiment with metadata queries, session opening and closure, synchronous queries (INLINE as well asEXTERNAL_LINKS) and async queries (INLINE as well asEXTERNAL_LINKS).

How is this tested?

  • Unit tests
  • E2E Tests
  • Manually
  • N/A

Related Tickets & Documents

https://docs.google.com/document/d/1Y-eXLhNqqhrMVGnOlG8sdFrCxBTN1GdQvuKG4IfHmo0/edit?usp=sharing

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
@varun-edachali-dbxvarun-edachali-dbx changed the base branch frommain tosea-res-setJune 9, 2025 10:09
Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
@samikshya-db
Copy link
Contributor

Just to confirm : these tests will be replaced by the E2E tests in the future, right?

@varun-edachali-dbx
Copy link
ContributorAuthor

Just to confirm : these tests will be replaced by the E2E tests in the future, right?

Yes, that's right. These scripts allow me to test the functionality against an actual warehouse before the e2e tests, which we will introduce after implementing the functionality completely.

@jayantsing-db
Copy link
Contributor

hey, why are e2e tests failing on this PR?

Signed-off-by: varun-edachali-dbx <varun.edachali@databricks.com>
@varun-edachali-dbx
Copy link
ContributorAuthor

varun-edachali-dbx commentedJun 27, 2025•
edited
Loading

hey, why are e2e tests failing on this PR?

Likely flaky tests since this PR doesn't affect any code paths that the integration tests should be running. I'll let them run once again before merging.

Edit: the tests are passing

@varun-edachali-dbxvarun-edachali-dbx merged commit45585d4 intosea-migrationJun 27, 2025
23 checks passed
@varun-edachali-dbxvarun-edachali-dbx mentioned this pull requestJul 22, 2025
5 tasks
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@jayantsing-dbjayantsing-dbjayantsing-db approved these changes

@deeksha-dbdeeksha-dbAwaiting requested review from deeksha-db

@samikshya-dbsamikshya-dbAwaiting requested review from samikshya-db

@jprakash-dbjprakash-dbAwaiting requested review from jprakash-db

@jackyhu-dbjackyhu-dbAwaiting requested review from jackyhu-db

@madhav-dbmadhav-dbAwaiting requested review from madhav-db

@gopalldbgopalldbAwaiting requested review from gopalldb

@vikrantpuppalavikrantpuppalaAwaiting requested review from vikrantpuppala

@shivam2680shivam2680Awaiting requested review from shivam2680

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@varun-edachali-dbx@samikshya-db@jayantsing-db
[8]ページ先頭
©2009-2025 Movatter.jp