Skip to content

Fix test-only compile error from missing cover_percent field - #652

Merged
edwin-zvs merged 1 commit into
mainfrom
fix-program-view-memory-cover-percent
Jul 1, 2026
Merged

Fix test-only compile error from missing cover_percent field#652
edwin-zvs merged 1 commit into
mainfrom
fix-program-view-memory-cover-percent

Conversation

@edwin-zvs

@edwin-zvs edwin-zvs commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • main is currently red: cargo test fails to compile with error[E0063]: missing field cover_percentin initializer ofapp::ProgramViewMemory`` (introduced by Clean up Program state when a session is deleted #650, which left a test literal out of sync with the cover_percent field Add resizable roll-down Program view #651 added).
  • Adds the missing field to the test-only ProgramViewMemory literal in deleting_a_session_clears_its_program_state, using the same PROGRAM_COVER_PERCENT_DEFAULT the rest of the test suite uses.

Test plan

  • cargo test -p agentd-cli deleting_a_session_clears_its_program_state passes locally

@edwin-zvs
Fix test-only compile error from missing cover_percent field
ba050b1 
PR #650 added scroll_offset/preferred_col cleanup but left a
ProgramViewMemory test literal missing the cover_percent field added by
#651, breaking cargo test on main (E0063).
@edwin-zvs
edwin-zvs merged commit a16a38e into main Jul 1, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@edwin-zvs