Skip to content

refactor(tests): convert the medium-high risk slice-to-map table tests#323

Merged
mwbrooks merged 12 commits intomainfrom
mwbrooks-table-test-consistent-p6-medium-high-risk
Feb 4, 2026
Merged

refactor(tests): convert the medium-high risk slice-to-map table tests#323
mwbrooks merged 12 commits intomainfrom
mwbrooks-table-test-consistent-p6-medium-high-risk

Conversation

@mwbrooks
Copy link
Member

@mwbrooks mwbrooks commented Jan 31, 2026 •
edited
Loading

Summary

Related to #315, #316, #320, #321, #322

This pull request refactors the remaining medium-high risk Table Tests from the Slice-to-Map Pattern. It's focused on tests with prompt mocking and complex setups.

Reviewers

This one is getting a little more messy to read over, but hopefully it goes smooth! 🙇🏻

Requirements

@mwbrooks mwbrooks added this to the Next Release milestone Jan 31, 2026
@mwbrooks mwbrooks self-assigned this Jan 31, 2026
@mwbrooks mwbrooks added docs M-T: Documentation work only semver:patch Use on pull requests to describe the release version increment labels Jan 31, 2026
@codecov
Copy link

codecov bot commented Jan 31, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.66%. Comparing base (a76e0a5) to head (01df4fc).
⚠️ Report is 14 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #323      +/-   ##
==========================================
+ Coverage   64.63%   64.66%   +0.02%     
==========================================
  Files         212      212              
  Lines       17771    17771              
==========================================
+ Hits        11487    11492       +5     
+ Misses       5209     5204       -5     
  Partials     1075     1075

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

@mwbrooks mwbrooks mentioned this pull request Jan 31, 2026
Merged
2 tasks
Base automatically changed from mwbrooks-table-test-consistent-p5-medium-risk to main February 4, 2026 23:02
@mwbrooks mwbrooks marked this pull request as ready for review February 4, 2026 23:17
@mwbrooks mwbrooks requested a review from a team as a code owner February 4, 2026 23:17
zimeg
zimeg approved these changes Feb 4, 2026
View reviewed changes
Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

@mwbrooks Love to see the test runner become more consistent! 💌

@mwbrooks
Copy link
Member Author

mwbrooks commented Feb 4, 2026

Woo, thanks for the quick review @zimeg! 🙇🏻

@mwbrooks mwbrooks merged commit ca9db20 into main Feb 4, 2026
8 checks passed
@mwbrooks mwbrooks deleted the mwbrooks-table-test-consistent-p6-medium-high-risk branch February 4, 2026 23:32
@mwbrooks mwbrooks added code health M-T: Test improvements and anything that improves code health and removed docs M-T: Documentation work only labels Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zimeg zimeg zimeg approved these changes

Assignees

@mwbrooks mwbrooks

Labels

code health M-T: Test improvements and anything that improves code health semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Milestone

v3.13.0

Development

Successfully merging this pull request may close these issues.

2 participants

@mwbrooks @zimeg

Comments