Skip to content

Conversation

asdacap
Copy link
Contributor

@asdacap asdacap commented Apr 3, 2025

  • Move some TestBlockchain code to DI, stopping at block processing as plugins have massively different way of initializing things.
  • Replace all 4 param of the Build with a single Action<ContainerBuilder>. Functionality replaced with a TestBlockchain.Configuration.
  • Make sure pruning trie store is used in most tests.
  • Make sure full receipt storage is used in most tests, which actually have some different behaviour compared to in memory receipt storage.

Types of changes

What types of changes does your code introduce?

  • Refactoring

Testing

Requires testing

  • Yes
  • No

If yes, did you write tests?

  • Yes
  • No

Notes on testing

  • The whole thing is tests.

@asdacap asdacap merged commit d22055d into master Apr 7, 2025
80 checks passed
@asdacap asdacap deleted the refactor-test/move-testblockchain-to-di branch April 7, 2025 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants