please also use the ticket tag in commit messages ([ECTM-846]) this enables integration with jira

@jvdp

The change is only for testmode so it should not affect normal production. We should probably at some point refactor the code so that the test specific parts are in a dedicated module though (there is a ticket for that)

The main problem here was that the rootHash for account storage was not set properly in getGenesisStateRoot.

Regarding the ditching of InMemoryWorldStateProxy :

From what I see InMemoryWorldStateProxy is more focused on building a transient state when running transactions, i.e, taking an initial state, modifying it and then commit the new resulting state.

But when we load the genesis state we do not have an initial state to start with so we have a kind of a chicken and egg problem (no initial to set the storage, and the initial state needs a storage).

My first working solution was to create the genesis state with empty contract storage, call the function with InMemoryWorldStateProxy, get the root hashes from there and then reload the genesis this time with the real contract storage.

I found it weird to have this logic (there was no reason to initialize the state in two passes), and with some guidance from Konrad I found how to directly store arbitrary MPTs. Calling directly the storage layer allows to create the desired state in the database directly without the double genesis initialization trick.