Aggregator refactoring: services and dependencies container

[ghubertpalo]

Content

This PR includes a rework of the Aggregator's code in order to tidy up its code:

• It gather all code related to dependency management into the dependency_injection module and rename the builded containers to DependencyContainer (from DependencyManager).
• It move all services that were at the source top level into a new services module.

Pre-submit checklist

• Branch
  • Tests are provided (if possible)
  • Crates versions are updated (if relevant)
  • Commit sequence broadly makes sense
  • Key commits have useful messages
• PR
  • No clippy warnings in the CI
  • Self-reviewed the diff
  • Useful pull request description
  • Reviewer requested
• Documentation
  • Update README file (if relevant)
  • Update documentation website (if relevant)
  • Add dev blog post (if relevant)

Comments

None

Issue(s)

Closes #1058

[github-actions]

Test Results

    3 files  ±0    16 suites  ±0   4m 56s ⏱️ -47s
635 tests ±0  635 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
673 runs  ±0  673 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 1ffaf6a. ± Comparison against base commit 12e3af1.

This pull request removes 22 and adds 22 tests. Note that renamed tests count towards both.

mithril-aggregator ‑ certifier_service::tests::should_clean_epoch_when_inform_epoch
mithril-aggregator ‑ certifier_service::tests::should_create_certificate_when_multi_signature_produced
mithril-aggregator ‑ certifier_service::tests::should_not_create_certificate_for_open_message_already_certified
mithril-aggregator ‑ certifier_service::tests::should_not_create_certificate_for_open_message_not_created
mithril-aggregator ‑ certifier_service::tests::should_not_create_certificate_when_no_multi_signature_produced
mithril-aggregator ‑ certifier_service::tests::should_not_register_invalid_single_signature
mithril-aggregator ‑ certifier_service::tests::should_not_register_single_signature_for_certified_open_message
mithril-aggregator ‑ certifier_service::tests::should_register_valid_single_signature
mithril-aggregator ‑ certifier_service::tests::test_epoch_gap_certificate_chain
mithril-aggregator ‑ certifier_service::tests::test_epoch_gap_certificate_chain_ok
…

mithril-aggregator ‑ services::certifier::tests::should_clean_epoch_when_inform_epoch
mithril-aggregator ‑ services::certifier::tests::should_create_certificate_when_multi_signature_produced
mithril-aggregator ‑ services::certifier::tests::should_not_create_certificate_for_open_message_already_certified
mithril-aggregator ‑ services::certifier::tests::should_not_create_certificate_for_open_message_not_created
mithril-aggregator ‑ services::certifier::tests::should_not_create_certificate_when_no_multi_signature_produced
mithril-aggregator ‑ services::certifier::tests::should_not_register_invalid_single_signature
mithril-aggregator ‑ services::certifier::tests::should_not_register_single_signature_for_certified_open_message
mithril-aggregator ‑ services::certifier::tests::should_register_valid_single_signature
mithril-aggregator ‑ services::certifier::tests::test_epoch_gap_certificate_chain
mithril-aggregator ‑ services::certifier::tests::test_epoch_gap_certificate_chain_ok
…

♻️ This comment has been updated with latest results.

[jpraynaud]

LGTM 👍

A more specific/detailed name for the PR is welcome!