Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix/sync test scenarios #59

Merged
merged 14 commits into from
Nov 13, 2024
Merged

Fix/sync test scenarios #59

merged 14 commits into from
Nov 13, 2024

Conversation

kaiden-araki
Copy link
Collaborator

Quick fix for synchronizer and generatepool2tx test cases

  • Main cause for test error was new user generation logic
  • I've added token_id for synchronizer test compatibility

TODO

Still need to update the TIL main logic, especially for users and accounts
Still need to remove token_id field completely while ensuring test scenario

Intizar-T and others added 14 commits November 8, 2024 03:29
@Intizar-T
merge feature/go-sequencer
f1c185b
@Intizar-T
merge smartcontract
dd30010
@Intizar-T
remove sequencer directory (js files)
0520d89
@Intizar-T
rename go_sequencer to sequencer
f5f3705
@Intizar-T
implement sequencer all tests workflow
abf1465
@Intizar-T
Merge pull request #45 from tokamak-network/feat/intizar/sequencer-al…
6b193d2 
…l-tests-workflow

implement sequencer all tests workflow
@Intizar-T @Shailu-s
update readme (#49)
b9ce2f5 
Co-authored-by: Shailendra Singh <[email protected]>
@Intizar-T
restore statedb (#54)
d36929b
@kaiden-araki @Shailu-s
Feature/go synchronizer (#53)
549d31f 
Co-authored-by: Shailu-s <[email protected]>
Co-authored-by: Shailendra Singh <[email protected]>
@kaiden-araki
fix/fix til and txprocessor for sync test scenario
031d697
@kaiden-araki
fix/comment out token_id from db
905f2cb
@kaiden-araki
fix/add todo mark to amount in l2tx struct
3d5bd1b
@kaiden-araki
fix/correct create new account logic and generate pooll2txs test scen…
ea2c545 
…ario
@kaiden-araki
fix/resolve conflicts with develop branch
ef9b134
Intizar-T
Intizar-T approved these changes Nov 13, 2024
View reviewed changes
Copy link
Collaborator

@Intizar-T Intizar-T left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@kaiden-araki kaiden-araki merged commit d3958c4 into develop Nov 13, 2024
@kaiden-araki kaiden-araki deleted the fix/sync_test_scenarios branch November 13, 2024 07:42
@Intizar-T Intizar-T restored the fix/sync_test_scenarios branch November 13, 2024 08:19
Shailu-s added a commit that referenced this pull request Nov 13, 2024
@kaiden-araki @Intizar-T @Shailu-s
Fix/sync test scenarios (#59)
2e642e3 
Co-authored-by: Intizar <[email protected]>
Co-authored-by: Shailendra Singh <[email protected]>
Co-authored-by: Shailu-s <[email protected]>
Shailu-s added a commit that referenced this pull request Nov 14, 2024
@kaiden-araki @Intizar-T @Shailu-s
Fix/sync test scenarios (#59)
a79f1b5 
Co-authored-by: Intizar <[email protected]>
Co-authored-by: Shailendra Singh <[email protected]>
Co-authored-by: Shailu-s <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Intizar-T Intizar-T Intizar-T approved these changes

@Shailu-s Shailu-s Awaiting requested review from Shailu-s

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kaiden-araki @Intizar-T