Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed show-timing work when no tty attached #1165

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

fedusia
Copy link

@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
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.

None yet

1 participant