#11443 closed enhancement (fixed)
Remove unnecessary web server restarts in functional test cases
Reported by: | Ryan J Ollos | Owned by: | Ryan J Ollos |
---|---|---|---|
Priority: | normal | Milestone: | 1.0.2 |
Component: | general | Version: | |
Severity: | normal | Keywords: | functional tests |
Cc: | Branch: | ||
Release Notes: |
Reduced execution time of functional tests by removing unnecessary web server restarts. |
||
API Changes: | |||
Internal Changes: |
Description
There are numerous calls to FunctionalTestEnvironment.restart
that seem to be unnecessary. I'm not sure why they were needed, but the issue may have been fixed with [12258].
The proposed changes reduce the execution time on my machine from 170-180 seconds to < 150 seconds.
Proposed changes in log:rjollos.git:t11443. Tested on Ubuntu 13.04. Please test execution on additional platforms if you can.
For reference the result of test execution on 1.0-stable at r12430 is:
$ PYTHONPATH=. python -m trac.test Ran 1647 tests in 270.404s
Attachments (0)
Change History (5)
comment:1 by , 11 years ago
Status: | new → assigned |
---|
comment:2 by , 11 years ago
comment:3 by , 11 years ago
Results of functional tests with the following environments:
log:rjollos.git:t11443 | 1.0-stable r12440 | ||
CentOS 5 with ext3 | Python 2.5 | 1068.505s | 1115.442s |
CentOS 6 with ext4 | Python 2.6 | 314.240s | 462.444s |
Windows XP sp3 | Python 2.5 | 547.047s | 567.766s |
comment:4 by , 11 years ago
Release Notes: | modified (diff) |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
A few more changes:
unittest.makeSuite
,test
is the default prefix so it doesn't need to be specified (at least for Python 2.5-2.7):unittest.main
in each test case module.