Skip to content

Fork test: PR #1345 - Fix deterministic fingerprints for set-valued arguments#1353

Open
ghanse wants to merge 4 commits into
mainfrom
fork-test/pr-1345
Open

Fork test: PR #1345 - Fix deterministic fingerprints for set-valued arguments#1353
ghanse wants to merge 4 commits into
mainfrom
fork-test/pr-1345

Conversation

@ghanse

@ghanse ghanse commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Automated sync from fork PR for CI testing.

Original PR: #1345

All tests, including unit and integration tests run on this PR (they are skipped for fork PRs).

@ghanse
ghanse requested a review from a team as a code owner July 20, 2026 16:38
@ghanse ghanse added do-not-merge The PR is blocked, e.g. being worked on and not ready. Not using draft to be able to test changes. fork-test labels Jul 20, 2026
@ghanse
ghanse requested review from tombonfert and removed request for a team July 20, 2026 16:38
@ghanse ghanse added do-not-merge The PR is blocked, e.g. being worked on and not ready. Not using draft to be able to test changes. fork-test labels Jul 20, 2026
@codecov

codecov Bot commented Jul 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.76%. Comparing base (ab30bde) to head (dbbe685).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1353      +/-   ##
==========================================
+ Coverage   91.55%   92.76%   +1.20%     
==========================================
  Files         102      102              
  Lines       10429    10527      +98     
==========================================
+ Hits         9548     9765     +217     
+ Misses        881      762     -119     
Flag Coverage Δ
anomaly 53.91% <50.00%> (+33.00%) ⬆️
anomaly-serverless 53.92% <50.00%> (+32.99%) ⬆️
integration 50.78% <50.00%> (+0.46%) ⬆️
integration-serverless 50.99% <50.00%> (+0.23%) ⬆️
unit 58.97% <100.00%> (+0.13%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

✅ 807/807 passed, 14 flaky, 50 skipped, 7h1m12s total

Flaky tests:

  • 🤪 test_save_checks_for_each_column_idempotency (10.704s)
  • 🤪 test_save_to_legacy_lakebase_table_adds_versioning_columns (9.092s)
  • 🤪 test_profiler_workflow_save_to_lakebase (1m0.733s)
  • 🤪 test_load_from_legacy_lakebase_table_adds_versioning_columns (15.429s)
  • 🤪 test_save_and_load_checks_from_lakebase_without_rule_set_fingerprint (12.711s)
  • 🤪 test_save_and_load_checks_from_lakebase_with_rule_set_fingerprint (11.152s)
  • 🤪 test_load_checks_when_lakebase_table_does_not_exist (11.93s)
  • 🤪 test_save_overwrite_replaces_existing_records_with_different_fingerprint (14.339s)
  • 🤪 test_save_and_load_checks_from_lakebase_table_with_client_id (10.649s)
  • 🤪 test_save_idempotency_overwrite_mode (16.812s)
  • 🤪 test_save_and_load_checks_from_lakebase_table (9.014s)
  • 🤪 test_save_and_load_checks_from_lakebase_table_with_run_config (11.598s)
  • 🤪 test_save_append_then_overwrite_same_run_config (16.944s)
  • 🤪 test_save_append_accumulates_multiple_versions (11.078s)

Running from acceptance #5247

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

✅ 194/194 passed, 2 skipped, 7h52m28s total

Running from anomaly #1361

Add equivalence-to-plain-set and negative-case assertions to guard
against the custom subclass masking behavior and the sort collapsing
distinct inputs.

Co-authored-by: Isaac
@github-actions

Copy link
Copy Markdown
Contributor

❌ 1 failed, 21s total

❌ test_mcp_server_end_to_end: AssertionError: MCP app deploy failed: (21.07s)
AssertionError: MCP app deploy failed:
STDOUT(tail):
runner_service_principal_id unset; TEST_SCHEMAing run_as to the deploying identity: 3fe685a1-96cc-4fec-8cdb-6944f5c9787e
::group::ensure artifacts volume (dqx.dqx_mcp_tmp.dqx_artifacts)
Ensuring dqx.dqx_mcp_tmp.dqx_artifacts exists (for runner wheels)...
Artifacts volume /Volumes/dqx/dqx_mcp_tmp/dqx_artifacts ready.
::endgroup::
::group::bundle deploy (mcp-dqx-it-04196f, target dev)


STDERR(tail):
Warning: The DATABRICKS_CLUSTER_ID variable is set but is ignored since the current target does not use 'mode: development'

Building dqx_mcp_runner_wheel...
Uploading .build/dqx_mcp_runner-0.1.0-py3-none-any.whl...
Uploading bundle files to /Workspace/Users/3fe685a1-96cc-4fec-8cdb-6944f5c9787e/.bundle/mcp-dqx/dev/files...
Deploying resources...
Error: cannot recreate resources.apps.mcp-dqx: deleting old id=mcp-dqx-it-7376c0: Cannot delete app mcp-dqx-it-7376c0 as it is not terminal with state STARTING, and was updated less than 35 minutes ago. Please wait before trying again. (400 BAD_REQUEST)

Endpoint: DELETE https://DATABRICKS_HOST/api/2.0/apps/mcp-dqx-it-7376c0?
HTTP Status: 400 Bad Request
API error_code: BAD_REQUEST
API message: Cannot delete app mcp-dqx-it-7376c0 as it is not terminal with state STARTING, and was updated less than 35 minutes ago. Please wait before trying again.

Error: cannot update resources.apps.mcp-dqx.permissions: dependency failed: resources.apps.mcp-dqx

Updating deployment state...
16:36 INFO [databricks.sdk] Using Databricks Metadata Service authentication
[gw0] linux -- Python 3.12.3 /home/runner/work/dqx/dqx/.venv/bin/python
16:36 INFO [databricks.sdk] Using Databricks Metadata Service authentication
16:36 INFO [databricks.sdk] Using Databricks Metadata Service authentication
[gw0] linux -- Python 3.12.3 /home/runner/work/dqx/dqx/.venv/bin/python

Running from mcp #68

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge The PR is blocked, e.g. being worked on and not ready. Not using draft to be able to test changes. fork-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants