Fix flaky test

[stephenkiers]

RE: https://github.com/ethyca/fides/actions/runs/19719977625/job/56500500100

Description Of Changes

This failed on me. I am attempting a fix.

Code Changes

Steps to Confirm

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
  • Updates unreleased work already in Changelog, no new entry necessary
• UX feedback:
  • All UX related changes have been reviewed by a designer
  • No UX review needed
• Followup issues:
  • Followup issues created
  • 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 GitHub.

2 Skipped Deployments

Project	Deployment	Preview	Comments	Updated (UTC)
fides-plus-nightly	Ignored			Nov 27, 2025 0:42am
fides-privacy-center	Ignored			Nov 27, 2025 0:42am

[greptile-apps]

Greptile Overview

Greptile Summary

Fixes a flaky Cypress test by adding an explicit wait for the complete task modal to fully close before attempting to open a second modal.

• Added cy.get('[data-testid="complete-task-modal"]').should("not.exist") assertion after closing first modal (line 442-443)
• Prevents race condition where the second modal click could occur before the first modal's closing animation completes
• Standard pattern for handling sequential modal interactions in Cypress tests

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• The change is a single-line test fix that addresses a clear race condition in a Cypress test. It uses standard Cypress best practices for sequential modal testing and doesn't modify any production code.
• No files require special attention

Important Files Changed

File Analysis

Filename	Score	Overview
clients/privacy-center/cypress/e2e/privacy-center/external-manual-tasks.cy.ts	5/5	Adds explicit wait for modal close before opening next modal to fix flaky test timing issue

[greptile-apps]

_{1 file reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}