Skip to content

fix: support DQ expectations without quarantine targets - #487

Merged
ravi-databricks merged 2 commits into
v0.1.1from
issue_485
Sep 25, 2026
Merged

ravi-databricks merged 2 commits into
v0.1.1from
issue_485

Conversation

@ravi-databricks

@ravi-databricks ravi-databricks commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Allow Bronze and Silver DQ rules without requiring quarantine metadata.
  • Preserve optional quarantine metadata when rules are absent.
  • Warn when expect_or_quarantine has an incomplete target.
  • Maintain compatibility with legacy onboarding files.
  • Add JSON/YAML demo, unit, and integration coverage.
  • Document quarantine behavior and limitations.

Validation

  • FEVM backward compatibility test passed: v0.1.1 → current worktree.
  • Added unit and integration assertions for both JSON and YAML configurations.
  • Verified legacy quarantine-only configurations continue onboarding unchanged.

@ravi-databricks ravi-databricks added this to the v0.1.1 milestone Sep 25, 2026
@ravi-databricks ravi-databricks self-assigned this Sep 25, 2026
@ravi-databricks ravi-databricks added the bug Something isn't working label Sep 25, 2026
@codecov

codecov Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.24%. Comparing base (6cf7255) to head (b1fc569).
⚠️ Report is 6 commits behind head on v0.1.1.

Additional details and impacted files
@@            Coverage Diff             @@
##           v0.1.1     #487      +/-   ##
==========================================
+ Coverage   89.17%   89.24%   +0.06%     
==========================================
  Files          18       18              
  Lines        5009     5030      +21     
  Branches     1024     1029       +5     
==========================================
+ Hits         4467     4489      +22     
  Misses        313      313              
+ Partials      229      228       -1     
Flag Coverage Δ
unittests 89.24% <100.00%> (+0.06%) ⬆️

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.

@brij-raghuwanshi-db brij-raghuwanshi-db left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved.

@ravi-databricks
ravi-databricks merged commit 53852ae into v0.1.1 Sep 25, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants