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

Use --verbose option to not list IGNORED tests & environments combinations #3726

Closed
costas-basdekis opened this issue Nov 3, 2020 · 0 comments
Assignees
Milestone

Comments

@costas-basdekis
Copy link

Description of problem

When you run tests, if you have a lot of environments, and only want to run one, it all the ignored environment are listed, which can make it tedious to scroll up. For example, when running tests for Marlin there are 78 environments, and if you want to run only one tests out of many, then there's multiple of those lines

In commands/run/command.py we hide those lines with --verbose:

        elif result.get("succeeded") is None:
            if not verbose:
                continue
            status_str = "IGNORED"

But not in commands/test/command.py:

        elif result.get("succeeded") is None:
            status_str = "IGNORED"

If there are no concerns for backwards compatibility (eg some people might expected IGNORED tests to show up), the change is simple:

-     print_testing_summary(results)
+     print_testing_summary(results, verbose)

- def print_testing_summary(results):
+ def print_testing_summary(results, verbose=False):

          elif result.get("succeeded") is None:
+             if not verbose:
+                 continue
              status_str = "IGNORED"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants