Skip to content

test: assert successful reconciliation for example projects#8815

Merged
ericpgreen2 merged 4 commits intomainfrom
e2e-assert-reconciliation
Feb 12, 2026
Merged

test: assert successful reconciliation for example projects#8815
ericpgreen2 merged 4 commits intomainfrom
e2e-assert-reconciliation

Conversation

@ericpgreen2
Copy link
Contributor

@ericpgreen2 ericpgreen2 commented Feb 11, 2026

  • Adds a waitForReconciliation utility that polls the runtime's ListResources API until all data resources are idle, then asserts no reconciliation errors
  • Extends the init E2E tests to call this after navigation, catching broken example projects (like the GitHub Analytics failure in APP-720)

Closes APP-720

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

Developed in collaboration with Claude Code

ericpgreen2 and others added 4 commits February 11, 2026 13:16
The init E2E tests only verified navigation to the correct file but never
waited for reconciliation or checked for errors. This meant broken example
projects (like the GitHub Analytics failure in APP-720) went undetected.

Add a `waitForReconciliation` utility that polls the runtime's ListResources
API until all data resources reach RECONCILE_STATUS_IDLE, then asserts none
have errors. The ProjectParser meta-resource is excluded since it stays
running while watching the repo.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ericpgreen2 ericpgreen2 merged commit 1dcf7b7 into main Feb 12, 2026
16 checks passed
@ericpgreen2 ericpgreen2 deleted the e2e-assert-reconciliation branch February 12, 2026 16:27
@ericpgreen2 ericpgreen2 mentioned this pull request Feb 12, 2026
8 tasks
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