Skip to content

fixed show-timing work when no tty attached#1165

Open
fedusia wants to merge 1 commit intobehave:mainfrom
fedusia:fix_show_timings_in_pretty_formatter
Open

fixed show-timing work when no tty attached#1165
fedusia wants to merge 1 commit intobehave:mainfrom
fedusia:fix_show_timings_in_pretty_formatter

Conversation

@fedusia
Copy link
Copy Markdown

@fedusia fedusia commented Mar 14, 2024

By default behave uses pretty formatter, it can write in 2 stream types - tty or simple character device, for example simple file or pipe.

Common logic of processing steps described here
If stream is tty character device. Firstly it prints step during matching phase (just only parsed step from feature file without additional data), then step executes and writes additional data like step status, duration and then it rewrites last line in terminal with latest data.

If steam is NOT tty (file or pipe). It prints step information only during matching phase and doesn't rewrite it during execution phase.

This PR should change behavior.
If stream is not tty character device then print step information skips during match phase and prints after execution phase.

@jenisys jenisys force-pushed the main branch 2 times, most recently from 0a4d73b to 2c11d2e Compare May 14, 2024 22:39
@jenisys jenisys force-pushed the main branch 2 times, most recently from 3e51dda to c6ab01c Compare May 26, 2024 15:00
@jenisys jenisys force-pushed the main branch 3 times, most recently from eccf022 to 93e1218 Compare June 16, 2024 16:10
@jenisys jenisys force-pushed the main branch 2 times, most recently from cc16ac0 to 22569f4 Compare October 14, 2024 20:09
@jenisys jenisys force-pushed the main branch 2 times, most recently from 01407cf to 42e64a9 Compare October 24, 2024 23:28
@jenisys jenisys force-pushed the main branch 9 times, most recently from 9dc617b to b8b23bc Compare July 16, 2025 21:43
@jenisys jenisys force-pushed the main branch 4 times, most recently from 612d6d1 to 5796057 Compare August 4, 2025 06:58
@jenisys jenisys force-pushed the main branch 4 times, most recently from 6971c4c to b1470c0 Compare August 26, 2025 04:38
@jenisys jenisys force-pushed the main branch 2 times, most recently from 370ce68 to cba3c4f Compare August 31, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant