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

Extend txsim to send all state machine messages #3509

Open
1 task
ninabarbakadze opened this issue May 28, 2024 · 0 comments
Open
1 task

Extend txsim to send all state machine messages #3509

ninabarbakadze opened this issue May 28, 2024 · 0 comments
Labels
WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc

Comments

@ninabarbakadze
Copy link
Member

ninabarbakadze commented May 28, 2024 •
edited
Loading

Description

Every time we add a new module we should add it to txsim.

This could take the form of a generic sequence that can work for any sdk.Msg or some equivalent.

Acceptance Criteria

  • Txsim should be able to submit all sdk.Msgs via a sequence
@ninabarbakadze ninabarbakadze added WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc needs:grooming items needs a status change, to be closed, or clearer AC. Item should be discussed next sync. needs:triage labels May 28, 2024
@evan-forbes evan-forbes removed needs:triage needs:grooming items needs a status change, to be closed, or clearer AC. Item should be discussed next sync. labels May 31, 2024
This was referenced Jun 7, 2024
Closed
Merged
ninabarbakadze added a commit that referenced this issue Jun 10, 2024
@ninabarbakadze @rootulp
test: consistent appHash between commits (#3513)
d851e3c 
## Overview

Fixes #2414 

### This test is split up in a few parts:

- Execute transactions deterministically on v1 and save the AppHash.
#3522 ([includes genesis
folder backport from
utils](#3520))
- Set up the same testApp environment on main, execute transactions and
assert that the AppHash matches the one from v1

### Verifying the AppHash: 

If you want to make sure that the expected AppHash is correct, you can
run the [non determinism test on
v1.x](#3522) that
generates it.

### Follow-ups:
#3509 
#3540

---------

Co-authored-by: Rootul P <[email protected]>
@ninabarbakadze ninabarbakadze mentioned this issue Jun 11, 2024
Closed
4 tasks
@staheri14 staheri14 mentioned this issue Jun 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
WS: Maintenance 🔧 includes bugs, refactors, flakes, and tech debt etc
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@evan-forbes @ninabarbakadze