Fix next_protocol_multi_signerbuild in FakeEpochService::with_data

Author: sfauvel

mithril-aggregator/src/services/epoch_service.rs

@@ -215,8 +215,6 @@ impl EpochService for MithrilEpochService {
         let next_signers_with_stake = self
             .get_signers_with_stake_at_epoch(next_signer_retrieval_epoch)
             .await?;
-
-        // TODO could it be better to create a EpochData::new and to this in the impl ? So, it'll not done twice in FakeEpochService
         let current_signers = Signer::vec_from(current_signers_with_stake.clone());
         let next_signers = Signer::vec_from(next_signers_with_stake.clone());
 
@@ -346,12 +344,10 @@ impl FakeEpochService {
                 .unwrap()
                 .build_multi_signer();
         let next_protocol_multi_signer =
-        //    SignerBuilder::new(current_signers_with_stake, protocol_parameters)
-        // TODO Is it correct to use current_signers_with_stake here and not next_signers_with_stake ?
-        SignerBuilder::new(current_signers_with_stake, protocol_parameters)
-            .with_context(|| "Could not build protocol_multi_signer for epoch service")
-            .unwrap()
-            .build_multi_signer();
+            SignerBuilder::new(next_signers_with_stake, next_protocol_parameters)
+                .with_context(|| "Could not build protocol_multi_signer for epoch service")
+                .unwrap()
+                .build_multi_signer();
 
         let current_signers_with_stake = current_signers_with_stake.to_vec();
         let next_signers_with_stake = next_signers_with_stake.to_vec();