Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0027113Openbravo ERPA. Platformpublic2014-07-17 14:072014-08-08 18:53
shuehner 
mtaal 
normalminorhave not tried
closedfixed 
5
 
3.0PR14Q4 
shankarb
Core
No
0027113: junit test HiddenUpdateTest is quite slow (2.5 minutes)
That test is quite slow:
https://ci.openbravo.com/job/int-checks-pgsql/lastCompletedBuild/testReport/org.openbravo.test.dal/HiddenUpdateTest/testHiddenUpdates/ [^]

Currently it does process every single record of all entities.

However after quick chat with MTA it looks not all of that is needed.

Test should just cover all available field-types.

So options:
- limit 1 and still do all entities
- manually selected list of entities just enough to cover all field-types
- combination of both options above
run that junit test and check runtime.
No tags attached.
Issue History
2014-07-17 14:07shuehnerNew Issue
2014-07-17 14:07shuehnerAssigned To => mtaal
2014-07-17 14:07shuehnerModules => Core
2014-07-17 14:07shuehnerTriggers an Emergency Pack => No
2014-07-19 19:59hgbotCheckin
2014-07-19 19:59hgbotNote Added: 0068739
2014-07-19 19:59hgbotStatusnew => resolved
2014-07-19 19:59hgbotResolutionopen => fixed
2014-07-19 19:59hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d4dc31fc531b46b30b6fcd964bed57a2d5c1e900 [^]
2014-07-29 08:47alostaleReview Assigned To => shankarb
2014-07-31 06:20shankarbNote Added: 0068992
2014-07-31 06:20shankarbStatusresolved => closed
2014-07-31 06:20shankarbFixed in Version => 3.0PR14Q4
2014-08-08 18:53hudsonbotCheckin
2014-08-08 18:53hudsonbotNote Added: 0069187

Notes
(0068739)
hgbot   
2014-07-19 19:59   
Repository: erp/devel/pi
Changeset: d4dc31fc531b46b30b6fcd964bed57a2d5c1e900
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sat Jul 19 19:59:28 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/d4dc31fc531b46b30b6fcd964bed57a2d5c1e900 [^]

Fixes issue 27113: junit test HiddenUpdateTest is quite slow (2.5 minutes)

---
M src-test/org/openbravo/test/dal/HiddenUpdateTest.java
---
(0068992)
shankarb   
2014-07-31 06:20   
Code reviewed and verified in pi changeset be933e11432c.
(0069187)
hudsonbot   
2014-08-08 18:53   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/4450016dee64 [^]
Maturity status: Test