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.

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

mempool tests fail on dev against new framework #1405

Open
fluidvanadium opened this issue Sep 18, 2024 · 2 comments
Open

mempool tests fail on dev against new framework #1405

fluidvanadium opened this issue Sep 18, 2024 · 2 comments
Assignees

Comments

@fluidvanadium
Copy link
Contributor

#1401 describes some of the details of the problem

experiment 02:
hypothesis: newer zcashd version caused slowdown

1726599570 || (HEAD detached at labs/dev)[ ] 51e3e1fc5
/home/cupress/ZINGO/libs/zingolib: 0
$cargo nextest run simpool_zero_value_change_orchard_to_sapling
    Finished `test` profile [optimized] target(s) in 0.79s
    Starting 1 test across 18 binaries (329 skipped)
        SLOW [> 60.000s] libtonode-tests::chain_generics chain_generics::simpool_zero_value_change_orchard_to_sapling
        SLOW [>120.000s] libtonode-tests::chain_generics chain_generics::simpool_zero_value_change_orchard_to_sapling
        PASS [ 134.465s] libtonode-tests::chain_generics chain_generics::simpool_zero_value_change_orchard_to_sapling
------------
     Summary [ 134.465s] 1 test run: 1 passed (1 slow), 329 skipped
1726680674 || (HEAD detached at labs/dev)[ ?? ] 51e3e1fc5
/home/cupress/ZINGO/libs/zingolib: 0
$cargo nextest run simpool_zero_value_change_orchard_to_sapling
    Finished `test` profile [optimized] target(s) in 0.16s
    Starting 1 test across 18 binaries (329 skipped)
        SLOW [> 60.000s] libtonode-tests::chain_generics chain_generics::simpool_zero_value_change_orchard_to_sapling
        PASS [  74.089s] libtonode-tests::chain_generics chain_generics::simpool_zero_value_change_orchard_to_sapling
------------
     Summary [  74.089s] 1 test run: 1 passed (1 slow), 329 skipped
1726680930 || (HEAD detached at labs/dev)[ ?? ?? ] 51e3e1fc5

result: upgraded zcashd version increased speed of test
hypothesis invalidated

@nachog00
Copy link
Contributor

Do you recall the version of zcashd on these runs @fluidvanadium ?

@fluidvanadium
Copy link
Contributor Author

tested both 5.7.0 and 5.7.10 with no issues

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

No branches or pull requests

4 participants