Merge lp:~brandontschaefer/unity/move-slow-tests-to-gtest-slow into lp:unity
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Marco Trevisan (Treviño) | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 3234 | ||||
Proposed branch: | lp:~brandontschaefer/unity/move-slow-tests-to-gtest-slow | ||||
Merge into: | lp:unity | ||||
Diff against target: |
311 lines (+161/-65) 5 files modified
tests/CMakeLists.txt (+25/-2) tests/test_im_text_entry.cpp (+2/-63) tests/test_im_text_entry.h (+47/-0) tests/test_im_text_entry_class.cpp (+41/-0) tests/test_im_text_entry_slow.cpp (+46/-0) |
||||
To merge this branch: | bzr merge lp:~brandontschaefer/unity/move-slow-tests-to-gtest-slow | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Marco Trevisan (Treviño) | Approve | ||
Review via email: mp+153951@code.launchpad.net |
Commit message
New test suite, test-gtest-slow for tests that are slow, now test-gtest runs at about ~30 seconds which is a huge improvement.
Description of the change
=== Problem ===
test-gtest was taking ~60-70 seconds to finish...which is to long for normal unit tests.
=== Fix ===
Since the tests that were slowing things down are still good tests, move it over into test-gtest-slow, which will now be a place to put unit tests that are slow (ie. half a second is slow).
=== Tests ===
All tests still pass, as this was just refactoring some tests.
[==========] 1081 tests from 83 test cases ran. (29894 ms total) <--- A lot faster!
Tests that still need to be moved over:
http://