Skip to content

Commit fb5e9a1

Browse files
committed
found rogue nullifier
1 parent a2310d9 commit fb5e9a1

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

yarn-project/end-to-end/src/bench/client_flows/transfers.test.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -163,14 +163,16 @@ describe('Transfer benchmark', () => {
163163

164164
/*
165165
* We should have created the following nullifiers:
166-
* - One per created note
167-
* - One for the transaction
166+
* - One per minted note
167+
* - One because the transaction is cancellable
168168
* - One for the private event commitment (note transfer for the recipient)
169-
* - One for the fee note, another one for the partial note validity commitment and an
169+
* - Private FPC: One for the fee note, another one for the partial note validity commitment and an
170170
* extra for the authwit invalidation if we're using a private fpc
171+
* - Any other payment method: kernel-injected non revertible nullifier due to abscence of nullifiers
172+
* during the setup phase of the tx
171173
*/
172174
expect(txEffects!.data.nullifiers.length).toBe(
173-
notesToCreate + 1 + 1 + (benchmarkingPaymentMethod === 'private_fpc' ? 3 : 0),
175+
notesToCreate + 1 + 1 + (benchmarkingPaymentMethod === 'private_fpc' ? 3 : 1),
174176
);
175177
/**
176178
* We should have created 4 new notes,

0 commit comments

Comments
 (0)