-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Rework CLI UI/UX #26
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
Comments
Emily Ekberg Provide percentiles for trend metrics Emily Ekberg Provide percentiles for trend metrics Emily Ekberg David Rosen Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Pepe Cano Ruiz The new group output looks better IMO. Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg We also can't really do much along the lines of real-time updates in a console, which is exactly why the web UI exists - but if we can get this UI working properly, we could flip --run back to --paused (start tests immediately) and default to quitting at the end of a test, as the same information will be presented at the end of a test! The web UI also helps ensure that the API is actually functional and usable to work with, since I'll have to work with it myself. Pepe Cano Ruiz All the details comes to my mind: As David says; you have to check the colours. Could you represent a nested grouping? 3 assertions of 5 passed, 2 failed. (in red) --> From QUnit David Rosen @uppfinnarn - The "front page" and "some article" sections, are those groups? If so, can we make that more clear by at least adding a header? Also, in the case of nested groups in scripts, would this output also reflect that? Lastly, is there a general need for "min" as an output metric? Would it not be more valuable to instead report e.g. 95th percentile? Emily Ekberg I'm particularly iffy about what information to present at the top and how to do it, along with the metrics at the bottom - the fields are supposed to be in order, but I couldn't think of any good way to present them. For machine-readable output, I'm thinking of having the option to output YAML or JSON rather than human-readable output, I don't think we can have both be the same. Emily Ekberg Pepe Cano Ruiz I didn't review deeply the information being presented. Let me know if you have any specific question. Emily Ekberg Emily Ekberg Emily Ekberg Some kind of idea I threw together, very much unfinished. Emily Ekberg David Rosen Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Emily Ekberg Pepe Cano Ruiz The new group output looks better IMO. Emily Ekberg Emily Ekberg Emily Ekberg We also can't really do much along the lines of real-time updates in a console, which is exactly why the web UI exists - but if we can get this UI working properly, we could flip --run back to --paused (start tests immediately) and default to quitting at the end of a test, as the same information will be presented at the end of a test! The web UI also helps ensure that the API is actually functional and usable to work with, since I'll have to work with it myself. Pepe Cano Ruiz All the details comes to my mind: As David says; you have to check the colours. Could you represent a nested grouping? 3 assertions of 5 passed, 2 failed. (in red) --> From QUnit David Rosen @uppfinnarn - The "front page" and "some article" sections, are those groups? If so, can we make that more clear by at least adding a header? Also, in the case of nested groups in scripts, would this output also reflect that? Lastly, is there a general need for "min" as an output metric? Would it not be more valuable to instead report e.g. 95th percentile? Emily Ekberg I'm particularly iffy about what information to present at the top and how to do it, along with the metrics at the bottom - the fields are supposed to be in order, but I couldn't think of any good way to present them. For machine-readable output, I'm thinking of having the option to output YAML or JSON rather than human-readable output, I don't think we can have both be the same. Emily Ekberg Pepe Cano Ruiz I didn't review deeply the information being presented. Let me know if you have any specific question. Emily Ekberg Emily Ekberg Some kind of idea I threw together, very much unfinished. Emily Ekberg |
test(output): add integration tests
Imported from https://trello.com/c/wRUf1Gbf/101-rework-cli-ui-ux
https://zapier.com/engine/hydrate/875802/.eJwVjEEOwiAQAP-y5x4syLLhEX6BLBQswaIpm5im6d_F68xkTiitC7eYfFnAzXdETdZOkEt6Lb7xlsABi3Bct9SkwwRxTbH6mg5wZA3d1EDvJsN6OT7__jGq-uX92cGdEHkfazBkAusQtFUZZ1RIWfFCCNf1A22HKgk:1cK0cU:wREbIdBbmX3oQe_hoxXtsh7YuRI/
The text was updated successfully, but these errors were encountered: