Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0017493Openbravo ERPA. Platformpublic2011-06-03 13:282011-06-07 08:34
Reporterplujan 
Assigned Tomtaal 
PriorityimmediateSeveritymajorReproducibilityhave not tried
StatusclosedResolutionno change required 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0MP0Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0017493: Selenium tests are broken because of a change in the Date fields behaviour
DescriptionBecause of fix to issue 0017434, Selenium tests are not working properly. Reported issue 0017462 was rejected since it cannot be reproduced manually. However, since the fix Selenium tests are not filling the date fields. See steps.
Steps To ReproduceWhen executing a Unit test that fills a date field, the field is not filled with the desired date. See an output at http://goo.gl/lHQm1 [^]

The observed behaviour is:
1. Selenium opens a Sales Order in Form
2. Selenium clears current Order Date content (this is because of issue 0016730 )
3. Selenium writes desired value.
4. The ERP then, clears the value, changing it back to empty
5. Selenium tries to save. It shows an error because a mandatory field is empty.
Proposed SolutionI've spent several hours trying to workaround this issue, with no luck. So may be some change from the ERP side could help Selenium to behave as expected. Until this issue were fixed, no automated smoke tests could be executed.
Additional Information
TagsNo tags attached.
Relationships
has duplicate defect 0017499 closed alostale It is not possible to enter text with selenium on date fields 
related to feature request 0017560 new Triage Platform Base Make date fields management Selenium friendly 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-06-03 13:28plujanNew Issue
2011-06-03 13:28plujanAssigned To => iperdomo
2011-06-03 13:28plujanModules => Core
2011-06-03 13:38jpabloaeIssue Monitored: jpabloae
2011-06-06 10:21iperdomoStatusnew => scheduled
2011-06-06 10:21iperdomoAssigned Toiperdomo => mtaal
2011-06-06 10:21iperdomofix_in_branch => pi
2011-06-06 10:22hgbotCheckin
2011-06-06 10:22hgbotNote Added: 0038016
2011-06-06 15:53plujanNote Added: 0038047
2011-06-06 17:13elopioRelationship addedhas duplicate 0017499
2011-06-06 18:53hgbotCheckin
2011-06-06 18:53hgbotNote Added: 0038067
2011-06-07 03:56hudsonbotCheckin
2011-06-07 03:56hudsonbotNote Added: 0038072
2011-06-07 08:34plujanNote Added: 0038081
2011-06-07 08:34plujanStatusscheduled => closed
2011-06-07 08:34plujanResolutionopen => no change required
2011-06-08 13:13plujanRelationship addedrelated to 0017560

Notes
(0038016)
hgbot   
2011-06-06 10:22   
Repository: erp/devel/pi
Changeset: 6f28c4fd4d65de0a4d5f384f6f9357e94d70feb9
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Jun 06 10:21:18 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/6f28c4fd4d65de0a4d5f384f6f9357e94d70feb9 [^]

Related to issue 17493: Selenium tests are broken because of a change in the Date fields behaviour

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-date.js
---
(0038047)
plujan   
2011-06-06 15:53   
I've implemented a workaround from automation code. I'll push it to development branch and if it works, I'll promote it to the stable branch.
(0038067)
hgbot   
2011-06-06 18:53   
Repository: tools/automation/int
Changeset: b3d7087a0fb1a65622e33e5497e6cf4e109d41b4
Author: Pablo Lujan <pablo.lujan <at> openbravo.com>
Date: Mon Jun 06 15:54:59 2011 +0200
URL: http://code.openbravo.com/tools/automation/int/rev/b3d7087a0fb1a65622e33e5497e6cf4e109d41b4 [^]

Related to issue 17493: Works around issue with Date fields

---
M src-test/com/openbravo/test/integration/erp/modules/userinterface/smartclient/gui/DateItem.java
M src-test/com/openbravo/test/integration/erp/modules/userinterface/smartclient/gui/TextItem.java
---
(0038072)
hudsonbot   
2011-06-07 03:56   
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/aefaedafb236 [^]

Maturity status: Test
(0038081)
plujan   
2011-06-07 08:34   
A fix was made in the Selenium code, so no extra change in the ERP required.