Skip to content

Comments

Fix failure_cases index value for MultiIndex schema errors#2186

Merged
cosmicBboy merged 2 commits intounionai-oss:mainfrom
amerberg:failure_cases_index_levels
Dec 12, 2025
Merged

Fix failure_cases index value for MultiIndex schema errors#2186
cosmicBboy merged 2 commits intounionai-oss:mainfrom
amerberg:failure_cases_index_levels

Conversation

@amerberg
Copy link
Contributor

This fixes a regression in #2103 where the index column of failure_cases for schema errors raised in validating a multiindex level only includes the level that failed validation, rather than the tuple of all multi-index values. The fix implemented here is to validate the level as a series indexed by the full multi-index. A unit test is also added to prevent similar regressions in the future.

…s will have correct index values

Signed-off-by: Adam Merberg <adam.merberg@equilibriumenergy.com>
Signed-off-by: Adam Merberg <adam.merberg@equilibriumenergy.com>
@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.67%. Comparing base (ae2ae88) to head (7c18d93).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2186       +/-   ##
===========================================
+ Coverage   81.51%   93.67%   +12.16%     
===========================================
  Files         137      137               
  Lines       10895    10898        +3     
===========================================
+ Hits         8881    10209     +1328     
+ Misses       2014      689     -1325     

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

@cosmicBboy cosmicBboy merged commit f5c55cb into unionai-oss:main Dec 12, 2025
224 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants