Skip to content
This repository has been archived by the owner on Nov 30, 2022. It is now read-only.

Segment Masking Test Failures #375

Merged
merged 3 commits into from
Apr 8, 2022
Merged

Segment Masking Test Failures #375

merged 3 commits into from
Apr 8, 2022

Conversation

pattisdr
Copy link
Contributor

@pattisdr pattisdr commented Apr 7, 2022

Purpose

Segment masking failures - add a slight pause after creating data before running the access/gdpr delete request.

Changes

Checklist

  • Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram)
  • Good unit test/integration test coverage
  • This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
  • The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket

Fixes #

@pattisdr pattisdr added the run unsafe ci checks Triggers running of unsafe CI checks label Apr 7, 2022
@pattisdr pattisdr added run unsafe ci checks Triggers running of unsafe CI checks and removed run unsafe ci checks Triggers running of unsafe CI checks labels Apr 8, 2022
@pattisdr pattisdr added run unsafe ci checks Triggers running of unsafe CI checks and removed run unsafe ci checks Triggers running of unsafe CI checks labels Apr 8, 2022
@seanpreston seanpreston merged commit dc34a14 into main Apr 8, 2022
@seanpreston seanpreston deleted the segment_masking_failures branch April 8, 2022 12:56
sanders41 pushed a commit that referenced this pull request Sep 22, 2022
* Investigate segment masking test failures.

* Remove pause - if there are future test failures we may need to investigate again.

* Restore pause.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
run unsafe ci checks Triggers running of unsafe CI checks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants