Additional testing for tracker and tx modules - mostly around serialization/deserialization#30

Merged

KPrasch merged 70 commits intonucypher:mainfrom derekpierre:additional-testing

Mar 11, 2024

+2,105-223

Commits on Mar 4, 2024

AsyncTxStrategy should do one of the following during execute: raise TransactionFaulted for unrecoverable/don't bother retrying errors, return updated params for an updated retry, or None if the st…
derekpierre
committed
Adjust strategies to accomodate latest paradigm for execute i.e. return None if the machine should do nothing and wait based on this strategy.
derekpierre
committed
Adjust existing strategy tests since None can now be returned from execute().
derekpierre
committed
Update timeout strategy logging.
derekpierre
committed
Update speedup strategy calcs to use math.ceil instead of round
derekpierre
committed
Move from a fixed max tip to a max tip factor based on the current suggested tip as a way to limit the speedup strategy getting out of control.
derekpierre
committed
Simplify legacy transaction speedup logic.
derekpierre
committed
Add tests for constructor parameters and legacy tx speed up functionality.
derekpierre
committed
Update logging of gas conditions.
derekpierre
committed
Better annotations about what the strategy is doing.
derekpierre
committed
Don't modify tx nonce as part of strategy.
derekpierre
committed
Fix outdated comment about value of warn factor.
derekpierre
committed
Clean up logging logic since old_tip, old_max_fee may or may not be present in transaction parameters (eip1559 allows one or both).
derekpierre
committed
Add tests for eip1559 transaction speedup.
derekpierre
committed
Rename speed up strategy; while the rate value is fixed the updates are expontential because they build on previous updates.
derekpierre
committed
Use constant instead of "maxPriorityFeePerGas".
derekpierre
committed
Use constant for minimum required speedup increase percentage.
derekpierre
committed
Use a max between a low default value and warn factor calc in case timeout is too short and warn threshold becomes too low.
derekpierre
committed
Add todo about best way of setting a cap on the speedup strategy.
derekpierre
committed
Use reactor to determine when hook is called instead of repeatedly calling _cycle() on the machine - the latter could hide bugs so don't do it.
derekpierre
committed
Add TODO to determine whether strategies can be overriden.
derekpierre
committed

Commits on Mar 5, 2024

Commits on Mar 6, 2024

Commits on Mar 7, 2024

Commits on Mar 8, 2024