[BFT] Compliance engine requires config

[durkmurder]

Context

As part of #4125 forced compliance engines (consensus, cluster and follower) to accept modulecompliance.Config as one of the constructor arguments. Updated usages of config to use getter function which respects the lower bound. Reason for that was that it was the optimal way to reduce boilerplate code when reading CLI flags and config creation.

[codecov-commenter]

Codecov Report

Merging #4372 (84fab08) into master (b427320) will increase coverage by 0.07%.
The diff coverage is 73.01%.

@@            Coverage Diff             @@
##           master    #4372      +/-   ##
==========================================
+ Coverage   53.75%   53.83%   +0.07%     
==========================================
  Files         871      868       -3     
  Lines       80769    80788      +19     
==========================================
+ Hits        43420    43489      +69     
+ Misses      33915    33864      -51     
- Partials     3434     3435       +1     

Flag	Coverage Δ
unittests	53.83% <73.01%> (+0.07%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
cmd/execution_builder.go	0.00% <0.00%> (ø)
cmd/scaffold.go	14.59% <0.00%> (-0.11%)	⬇️
cmd/verification_builder.go	0.00% <0.00%> (ø)
...e/common/synchronization/request_handler_engine.go	22.00% <0.00%> (-3.00%)	⬇️
fvm/environment/programs.go	58.66% <0.00%> (ø)
module/metrics/herocache.go	0.00% <0.00%> (ø)
network/alsp/internal/cache_entity.go	50.00% <ø> (ø)
network/p2p/conduit/conduit.go	0.00% <0.00%> (ø)
network/p2p/p2pbuilder/libp2pNodeBuilder.go	0.00% <ø> (ø)
engine/common/follower/compliance_engine.go	81.10% <50.00%> (-0.38%)	⬇️
... and 9 more

... and 7 files with indirect coverage changes

[durkmurder]

bors merge

[bors]

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

• Integration Tests (make -C integration access-tests)
• Integration Tests (make -C integration bft-tests)
• Integration Tests (make -C integration collection-tests)
• Integration Tests (make -C integration consensus-tests)
• Integration Tests (make -C integration epochs-tests)
• Integration Tests (make -C integration execution-tests)
• Integration Tests (make -C integration ghost-tests)
• Integration Tests (make -C integration mvp-tests)
• Integration Tests (make -C integration network-tests)
• Integration Tests (make -C integration verification-tests)
• Lint (./)
• Lint (./crypto/)
• Lint (./integration/)
• Unit Tests (access)
• Unit Tests (admin)
• Unit Tests (cmd)
• Unit Tests (consensus)
• Unit Tests (engine)
• Unit Tests (fvm)
• Unit Tests (ledger)
• Unit Tests (module)
• Unit Tests (network)
• Unit Tests (others)
• Unit Tests (utils)