catchorg / catchorg/Catch2

Make "Randomness seeded to: X" message optional?

Open
#2,664 1 comment 5 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
21.5k
Forks
3.5k
Avg merge
3d 16h
Merged PRs (30d)
2

Description

**Description**
Currently the message in unconditional:
https://github.com/catchorg/Catch2/blob/1f881ab4641b866304b818a063164dae976dba5f/src/catch2/reporters/catch_reporter_console.cpp#L514-L521

So it is shown every single time:
```
Randomness seeded to: 784548612
===============================================================================
All tests passed (5147 assertions in 174 test cases)
```

Not denying that this feature is supposedly important in certain use cases, I don't explicitly use Catch2's RNG in my tests and could not care less to what exactly the randomness was seeded, so the message only distracts.

I'd expect it to be exposed by xml and other machine-readable reporters, and even by console when `--verbosity high` is set, why not, but doing so unconditionally and by default is just annoying.

Thanks

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.