LJ-369 Moved ExecutionLogs into its own file, added tests

[JadeCara]

Description Of Changes

Doing some clean up and adding some unit tests for elements in src/fides/api/models/privacy_requests.py. This file has gotten really large so moving this into a directory and breaking out well defined elements into their own files. This will also make adding unit tests less cumbersome.

This PR splits out Execution Log specific elements into their own file. I have also added several unit tests for each class/function.

Previous PR:

• consent.py: Consent, ConsentRequest LJ-369 Moved Consent/Consent request into their own file, added tests #5900

Current PR:

• execution_logs.py - ExecutionLogs, EXECUTION_CHECKPOINTS, COMPLETED_EXECUTION_LOG_STATUSES, EXITED_EXECUTION_LOG_STATUSES, can_run_checkpoint

Future PRs in the pipeline will further break out:

• provided_identity.py: ProvidedIdentityType, ProvidedIdentity LJ-369 Moved Provided Identity into its own file, added tests #5904
• request_task.py: TraversalDetails, RequestTask LJ-369 Moved RequestTask into its own file #5905
• webhook.py: CallbackType, SecondPartyRequestFormat, generate_request_callback_resume_jwe,
  generate_request_callback_pre_approval_jwe, generate_request_task_callback_jwe LJ-369 Moved CallbackType, SecondPartyRequestFormat to webhook.py #5907

Code Changes

• Added src/fides/api/models/privacy_requests/execution_logs.py and moved ExecutionLogs, EXECUTION_CHECKPOINTS, COMPLETED_EXECUTION_LOG_STATUSES, EXITED_EXECUTION_LOG_STATUSES, can_run_checkpoint to that file
• Added import statements to the __init__.py so that existing imports will continue to work.
• Added unit tests for ExecutionLogs class and can_run_checkpoint.

Steps to Confirm

1. There should be no change in functionality. All tests should pass.

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 ↗︎

Name	Status	Preview	Comments	Updated (UTC)
fides-privacy-center	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 19, 2025 11:50pm

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	⬜️ Ignored (Inspect)	Visit Preview		Mar 19, 2025 11:50pm

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.03%. Comparing base (6750aa8) to head (4a6ba1b).

Additional details and impacted files

@@                                            Coverage Diff                                             @@
##           LJ-369-be-update-privacy-requests-class-with-comments-and-attachments_pr1    #5903   +/-   ##
==========================================================================================================
  Coverage                                                                      87.02%   87.03%           
==========================================================================================================
  Files                                                                            410      411    +1     
  Lines                                                                          25417    25428   +11     
  Branches                                                                        2749     2749           
==========================================================================================================
+ Hits                                                                           22120    22131   +11     
  Misses                                                                          2704     2704           
  Partials                                                                         593      593           

☔ View full report in Codecov by Sentry.
📢 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.

[cypress]

fides    Run #12693

Run Properties:   Passed #12693  •  2341db0903: LJ-369 Moved ExecutionLogs into its own file, added tests (#5903)

Project	fides
Branch Review	main
Run status	Passed #12693
Run duration	00m 53s
Commit	2341db0903: LJ-369 Moved ExecutionLogs into its own file, added tests (#5903)
Committer	JadeWibbels
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	0
Skipped	0
Passing	5
View all changes introduced in this branch ↗︎