google / googletest

GoogleTest - Google Testing and Mocking Framework
https://google.github.io/googletest/
BSD 3-Clause "New" or "Revised" License
34.94k stars 10.16k forks source link

[FR]: Streaming results format to stdout #4665

Open davidmatson opened 1 week ago

davidmatson commented 1 week ago

Does the feature exist in the most recent commit?

No

Why do we need this feature?

Currently, we have per-OS code for stream_result_to, and Windows support is not yet available.

Describe the proposal.

Provide a switch to have stdout report progress in the same format used by stream_result_to rather than the human-readable format.

Perhaps: --gtest_machine_results or --gtest_stream_result_to=stdout

Is the feature specific to an operating system, compiler, or build system version?

No

RedMarcher commented 1 week ago

Hello, I'm new to this particular project. Would you mind telling me which file the current implementation for stream_result_to is located at? You mentioned that it was different for each OS, so is the feature implementation currently spread out among multiple different files?

RedMarcher commented 1 week ago

Looking over your comment more closely, do you just want a feature where whatever is currently spit out of stream_result_to is also available to stdout?

davidmatson commented 1 week ago

Looking over your comment more closely, do you just want a feature where whatever is currently spit out of stream_result_to is also available to stdout?

yes (and also not having the pretty-print format written stdout)

RedMarcher commented 1 week ago

Looking over your comment more closely, do you just want a feature where whatever is currently spit out of stream_result_to is also available to stdout?

yes (and also not having the pretty-print format written stdout)

So basically I should add a new flag using one of the names you proposed to activate this feature?

davidmatson commented 1 week ago

So basically I should add a new flag using one of the names you proposed to activate this feature?

This is a feature request to add a new flag that would control this behavior, yes.