feat: add metrics in GlobalBlobAwareConflationCalculator and move bri…

[jonesho]

…dge logs and state root hash retrieval from execution prover client to coordinator

This PR implements issue(s) #604

Checklist

• I wrote new tests for my new core changes.
• I have successfully ran tests, style checker and build against my new changes locally.
• I have informed the team of any breaking changes if there are any.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 83.33333% with 26 lines in your changes missing coverage. Please review.

Project coverage is 68.67%. Comparing base (1edc38b) to head (984ce59).

Files with missing lines	Patch %	Lines
...in/kotlin/net/consensys/zkevm/domain/Conflation.kt	25.00%	7 Missing and 2 partials ⚠️
.../conflation/GlobalBlobAwareConflationCalculator.kt	90.12%	4 Missing and 4 partials ⚠️
.../consensys/zkevm/coordinator/app/L1DependentApp.kt	0.00%	5 Missing ⚠️
...on/proofcreation/ZkProofCreationCoordinatorImpl.kt	93.75%	2 Missing ⚠️
...clients/prover/FileBasedExecutionProverClientV2.kt	93.75%	1 Missing ⚠️
.../coordinator/clients/prover/ProverClientFactory.kt	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #654      +/-   ##
============================================
+ Coverage     68.38%   68.67%   +0.29%     
- Complexity     1185     1201      +16     
============================================
  Files           323      323              
  Lines         13131    13227      +96     
  Branches       1319     1330      +11     
============================================
+ Hits           8979     9084     +105     
+ Misses         3600     3589      -11     
- Partials        552      554       +2     

Flag	Coverage Δ		*Carryforward flag
hardhat	98.52% <ø> (ø)		Carriedforward from e71c0c8
kotlin	66.37% <83.33%> (+0.33%)	⬆️

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines	Coverage Δ
...tor/clients/BatchExecutionProverRequestResponse.kt	81.81% <100.00%> (+37.37%)	⬆️
...on/conflation/BlockToBatchSubmissionCoordinator.kt	92.10% <100.00%> (+0.43%)	⬆️
...conflation/ConflationCalculatorByDataCompressed.kt	96.00% <100.00%> (ø)
...clients/prover/FileBasedExecutionProverClientV2.kt	48.97% <93.75%> (-10.04%)	⬇️
.../coordinator/clients/prover/ProverClientFactory.kt	62.50% <0.00%> (+2.15%)	⬆️
...on/proofcreation/ZkProofCreationCoordinatorImpl.kt	94.28% <93.75%> (+94.28%)	⬆️
.../consensys/zkevm/coordinator/app/L1DependentApp.kt	1.46% <0.00%> (-0.01%)	⬇️
.../conflation/GlobalBlobAwareConflationCalculator.kt	93.52% <90.12%> (-1.56%)	⬇️
...in/kotlin/net/consensys/zkevm/domain/Conflation.kt	59.37% <25.00%> (-0.28%)	⬇️

... and 5 files with indirect coverage changes

[jpnovais]

Same goes here.

Also, we should add tests to check that the metrics are correct.

[jonesho]

Agree to add tests for the metrics and would use mocks here for metricsFacade and histogram instances to check the metric values passed to the histogram record function