Improved dataset validation

[galvana]

Closes []

Description Of Changes

Write some things here about the changes and any potential caveats

Code Changes

• list your code changes here

Steps to Confirm

1. list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

• Issue requirements met
• All CI pipelines succeeded
• CHANGELOG.md updated
  • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
  • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
• Followup issues:
  • Followup issues created (include link)
  • No followup issues
• Database migrations:
  • Ensure that your downrev is up to date with the latest revision on main
  • Ensure that your downgrade() migration is correct and works
    • If a downgrade migration is not possible for this change, please call this out in the PR description!
  • No migrations
• Documentation:
  • Documentation complete, PR opened in fidesdocs
  • Documentation issue created in fidesdocs
  • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
  • No documentation updates required

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	⬜️ Ignored (Inspect)	Visit Preview		Feb 13, 2025 2:33am

[cypress]

fides    Run #12388

Run Properties:   Passed #12388  •  777c8e04d5 ℹ️: Merge 39b132452997656ccc236077046a3491cb8ae68c into 410758265edbec57d7e2389c3907...

Project	fides
Branch Review	refs/pull/5744/merge
Run status	Passed #12388
Run duration	00m 48s
Commit	777c8e04d5 ℹ️: Merge 39b132452997656ccc236077046a3491cb8ae68c into 410758265edbec57d7e2389c3907...
Committer	Adrian Galvan
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	0
Skipped	0
Passing	5
⚠️ You've recorded test results over your free plan limit. Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

[codecov]

Codecov Report

Attention: Patch coverage is 86.58537% with 44 lines in your changes missing coverage. Please review.

Project coverage is 87.22%. Comparing base (29f3415) to head (49d3053).
Report is 13 commits behind head on main.

Files with missing lines	Patch %	Lines
src/fides/service/dataset/dataset_service.py	74.80%	31 Missing and 1 partial ⚠️
...rc/fides/api/api/v1/endpoints/dataset_endpoints.py	82.75%	5 Missing ⚠️
.../service/dataset/validation_steps/data_category.py	92.30%	2 Missing and 1 partial ⚠️
...ides/service/dataset/validation_steps/traversal.py	87.50%	2 Missing and 1 partial ⚠️
src/fides/api/models/datasetconfig.py	96.15%	1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (86.58%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5744      +/-   ##
==========================================
+ Coverage   86.88%   87.22%   +0.33%     
==========================================
  Files         389      395       +6     
  Lines       24227    24361     +134     
  Branches     2624     2634      +10     
==========================================
+ Hits        21049    21248     +199     
+ Misses       2611     2539      -72     
- Partials      567      574       +7     

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