Skip to content

fixes tests for v2 #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Feb 20, 2025
Merged

Conversation

SwenSchaeferjohann
Copy link

Overview

  • Summary of changes

Testing

  • Testing performed to validate the changes
  • snapshotter not tested

@SwenSchaeferjohann SwenSchaeferjohann changed the base branch from main to sergey/input-context February 20, 2025 16:21
Update src/dao/generated/accounts.rs

Co-authored-by: ananas-block <58553958+ananas-block@users.noreply.github.com>

Update src/ingester/parser/batch_event_parser.rs

Co-authored-by: ananas-block <58553958+ananas-block@users.noreply.github.com>

refactor: remove getSubtrees method and related API documentation

Update src/ingester/parser/mod.rs

Co-authored-by: ananas-block <58553958+ananas-block@users.noreply.github.com>

Refactor struct fields to use camelCase naming convention

replace the `calculate_two_inputs_hash_chain` function with the `create_two_inputs_hash_chain` method from the `light_compressed_account` crate.

Refactor error handling in `parse_public_transaction_event_v2`

format

Remove obsolete and commented-out account update code

Move `node_index_to_leaf_index` function to the appropriate location and remove dead code

Add comments to clarify nullifier field usage

Add comments to clarify tx_hash field usage in account struct

Add comments to clarify seq field usage in account struct

Add comments to clarify nullifier_queue_index field usage in account struct

Refactor get_compressed_accounts_by_owner module and add common utilities for account filtering

add validity proof v2

Add get_validity_proof_v2 and update API specifications

remove unnecessary logging

fix: mock tests

fix for getValidityProof (v1)

refactor: remove unused tree height parameters and add getValidityProofV2 method

fix: update method name from address_from_bytes to state_from_bytes in mod.rs consistency.

fix indexed_accounts query

fix get_compressed_account_by_owner v1

refactor: simplify account handling and improve code consistency in transaction processing

refactor: remove queue_position handling and update related queries and indexes

upd .gitignore

cargo fmt and fixed tests
@sergeytimoshin sergeytimoshin merged commit 7a7984c into sergey/input-context Feb 20, 2025
@sergeytimoshin sergeytimoshin deleted the swen/fmt-fix-tests branch February 20, 2025 17:18
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.

2 participants