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

Format CLI test list #1180

Closed
schoren opened this issue Sep 8, 2022 · 2 comments · Fixed by #1241
Closed

Format CLI test list #1180

schoren opened this issue Sep 8, 2022 · 2 comments · Fixed by #1241
Assignees
Labels

Comments

@schoren
Copy link
Contributor

schoren commented Sep 8, 2022

the test list command still outputs json. We need to implement a simple table, and support the --json option (see #1179)

Example:

$ tracetest test list

| Name            | Last run            | Passed | Failed | Link                                                            |
| --------------- | ------------------- | ------ | ------ | --------------------------------------------------------------- |
| List Pokemons   | 2022-05-23 17:34:21 |    5   |    0   | http://localhost:8080/test/41985306-2fdc-4329-a668-08b1bddcfa42 |
| Delete Pokemons | 2022-05-23 14:00:00 |   11   |    1   | http://localhost:8080/test/41985306-2fdc-4329-a668-08b1bddcfa42 |
| Import Pokemons | 2022-05-23 11:10:54 |    0   |    1   | http://localhost:8080/test/41985306-2fdc-4329-a668-08b1bddcfa42 |
@schoren schoren self-assigned this Sep 8, 2022
@schoren schoren added the CLI label Sep 8, 2022
@schoren
Copy link
Contributor Author

schoren commented Sep 8, 2022

I'm not convinced that we should show the last run, passed, failed columns, specially because the link points to the test page and not the test run details. I also want to point to that page to keep the links as short as possible.

What do you think @mathnogueira?

@mathnogueira
Copy link
Contributor

I dont have a strong opinion about it. Maybe just the last execution time would be nice to have. But the passed/failed could be removed

@schoren schoren moved this from Todo to In Progress in Tracetest - 0.7 Release Sep 15, 2022
@schoren schoren moved this from In Progress to Waiting for approval in Tracetest - 0.7 Release Sep 16, 2022
@schoren schoren moved this from Waiting for approval to In Progress in Tracetest - 0.7 Release Sep 16, 2022
@schoren schoren linked a pull request Sep 19, 2022 that will close this issue
4 tasks
@schoren schoren moved this from In Progress to Waiting for approval in Tracetest - 0.7 Release Sep 19, 2022
Repository owner moved this from Waiting for approval to Done in Tracetest - 0.7 Release Sep 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants