Skip to content

[test][node] Cover join stage-1 with no known right neighbor - #141

Open
TheP2P (thep2p) wants to merge 1 commit into
thep2p/90-join-stage1-one-node-graph-testfrom
thep2p/90-join-stage1-no-right-neighbor-test
Open

TheP2P (thep2p) wants to merge 1 commit into
thep2p/90-join-stage1-one-node-graph-testfrom
thep2p/90-join-stage1-no-right-neighbor-test

Conversation

@thep2p

Copy link
Copy Markdown
Collaborator

Summary

  • Add test_join_stage1_link_level0_no_right_neighbor_sends_single_link_request: when s's neighbor query resolves z as None, join_stage1_link_level0 never sends a second GetLinkOp and resolves once the single s-side request is applied, leaving the right-side table entry unset.
  • Required acceptance-criteria coverage for [Node] Implement: level-0 join linking (Phase 1) #90 (issue [Node] Implement: level-0 join linking (Phase 1) #90's third required test scenario). No production code change: the None-handling branch this exercises already shipped in the orchestration PR earlier in this stack.

Test plan

  • cargo fmt --check
  • cargo clippy --all-targets -- -D warnings
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test

@thep2p
TheP2P (thep2p) added this pull request to stack #143 September 14, 2026 20:23
@thep2p
TheP2P (thep2p) force-pushed the thep2p/90-join-stage1-no-right-neighbor-test branch from b3343b9 to 1ea3dff Compare September 15, 2026 17:27
@thep2p
TheP2P (thep2p) force-pushed the thep2p/90-join-stage1-no-right-neighbor-test branch from 1ea3dff to dd6863e Compare September 15, 2026 17:50
@thep2p
TheP2P (thep2p) force-pushed the thep2p/90-join-stage1-no-right-neighbor-test branch from dd6863e to 98cbcd6 Compare September 15, 2026 21:09
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