search

parallelchain-io / hotstuff_rs

Rust implementation of the HotStuff consensus algorithm.
38 stars 5 forks source link

Suppress log messages from integration tests by default #38

Closed lyulka closed 3 months ago

lyulka commented 3 months ago

Currently, running cargo test causes log messages (a lot of them) to be printed to the terminal.

Ideally, these messages should be suppressed and only displayed if —nocapture is passed.

If there’s no straightforward way to do this with the log/fern, then maybe it’d be better if we switch to just using println! to log out messages from integration tests.

lyulka commented 3 months ago

We should also make the logs more informative and readable by:

  1. Identifying the source of log messages by prefixing them by the origin Node's VerifyingKey, instead of a ThreadID.
  2. Removing the "[Debug]" infix.
lyulka commented 3 months ago

Done in 278a1f4.