Skip to content

Commit

Permalink
Fix bug in TestSendDisembargo/SendQueuedResultToCaller
Browse files Browse the repository at this point in the history
Sadly, I don't think this is capnproto#426, but I noticed it while hunting that
down: we set Which to receiverHosted but then set the SenderHosted
field. This patch fixes that.

I think we're getting lucky in the test such that it passes anyway
because we only ever put one export on the wire so the ID is always zero
anyway.
  • Loading branch information
zenhack committed Feb 4, 2023
1 parent e9a1ca6 commit 27aa4bd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions rpc/level1_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -255,8 +255,8 @@ func testSendDisembargo(t *testing.T, sendPrimeTo rpccp.Call_sendResultsTo_Which
Content: results.ToPtr(),
CapTable: []rpcCapDescriptor{
{
Which: rpccp.CapDescriptor_Which_receiverHosted,
SenderHosted: importID,
Which: rpccp.CapDescriptor_Which_receiverHosted,
ReceiverHosted: importID,
},
},
},
Expand Down

0 comments on commit 27aa4bd

Please sign in to comment.