Merge lp:~townsend/unity/fix-icon-loader-test into lp:unity
Proposed by
Christopher Townsend
Status: | Merged |
---|---|
Approved by: | Brandon Schaefer |
Approved revision: | no longer in the source branch. |
Merged at revision: | 3508 |
Proposed branch: | lp:~townsend/unity/fix-icon-loader-test |
Merge into: | lp:unity |
Diff against target: |
39 lines (+5/-3) 1 file modified
tests/test_icon_loader.cpp (+5/-3) |
To merge this branch: | bzr merge lp:~townsend/unity/fix-icon-loader-test |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Brandon Schaefer (community) | Approve | ||
Review via email: mp+185880@code.launchpad.net |
Commit message
Set the wait timeout to a longer period (1000ms) since many Jenkins CI tests are failing in the TestIconLoader tests.
Description of the change
= Issue =
Up to 3 TestIconLoader unit tests fail intermittently due the the 500ms default timeout elapsing during CI. It appears the Jenkins server may be under some sort of load when these tests fail.
= Fix =
For now, try increasing the timeout to 1000ms in hopes that this will get the test to complete. This timeout may need to be adjusted if the tests still fail.
To post a comment you must log in.
LGTM