blockblaz / blockblaz/zig-poseidon

Update test vectors to match current implementation

Open
#8 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Zig
Stars
19
Forks
5
PR merge metrics
No merged PRs in 30d

Description

## Description

The current test vectors need to be updated to match the current implementation. There's a commented out test case that needs to be updated with correct expected values.

## Current TODO Comment

```zig
// TODO: Update test vectors to match current implementation
// .{
// .input_state = [_]F.FieldElem{42} ** 16,
// .output_state = .{ 834546835, 1886829340, 1792314086, 1487871337, 567666274, 1133976664, 445360408, 630502830, 161668903, 153566288, 448274346, 619034796, 1156499614, 1851146900, 777523375, 393617892 },
// },
```

## Tasks

- [ ] Run the current implementation with the input state `[42] ** 16`
- [ ] Capture the actual output state
- [ ] Update the test vector with the correct expected output
- [ ] Verify the test passes with the updated values
- [ ] Ensure all other test vectors are still valid

## Priority

Medium - This affects test coverage and validation of the implementation.

## Labels

- enhancement
- testing
- poseidon2

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.