Skip to content

test(ITL-513): verify iter_data after cross-session recompute; mark F7 resolved#219

Merged
eywalker merged 6 commits into
mainfrom
eywalker/itl-513-add_pipeline_record-skips-stale-entry-overwrite-due-to
Jul 9, 2026
Merged

test(ITL-513): verify iter_data after cross-session recompute; mark F7 resolved#219
eywalker merged 6 commits into
mainfrom
eywalker/itl-513-add_pipeline_record-skips-stale-entry-overwrite-due-to

Conversation

@kurodo3

@kurodo3 kurodo3 Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Marks DESIGN_ISSUES.md entry F7 as resolved (fixed by ITL-508 indexed entry-ID versioning).
  • Adds test_iter_data_serves_result_after_cross_session_recompute to TestEphemeralWritePath in test_ephemeral_result.py, satisfying ITL-513's explicit success criterion for an iter_data() coverage test.

No production code changes — ITL-508 already fixed the root cause.

The new test exercises the iter_data() DB hot-load path (_load_cached_entries_fetch_joined_records) after a cross-session ephemeral miss recompute, verifying:

  1. iter_data() serves the recomputed result without triggering further computation
  2. Repeated iter_data() calls are idempotent (no second recompute)

Closes ITL-513

@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@eywalker eywalker merged commit 7826b2b into main Jul 9, 2026
11 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.

1 participant