fix(core-blockchain): round deletion during rollback

[spkjp]

Summary

We must only delete rounds greater than the target round during rollback, otherwise we could end up with holes in the table.

Also removes the useless analyseFork state which did nothing.

Fixes #2935

Checklist

• Documentation (if necessary)
• Tests (if necessary)
• Ready to be merged

[codecov]

Codecov Report

Merging #2970 into develop will decrease coverage by 14.91%.
The diff coverage is 60%.

@@             Coverage Diff              @@
##           develop    #2970       +/-   ##
============================================
- Coverage     68.4%   53.49%   -14.92%     
============================================
  Files          405      404        -1     
  Lines         9829     9828        -1     
  Branches       509      509               
============================================
- Hits          6724     5257     -1467     
- Misses        3065     4536     +1471     
+ Partials        40       35        -5

Impacted Files	Coverage Δ
...ackages/core-blockchain/src/machines/blockchain.ts	100% <ø> (ø)	⬆️
.../core-database-postgres/src/postgres-connection.ts	0% <0%> (-69%)	⬇️
packages/core-blockchain/src/state-machine.ts	89.74% <100%> (ø)	⬆️
...ges/core-tester-cli/src/commands/debug/identity.ts	0% <0%> (-100%)	⬇️
...ages/core-api/src/handlers/transactions/methods.ts	0% <0%> (-100%)	⬇️
...s/core-api/src/handlers/shared/schemas/block-id.ts	0% <0%> (-100%)	⬇️
packages/core-tester-cli/src/logger.ts	0% <0%> (-100%)	⬇️
.../core-api/src/handlers/transactions/transformer.ts	0% <0%> (-100%)	⬇️
packages/core-api/src/handlers/peers/schema.ts	0% <0%> (-100%)	⬇️
...kages/core-api/src/handlers/transactions/routes.ts	0% <0%> (-100%)	⬇️
... and 102 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84aedb1...1e41137. Read the comment docs.