Skip to content

disable progress bars when running the tests to avoid messing up test suite output #3835

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.

Sign up for GitHub

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

Merged
akesandgren merged 3 commits into from
Sep 20, 2021
Merged

disable progress bars when running the tests to avoid messing up test suite output #3835

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
467c29c
disable progress bars when running the tests to avoid messing up test…
boegel Sep 14, 2021
d5ccae9
fix comment
boegel Sep 17, 2021
697955b
Merge branch 'develop' into disable_show_progress_bar_in_tests
boegel Sep 17, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions test/framework/output.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,9 @@ class OutputTest(EnhancedTestCase):
def test_create_progress_bar(self):
"""Test create_progress_bar function."""

# restore default (was disabled in EnhancedTestCase.setUp to avoid messing up test output)
update_build_option('show_progress_bar', True)

if HAVE_RICH:
expected_progress_bar_class = rich.progress.Progress
else:
Expand Down
7 changes: 6 additions & 1 deletion test/framework/utilities.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@
from easybuild.framework.easyblock import EasyBlock
from easybuild.main import main
from easybuild.tools import config
from easybuild.tools.config import GENERAL_CLASS, Singleton, module_classes
from easybuild.tools.config import GENERAL_CLASS, Singleton, module_classes, update_build_option
from easybuild.tools.configobj import ConfigObj
from easybuild.tools.environment import modify_env
from easybuild.tools.filetools import copy_dir, mkdir, read_file, which
Expand Down Expand Up @@ -137,6 +137,11 @@ def setUp(self):

init_config()

# disable progress bars when running the tests,
# since it messes up with test suite progress output when test installations are performed
os.environ['EASYBUILD_DISABLE_SHOW_PROGRESS_BAR'] = '1'
update_build_option('show_progress_bar', False)

import easybuild
# try to import easybuild.easyblocks(.generic) packages
# it's OK if it fails here, but important to import first before fiddling with sys.path
Expand Down