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

Incorrect output order in CLI output #644

Closed
vojtech-pejsa-redbit opened this issue Feb 26, 2024 · 1 comment
Closed

Incorrect output order in CLI output #644

vojtech-pejsa-redbit opened this issue Feb 26, 2024 · 1 comment

Comments

@vojtech-pejsa-redbit
Copy link
Contributor

Hi and thank you for this wonderful tool 🙂.

I have found small bug with output order when running from cli.

test.http

### test 1

GET https://reqres.in/api/users/2

?? status == 400

### test 2

GET https://reqres.in/api/users/2

?? status == 401

output

httpyac send test.http --all --output short --output-failed short

obrazek

Affected versions

I replicated this behavior with current version (6.11.4), first version with this output is 6.6.3, version 6.6.2 works fine.

Related issues

I am not sure if this issue is the same as #237, but it looks similar.

@AnWeber
Copy link
Owner

AnWeber commented Feb 27, 2024

thanks for the hint with the version. You were right Commit e4f867a was the culprit. I have a hen/egg problem. I only want to issue the request if the tests are successful. However, the tests should appear after the request. I added a special log method to support changing order for the log messages

@AnWeber AnWeber closed this as completed Feb 27, 2024
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

No branches or pull requests

2 participants