Unit tests improvements

[AurelienFT]

Context

The goal is to ensure security in the node before mainnet release. We want to improve the unit test base to help us achieve this goal. There is a list of criteria that we want to validate :

• 80% of the code tested and all in the sub crates (All coverage is calculated using cargo-llvm-cov)
• Tests to ensure all low limit and high limit of serializer/deserializer are tested

Current test coverage on the whole project : 44.61% (Update: 20/09/2023) (without bootstrap as it's impossible to run tarpaulin with the bootstrap test)

I will update the issue module by module :

massa-api

Managed by @modship and @aoudiamoncef

massa async pool

Actual code coverage

11,69 % (Updated : 20/09/2023) (command used cargo llvm-cov test --open --package massa_async_pool --features testing --ignore-filename-regex "(massa-models)|(massa-hash)|(massa-time)|(massa-signature)|(massa-ledger-exports)|(massa-serialization)")

• Serializers and deserializers limits tested

massa-bootstrap

Managed by @Leo-Besancon and @litchipi

massa-client

@damip we skip this, no ?

massa-cipher

0,00 % (Updated : 20/09/2023) (command used cargo llvm-cov test --open --package massa_cipher --ignore-filename-regex "(massa-models)|(massa-hash)|(massa-time)|(massa-signature)|(massa-ledger-exports)|(massa-serialization)")

massa-consensus-worker

58,27 % (Updated: 20/09/2023) (command used cargo llvm-cov test --open --package massa_consensus_worker --features testing --ignore-filename-regex "(massa-models)|(massa-hash)|(massa-time)|(massa-signature)|(massa-ledger-exports)|(massa-serialization)|(massa-consensus-exports)|(massa-channel)|(massa-async-pool)|(massa-db-exports)|(massa-executed-ops)|(massa-execution-exports)|(massa-final-state)|(massa-metrics)|(massa-module-cache)|(massa-pool-exports)|(massa-protocol-exports)|(massa-pos-exports)|(massa-storage)|(massa-versioning)")

• Serializers and deserializers limits tested

massa-db-worker

0,00 % (Updated: 20/09/2023) (command used cargo llvm-cov test --open --package massa_db_worker --ignore-filename-regex "(massa-models)|(massa-hash)|(massa-time)|(massa-signature)|(massa-ledger-exports)|(massa-serialization)|(massa-consensus-exports)|(massa-channel)|(massa-async-pool)|(massa-db-exports)|(massa-executed-ops)|(massa-execution-exports)|(massa-final-state)|(massa-metrics)|(massa-module-cache)|(massa-pool-exports)|(massa-protocol-exports)|(massa-pos-exports)|(massa-storage)|(massa-versioning)")

• Serializers and deserializers limits tested

massa-executed-ops

0,00% (Updated: 20/09/2023) (command used cargo llvm-cov test --open --package massa_executed_ops --ignore-filename-regex "(massa-models)|(massa-hash)|(massa-time)|(massa-signature)|(massa-ledger-exports)|(massa-serialization)|(massa-consensus-exports)|(massa-channel)|(massa-async-pool)|(massa-db-exports)|(massa-execution-exports)|(massa-final-state)|(massa-metrics)|(massa-module-cache)|(massa-pool-exports)|(massa-protocol-exports)|(massa-pos-exports)|(massa-storage)|(massa-versioning)" )

• Serializers and deserializers limits tested

[damip]

Yes we can skip the client for now :)

[litchipi]

Tests improvements on the bootstrap: #4454