Skip to content

Fix: breaking SDK change - use asUuid in tests - #22443

Merged
eliykat merged 1 commit into
mainfrom
ac/pm-35785/add-strongly-typed-policy-data-handling
Aug 17, 2026
Merged

Fix: breaking SDK change - use asUuid in tests#22443
eliykat merged 1 commit into
mainfrom
ac/pm-35785/add-strongly-typed-policy-data-handling

Conversation

@eliykat

@eliykat eliykat commented Aug 15, 2026

Copy link
Copy Markdown
Member

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-35785

📔 Objective

Same as #22392, I just missed test code that also needed to be updated to use asUuid.

📸 Screenshots

@eliykat
eliykat requested a review from a team as a code owner August 15, 2026 03:05
@eliykat
eliykat requested a review from JimmyVo16 August 15, 2026 03:05
@eliykat eliykat added the t:tech-debt Change Type - Tech debt label Aug 15, 2026
@eliykat
eliykat enabled auto-merge (squash) August 15, 2026 03:06
@codecov

codecov Bot commented Aug 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.05%. Comparing base (8737819) to head (0e49dd2).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #22443   +/-   ##
=======================================
  Coverage   54.05%   54.05%           
=======================================
  Files        4308     4308           
  Lines      136885   136885           
  Branches    21627    21627           
=======================================
+ Hits        73989    73993    +4     
+ Misses      57504    57500    -4     
  Partials     5392     5392           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@eliykat
eliykat merged commit 42b09ca into main Aug 17, 2026
157 of 159 checks passed
@eliykat
eliykat deleted the ac/pm-35785/add-strongly-typed-policy-data-handling branch August 17, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:tech-debt Change Type - Tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants