test(core-transaction-pool): collator test

[rainydio]

Summary

Part of #3482. It's missing catch coverage, because I need to implement logging first. It's difficult to mock otherwise. Also I noticed that it makes more sense to check for transaction size first before trying to validate it.

Checklist

• Tests
• Ready to be merged

[codecov]

Codecov Report

Merging #3533 into 3.0 will decrease coverage by 0.61%.
The diff coverage is 21.81%.

@@            Coverage Diff             @@
##              3.0    #3533      +/-   ##
==========================================
- Coverage   48.69%   48.07%   -0.62%     
==========================================
  Files         539      548       +9     
  Lines       13549    13861     +312     
  Branches     1823     1901      +78     
==========================================
+ Hits         6598     6664      +66     
- Misses       6922     7168     +246     
  Partials       29       29

Impacted Files	Coverage Δ
packages/core-p2p/src/utils/build-rate-limiter.ts	66.66% <ø> (ø)	⬆️
...s/core-test-framework/src/app/generators/crypto.ts	91.17% <ø> (ø)	⬆️
packages/core-p2p/src/schemas.ts	100% <ø> (ø)	⬆️
...kages/crypto/src/transactions/types/transaction.ts	78.37% <ø> (ø)	⬆️
packages/crypto/src/validation/schemas.ts	100% <ø> (ø)	⬆️
packages/core-api/src/resources/delegate.ts	80% <ø> (ø)	⬆️
...ges/core-magistrate-api/src/routes/bridgechains.ts	0% <ø> (ø)	⬆️
packages/core-p2p/src/utils/socket.ts	11.53% <ø> (ø)	⬆️
packages/core/src/commands/snapshot-rollback.ts	50% <ø> (ø)	⬆️
...ypto/src/transactions/utils/bridgechain-schemas.ts	100% <ø> (ø)	⬆️
... and 72 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 95a546a...a17c491. Read the comment docs.

[rainydio]

Already merged in #3499