#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 , 12 years ago
| Status: | new → assigned | 
|---|
comment:2 by , 12 years ago
comment:3 by , 12 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 , 12 years ago
| Release Notes: | modified (diff) | 
|---|---|
| Resolution: | → fixed | 
| Status: | assigned → closed | 



  
A few more changes:
unittest.makeSuite,testis the default prefix so it doesn't need to be specified (at least for Python 2.5-2.7):unittest.mainin each test case module.