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

Sort output in Documentation format #416

Closed
marcinpraczko opened this issue Jan 9, 2019 · 4 comments · Fixed by #814
Closed

Sort output in Documentation format #416

marcinpraczko opened this issue Jan 9, 2019 · 4 comments · Fixed by #814

Comments

@marcinpraczko
Copy link

As I can see - documentation format is displaying different order for checks. This is making work with this format quite hard especially if results presents many checks / results. Having the same order allows quicker to see output when refresing web brwoser (to re-check status of check)

(As I do understand - I can be wrong - Goss displays results based on dictionary - which is not sorted by default - can we add some OrderDict function on Sort? - I am familar with python and this suggestion is based on python experience).

@aelsabbahy
Copy link
Member

It's not actually a dictionary. Rather the tests run in parallel and are streamed to the output as they're completed.

Sorting the output is possible, essentially the output driver would have to record everything first, then sort it all, then print it. Rather than printing it as it comes in.

@marcinpraczko
Copy link
Author

As I can see this was approved - so with my observation this is really helpful during debug of issue. For example - error is detected / one is working on soluton and expecting to see 'OK' in the same line ... (how human mind is working). So unsorted list generating different output on the same line - so one things 'is fixed' - but maybe is not really fixed - only line changed.
I assume that is dictionary - based on dicts in python. Thank you for clarification :).

aelsabbahy added a commit that referenced this issue Jun 24, 2023
@aelsabbahy aelsabbahy mentioned this issue Jun 25, 2023
3 tasks
@marcinpraczko
Copy link
Author

Hi @aelsabbahy, thank you for implementing this - I've just tested and seems be working as expected (at least for cases which are useful for me). Great work - thank you.

@aelsabbahy
Copy link
Member

aelsabbahy commented Nov 22, 2023

Sorry for the extremely long delay on this, has been an interesting journey. Glad you're still using goss, and glad this helps.

Feel free to open up any feature requests or bug reports.. always good to get feedback from long term goss users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants