Skip to content

Fork test: PR #1275 - Guard LLM-generated sql_query rules#1283

Closed
mwojtyczka wants to merge 7 commits into
mainfrom
fork-test/pr-1275
Closed

Fork test: PR #1275 - Guard LLM-generated sql_query rules#1283
mwojtyczka wants to merge 7 commits into
mainfrom
fork-test/pr-1275

Conversation

@mwojtyczka

Copy link
Copy Markdown
Contributor

Automated sync from fork PR for CI testing.

Original PR: #1275

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

@mwojtyczka
mwojtyczka requested a review from a team as a code owner June 29, 2026 12:17
@mwojtyczka mwojtyczka 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 Jun 29, 2026
@mwojtyczka
mwojtyczka requested review from gergo-databricks and removed request for a team June 29, 2026 12:17
@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.32%. Comparing base (c98da6c) to head (3ba9bfc).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1283      +/-   ##
==========================================
- Coverage   92.53%   91.32%   -1.21%     
==========================================
  Files         102      102              
  Lines       10075    10102      +27     
==========================================
- Hits         9323     9226      -97     
- Misses        752      876     +124     
Flag Coverage Δ
anomaly 54.24% <6.89%> (-0.13%) ⬇️
anomaly-serverless 54.25% <6.89%> (-0.13%) ⬇️
integration 49.24% <62.06%> (-1.28%) ⬇️
integration-serverless 49.40% <62.06%> (-1.23%) ⬇️
unit 57.35% <100.00%> (+0.33%) ⬆️

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 Jun 29, 2026

Copy link
Copy Markdown
Contributor

✅ 774/774 passed, 17 flaky, 49 skipped, 5h36m24s total

Flaky tests:

  • 🤪 test_save_and_load_checks_from_lakebase_table_with_client_id (9.737s)
  • 🤪 test_load_checks_when_lakebase_table_does_not_exist (11.197s)
  • 🤪 test_save_and_load_checks_from_lakebase_table (11.321s)
  • 🤪 test_save_and_load_checks_from_lakebase_table_with_run_config (9.642s)
  • 🤪 test_save_and_load_checks_from_lakebase_table_with_output_modes (7.976s)
  • 🤪 test_profiler_workflow_save_to_lakebase (28.741s)
  • 🤪 test_save_and_load_checks_from_lakebase_table_with_user_installation (10.868s)
  • 🤪 test_save_checks_for_each_column_idempotency (9.098s)
  • 🤪 test_save_and_load_checks_from_lakebase_without_rule_set_fingerprint (14.573s)
  • 🤪 test_save_to_legacy_lakebase_table_adds_versioning_columns (8.519s)
  • 🤪 test_save_and_load_checks_from_lakebase_with_rule_set_fingerprint (14.2s)
  • 🤪 test_load_from_legacy_lakebase_table_adds_versioning_columns (12.18s)
  • 🤪 test_save_and_load_checks_from_lakebase_rule_set_fingerprint_already_exists (8.237s)
  • 🤪 test_save_overwrite_replaces_existing_records_with_different_fingerprint (8.849s)
  • 🤪 test_save_append_then_overwrite_same_run_config (11.26s)
  • 🤪 test_save_idempotency_overwrite_mode (9.698s)
  • 🤪 test_save_append_accumulates_multiple_versions (9.955s)

Running from acceptance #4998

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

✅ 194/194 passed, 2 skipped, 6h44m40s total

Running from anomaly #1112

@mwojtyczka mwojtyczka changed the title Fork test: PR #1275 - feat(llm): guard LLM-generated sql_query rules with is_sql_query_safe Fork test: PR #1275 - Guard LLM-generated sql_query rules Jun 29, 2026
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.

2 participants