Beef up execution tests

[MitchTurner]

Each of the tests should:

1. Each produced block also is valdiated
2. We produce the same block twice. First from intial block, the second from the first iteratiopn block
3. All modification are done in InMemoryTransaction and we compare them after production and validation. The idea is that they are exactly the same

In addition, there are probably missing tests. Add any tests that seem to be missing.

[MitchTurner]

It doesn't look like we have any tests that check for Errs.

[MitchTurner]

Also, maybe as another PR, we should move the executor tests back to the executor service?

I think we should be testing the trait interfaces too. That might be another PR too. lol.