We rely on QBFTMessage identifier for current tests. I am not sure why this field is needed... The MessageID in the envelope should lead us to the proper runner and the slot/height to the proper instance. I think we need to remove this field from the message. Once this logic is removed dutyexe and identifier tests should be removed.
GalRogozinski
commented
3 months ago
Description
We rely on QBFTMessage
identifierfor current tests. I am not sure why this field is needed... TheMessageIDin the envelope should lead us to the proper runner and the slot/height to the proper instance. I think we need to remove this field from the message. Once this logic is removeddutyexeandidentifiertests should be removed.