dbt-core icon indicating copy to clipboard operation
dbt-core copied to clipboard

Core 9775/respect sql header in tests

Open VersusFacit opened this issue 7 months ago • 1 comments

Addresses part of #9775

Problem

We're not respect sql header.

Solution

I'm not a core dev so you may have a better solution in mind. I used some free cycles to upcycle some community code and will also do a followon for dbt bigquery of this feature.

Data tests in general will need support that I don't right now have the expertise to do (we can pair up if you like!)

Checklist

  • [x] I have read the contributing guide and understand what's expected of me.
  • [x] I have run this code in development, and it appears to resolve the stated issue.
  • [x] This PR includes tests, or tests are not required or relevant for this PR.
  • [x] This PR has no interface changes (e.g., macros, CLI, logs, JSON artifacts, config files, adapter interface, etc.) or this PR has already received feedback and approval from Product or DX.
  • [x] This PR includes type annotations for new and modified functions.

VersusFacit avatar May 13 '25 03:05 VersusFacit

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 89.47%. Comparing base (f64a488) to head (1c46fe2). :warning: Report is 89 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11614      +/-   ##
==========================================
+ Coverage   88.81%   89.47%   +0.66%     
==========================================
  Files         197      199       +2     
  Lines       24790    28220    +3430     
==========================================
+ Hits        22016    25249    +3233     
- Misses       2774     2971     +197     
Flag Coverage Δ
integration 86.96% <100.00%> (+1.15%) :arrow_up:
unit 65.26% <33.33%> (+2.40%) :arrow_up:

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

Components Coverage Δ
Unit Tests 65.26% <33.33%> (+2.40%) :arrow_up:
Integration Tests 86.96% <100.00%> (+1.15%) :arrow_up:
:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar May 13 '25 03:05 codecov[bot]