Skip to content

Add Snowflake VARIANT JSON integration tests#8173

Merged
eastandwestwind merged 2 commits into
mainfrom
snowflake-variant-json-tests
May 15, 2026
Merged

Add Snowflake VARIANT JSON integration tests#8173
eastandwestwind merged 2 commits into
mainfrom
snowflake-variant-json-tests

Conversation

@eastandwestwind
Copy link
Copy Markdown
Contributor

Summary

  • Adds a new test fixture (snowflake_resources_with_json_variant) that inserts actual nested JSON into the Snowflake VARIANT column using parse_json() instead of to_variant() on a plain string
  • Adds test_access_request_snowflake_with_json_variant — verifies nested JSON objects and arrays are returned in the DSR access package
  • Adds test_erasure_request_snowflake_with_json_variant — verifies the VARIANT column containing nested JSON is nullified after an erasure request

Test plan

  • Run Snowflake integration tests: pytest tests/ops/service/privacy_request/test_snowflake_privacy_requests.py -m "integration_snowflake"

🤖 Generated with Claude Code

@eastandwestwind eastandwestwind requested a review from a team as a code owner May 13, 2026 15:36
@eastandwestwind eastandwestwind requested review from gilluminate and removed request for a team May 13, 2026 15:36
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
fides-plus-nightly Ignored Ignored Preview May 13, 2026 4:08pm
fides-privacy-center Ignored Ignored May 13, 2026 4:08pm

Request Review

@github-actions
Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
See the Details below.

Scanned Files

  • clients/fides-js/package.json
  • docs/fides/requirements.txt
  • qa/requirements.txt

Adds a fixture that inserts actual nested JSON (with objects and arrays)
into the VARIANT column via parse_json(), and two new tests verifying
that access requests return the full JSON structure and erasure requests
nullify it.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Title Lines Statements Branches Functions
admin-ui Coverage: 8%
6.71% (3082/45920) 6.06% (1604/26427) 4.66% (636/13639)
fides-js Coverage: 78%
79.17% (1977/2497) 66.25% (1249/1885) 73.31% (349/476)
privacy-center Coverage: 85%
82.53% (364/441) 79.74% (189/237) 74.07% (60/81)

@eastandwestwind eastandwestwind force-pushed the snowflake-variant-json-tests branch from 4d7c4cd to 3b9ba8a Compare May 13, 2026 15:46
@eastandwestwind eastandwestwind added the run unsafe ci checks Runs fides-related CI checks that require sensitive credentials label May 13, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.46%. Comparing base (2f9f5e5) to head (463ad90).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8173      +/-   ##
==========================================
- Coverage   85.47%   85.46%   -0.01%     
==========================================
  Files         650      650              
  Lines       42416    42416              
  Branches     4980     4980              
==========================================
- Hits        36253    36252       -1     
  Misses       5055     5055              
- Partials     1108     1109       +1     

☔ 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.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@eastandwestwind eastandwestwind added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit d64e6f1 May 15, 2026
69 of 70 checks passed
@eastandwestwind eastandwestwind deleted the snowflake-variant-json-tests branch May 15, 2026 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run unsafe ci checks Runs fides-related CI checks that require sensitive credentials

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants