Fix state:modified check for exports

[aliceliu]

Resolves #10138

Problem

Saved queries always show up as modified in CI because of the check old_export.config.schema_name == new_export.config.schema_name (old schema is prod schema and new schema is CI schema).

Solution

Add unrendered_config to exports and compare the unrendered config instead of the config

Checklist

• I have read the contributing guide and understand what's expected of me.
• I have run this code in development, and it appears to resolve the stated issue.
• This PR includes tests, or tests are not required or relevant for this PR.
• 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.
• This PR includes type annotations for new and modified functions.

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[aliceliu]

• Fix state:modified check for exports #10565 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @aliceliu and the rest of your teammates on Graphite

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.86%. Comparing base (bba020f) to head (a95fa4f).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #10565      +/-   ##
==========================================
- Coverage   88.91%   88.86%   -0.05%     
==========================================
  Files         180      180              
  Lines       22755    22760       +5     
==========================================
- Hits        20232    20226       -6     
- Misses       2523     2534      +11     

Flag	Coverage Δ
integration	86.10% <100.00%> (-0.12%)	⬇️
unit	62.33% <16.66%> (-0.01%)	⬇️

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

Components	Coverage Δ
Unit Tests	62.33% <16.66%> (-0.01%)	⬇️
Integration Tests	86.10% <100.00%> (-0.12%)	⬇️

[gshank]

I think the changes are fine, but we need an update to the schemas: https://github.com/dbt-labs/dbt-core/blob/main/core/dbt/artifacts/README.md#updating-schemasgetdbtcom. I'll approve this but it would be best to get the schema updated fairly soon.

To skip the failure in checking schemas, you add the "artifact_minor_update" label.

[aliceliu]

dbt-labs/schemas.getdbt.com#50 is the PR for the schema change

[aliceliu]

The check is out of date, the newest one show 100% coverage https://github.com/dbt-labs/dbt-core/pull/10565/checks?check_run_id=29180771199

[aliceliu]

Same here