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

Add --progress option to print progressive status and executed count #236

Closed
lepapareil opened this issue Jul 8, 2021 · 2 comments · Fixed by #249
Closed

Add --progress option to print progressive status and executed count #236

lepapareil opened this issue Jul 8, 2021 · 2 comments · Fixed by #249
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@lepapareil
Copy link
Collaborator

hurl version = 1.2.0

I write 4 test files, 3 will succeed, 1 will fail.

Currently I get:

$ hurl test*.hurl
error: Assert Status
  --> /dir1/dir2/dir3/file3.hurl:7:10
   |
 7 | HTTP/1.1 200
   |          ^^^ actual value is <500>
   |

What i expect using the new parameter --progress

$ hurl --progress test*.hurl
/dir1/dir2/dir3/file1.hurl: success         
/dir1/dir2/dir3/file2.hurl: success         
/dir1/dir2/dir3/file3.hurl: failure          
error: Assert Status
  --> /dir1/dir2/dir3/file3.hurl:7:10
   |
 7 | HTTP/1.1 200
   |          ^^^ actual value is <500>
   |
/dir1/dir2/dir3/file4.hurl: success

Which could give on screen:

--progress

@fabricereix
Copy link
Collaborator

fabricereix commented Jul 27, 2021

For the next version, we can start by simply adding these 2 lines:

  • FILENAME: running before running the test
  • FILENAME: STATUS after runing the test

Replacing 'running' by the test status may not be possible if errors are printed.
We also need to clarify the behavior in verbose mode.

@fabricereix fabricereix linked a pull request Jul 27, 2021 that will close this issue
@fabricereix fabricereix changed the title add --progress option to print progressive status and executed count Add --progress option to print progressive status and executed count Sep 1, 2021
@fabricereix
Copy link
Collaborator

We can't really replace running by success| failure.
The cursor is rather moved with a terminal instruction such as tput.
In this case, the ouput would not be greppable anymore.

@lepapareil lepapareil added this to the 1.3.0 milestone Oct 12, 2021
@lepapareil lepapareil removed the 1.3.0 label Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants