Skip to content

test: cover partitionValues_parsed in partitioned write tests#2467

Open
scottsand-db wants to merge 1 commit intodelta-io:mainfrom
scottsand-db:stack/partition-stats-cross-test
Open

test: cover partitionValues_parsed in partitioned write tests#2467
scottsand-db wants to merge 1 commit intodelta-io:mainfrom
scottsand-db:stack/partition-stats-cross-test

Conversation

@scottsand-db
Copy link
Copy Markdown
Collaborator

@scottsand-db scottsand-db commented Apr 25, 2026

What changes are proposed in this pull request?

Update test_write_partitioned_normal_values_roundtrip and test_write_partitioned_null_values_roundtrip in write_partitioned.rs to also cross-test writing parsed partition values (on/off)

How was this change tested?

cargo nextest run -p delta_kernel --test write_partitioned --all-features — 90 cases pass.

@scottsand-db scottsand-db changed the title test: cross-test partitioned write with partitionValues_parsed on/off test: cover partitionValues_parsed in partitioned write tests Apr 25, 2026
@scottsand-db scottsand-db force-pushed the stack/partition-stats-cross-test branch from f9020d2 to 94af5cb Compare April 25, 2026 22:43
@scottsand-db scottsand-db force-pushed the stack/partition-stats-cross-test branch from 94af5cb to 1b9f4b9 Compare April 25, 2026 22:48
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.40%. Comparing base (6486bd2) to head (1b9f4b9).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2467   +/-   ##
=======================================
  Coverage   88.40%   88.40%           
=======================================
  Files         178      178           
  Lines       58235    58235           
  Branches    58235    58235           
=======================================
  Hits        51485    51485           
  Misses       4783     4783           
  Partials     1967     1967           

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

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.

1 participant