Skip to content

feat: fe test coverage#7787

Merged
speaker-ender merged 1 commit intomainfrom
feat/fe-test-coverage
Mar 31, 2026
Merged

feat: fe test coverage#7787
speaker-ender merged 1 commit intomainfrom
feat/fe-test-coverage

Conversation

@speaker-ender
Copy link
Copy Markdown
Contributor

@speaker-ender speaker-ender commented Mar 30, 2026

Ticket []

Description Of Changes

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
Copy link
Copy Markdown
Contributor

vercel bot commented Mar 30, 2026 •
edited
Loading

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

Project Deployment Actions Updated (UTC)
fides-plus-nightly Ready Ready Preview, Comment Mar 31, 2026 2:48pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
fides-privacy-center Ignored Ignored Mar 31, 2026 2:48pm

Request Review

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 30, 2026 •
edited
Loading

Dependency Review

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

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 950829f.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

  • .github/workflows/frontend_checks.yml

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 30, 2026 •
edited
Loading

Title Lines Statements Branches Functions
admin-ui Coverage: 7%
 5.66% (2385/42079) 4.6% (1094/23761) 3.81% (477/12512)
fides-js Coverage: 78%
 78.98% (1962/2484) 65.55% (1214/1852) 72.57% (336/463)
privacy-center Coverage: 86%
 83.18% (287/345) 77.15% (152/197) 75% (48/64)

@speaker-ender speaker-ender changed the title feat/fe-test-coverage feat: fe test coverage Mar 30, 2026
@speaker-ender speaker-ender marked this pull request as ready for review March 31, 2026 14:42
@speaker-ender speaker-ender requested a review from a team as a code owner March 31, 2026 14:42
@speaker-ender speaker-ender requested review from lucanovera and removed request for a team March 31, 2026 14:42
@speaker-ender speaker-ender force-pushed the feat/fe-test-coverage branch from a84c815 to 848aab4 Compare March 31, 2026 14:42
@speaker-ender
feat/fe-test-coverage
950829f 
fix: missing artifacts

refactor: artifact uploads

fix: paths

refactor: turbo dep

fix: use correct script

fix: paths again

fix: include hidden files

refactor: attempt single directory

refactor: more path attempts

test: check if works with file that should exist

test: using workspace variable in path

fix: more paths

fix: all the paths

chore: updating action versions and clean ups

build: updating node

chore: always the paths

test: path contents

fix: always the naming

fix: at least partial paths

refactor: correct formatting

debug: more of path

fix: missing reporter configs
@speaker-ender speaker-ender force-pushed the feat/fe-test-coverage branch from 848aab4 to 950829f Compare March 31, 2026 14:42
lucanovera
lucanovera approved these changes Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@lucanovera lucanovera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for adding the report! ship it

@speaker-ender speaker-ender added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit f24d001 Mar 31, 2026
48 of 49 checks passed
@speaker-ender speaker-ender deleted the feat/fe-test-coverage branch March 31, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucanovera lucanovera lucanovera approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@speaker-ender @lucanovera