-
Notifications
You must be signed in to change notification settings - Fork 6.8k
remove 'foo' and other print msg from test #16088
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LGTM. Thanks for your contribution! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Looks like the CI is not progressing. Can you retrigger the CI? I will merge once CI passes. Thanks.
2d718c1
to
5489260
Compare
* remove 'foo' and other print msg from test * address review comment * re-trigger CI
* remove 'foo' and other print msg from test * address review comment * re-trigger CI
@apeforest This PR is causing tests to fail . ====================================================================== FAIL: test_profiler.test_profile_task Traceback (most recent call last): File "C:\Python37\lib\site-packages\nose\case.py", line 198, in runTest
File "C:\jenkins_slave\workspace\ut-python-cpu@2\tests\python\unittest\test_profiler.py", line 115, in test_profile_task
AssertionError ====================================================================== FAIL: test_profiler.test_profile_tune_pause_resume Traceback (most recent call last): File "C:\Python37\lib\site-packages\nose\case.py", line 198, in runTest
File "C:\jenkins_slave\workspace\ut-python-cpu@2\tests\python\unittest\test_profiler.py", line 178, in test_profile_tune_pause_resume
File "C:\jenkins_slave\workspace\ut-python-cpu@2\tests\python\unittest\test_profiler.py", line 115, in test_profile_task
AssertionError |
Description
Many messages are printed from test_profiler unit test and clogged CI output. Removing all prints from test.