Skip to content

Commit

Permalink
test fix
Browse files Browse the repository at this point in the history
  • Loading branch information
benesjan committed Sep 16, 2024
1 parent a5d9196 commit 303e651
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ export class L1NotePayload extends L1Payload {
remainingData.length,
);
if (publicValuesData.length % Fr.SIZE_IN_BYTES !== 0) {
throw new Error('Public values byte length is not a multiple of Fr size');
throw new Error(`Public values byte length is not a multiple of Fr size. Length: ${publicValuesData.length}`);
}
const publicValues = [];
for (let i = 0; i < publicValuesData.length; i += Fr.SIZE_IN_BYTES) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ describe('L1 Note Payload', () => {
);
});

it.only('encrypted tagged log of finalized partial note matches Noir', () => {
it('encrypted tagged log of finalized partial note matches Noir', () => {
const numPublicValues = 2;

// All the values in this test were arbitrarily set and copied over to `payload.nr`
Expand Down
2 changes: 2 additions & 0 deletions yarn-project/circuit-types/src/logs/l1_payload/tagged_log.ts
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,7 @@ export class TaggedLog<Payload extends L1NotePayload | L1EventPayload> {
if (
!(e instanceof NotOnCurveError) &&
!e.message.endsWith('is greater or equal to field modulus.') &&
!e.message.startsWith('Public values byte length is not a multiple of Fr size.') &&
!e.message.startsWith('Invalid AztecAddress length') &&
!e.message.startsWith('Selector must fit in') &&
!e.message.startsWith('Attempted to read beyond buffer length')
Expand Down Expand Up @@ -145,6 +146,7 @@ export class TaggedLog<Payload extends L1NotePayload | L1EventPayload> {
if (
!(e instanceof NotOnCurveError) &&
!e.message.endsWith('is greater or equal to field modulus.') &&
!e.message.startsWith('Public values byte length is not a multiple of Fr size.') &&
!e.message.startsWith('Invalid AztecAddress length') &&
!e.message.startsWith('Selector must fit in') &&
!e.message.startsWith('Attempted to read beyond buffer length')
Expand Down

0 comments on commit 303e651

Please sign in to comment.