Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0036938 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] Y. DBSourceManager | minor | have not tried | 2017-09-25 11:15 | 2018-01-03 12:38 | |||
Reporter | alostale | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | 3.0PR18Q1 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 15b45f710a6e | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | caristu | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0036938: dbsm test: src changes not applied | |||||||
Description | When dbsm test cases are executed, modifications in src data are not correctly applied. | |||||||
Steps To Reproduce | 1. Execute org.openbravo.dbsm.test.sourcedata.SourcedataUpdates.allChangesInARowAreExecutedAltogether() -> It fails: java.lang.AssertionError: updated value in test.col1 Expected: is "v2" but: was "v1" | |||||||
Proposed Solution | Test infrastructure should be refactored to reuse actual production code rather than maintaining an ad-hoc copy of it. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|||||||||||||||
|
![]() |
|
(0099530) hgbot (developer) 2017-09-25 11:23 |
Repository: erp/devel/dbsm-main Changeset: 5711f00fc2938fee4b0cf72edc53ac3e2a68323c Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Mon Sep 25 11:00:49 2017 +0200 URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/5711f00fc2938fee4b0cf72edc53ac3e2a68323c [^] related to bug 36938: src changes not applied in dbsm tests Temporaryly disabled test case that relies in this functionality. --- M src-test/src/org/openbravo/dbsm/test/sourcedata/SourcedataUpdates.java --- |
(0099708) hgbot (developer) 2017-10-03 08:14 |
Repository: erp/devel/dbsm-main Changeset: 3a727b10ea614c008910c737983459d4996254cb Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Tue Oct 03 07:38:23 2017 +0200 URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/3a727b10ea614c008910c737983459d4996254cb [^] related to bug 36984, related to bug 36938: added test case for 36984, disabled due to 36938 --- M src-test/src/org/openbravo/dbsm/test/sourcedata/SourcedataUpdates.java A src-test/data/datachanges1/v1/TEST.xml A src-test/data/datachanges1/v2/TEST.xml A src-test/model/constraints/SIMPLE_UNIQUE.xml --- |
(0099719) hgbot (developer) 2017-10-03 14:07 |
Repository: erp/devel/pi Changeset: 900d0380bca96341fd0063d860b3c2ef541d877d Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Tue Oct 03 13:50:17 2017 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/900d0380bca96341fd0063d860b3c2ef541d877d [^] fixed issue 36938, fixed issue 36996: dbsm for these issues - removed unused platforms - refactorized code to better reuse in tests - test properly apply ad updates --- M src-db/database/lib/dbsourcemanager.jar --- |
(0099721) hgbot (developer) 2017-10-03 14:07 |
Repository: erp/devel/dbsm-main Changeset: 15b45f710a6e86faf4c09f74ce560f0ba76394ee Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Tue Sep 26 16:27:14 2017 +0200 URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/15b45f710a6e86faf4c09f74ce560f0ba76394ee [^] fixed bug 36938: dbsm test: src changes not applied AD data was only inserted when it was new, but no changes were detected. This was caused because dataset relied on includeAllColumn preference, which is actually used only internally to add columns. Fixed including explicitilly all column when building manual datasets for testing. --- M src/org/openbravo/ddlutils/process/DBUpdater.java --- |
(0099722) hgbot (developer) 2017-10-03 14:08 |
Repository: erp/devel/dbsm-main Changeset: 75310c45131b2264c04b4e6c77f05701ff3175aa Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Tue Sep 26 16:30:44 2017 +0200 URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/75310c45131b2264c04b4e6c77f05701ff3175aa [^] related to bug 36938: disable some test cases When an AD column with default value but without on create default is created, data exported without this column will not get this default value (it would in case on create default is defined). It was wrongly tested the opposite behavior (default value is applied for AD data which is missing). This was not failing before because AD data was not being updated. Disabled now test case for AD, but kept for instance data. --- M src-test/src/org/openbravo/dbsm/test/model/data/CreateDefault.java --- |
(0099723) hgbot (developer) 2017-10-03 14:08 |
Repository: erp/devel/dbsm-main Changeset: d008499bc0cd33a5d975f930babef96c50a473ea Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Tue Sep 26 16:31:10 2017 +0200 URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/d008499bc0cd33a5d975f930babef96c50a473ea [^] related to bug 36938: enable temporaryly disabled test case --- M src-test/src/org/openbravo/dbsm/test/sourcedata/SourcedataUpdates.java --- |
(0099725) hgbot (developer) 2017-10-03 14:09 |
Repository: erp/devel/dbsm-main Changeset: 0a082650b68deb2338c2212d4d0e5455bd2ae7d3 Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Tue Oct 03 09:03:30 2017 +0200 URL: http://code.openbravo.com/erp/devel/dbsm-main/rev/0a082650b68deb2338c2212d4d0e5455bd2ae7d3 [^] related to bug 36984, related to bug 36938: enabled test case --- M src-test/src/org/openbravo/dbsm/test/sourcedata/SourcedataUpdates.java --- |
(0099782) caristu (viewer) 2017-10-06 08:42 |
Code reviewed[1] and tested OK. [1] https://docs.google.com/spreadsheets/d/1IWrY7VDtQj8MMQ-vZ7vgO2cY01BMUzXiGNhFMuTpcGk/edit#gid=0 [^] |
(0101309) hudsonbot (viewer) 2018-01-03 12:38 |
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/c81e0d3cbab5 [^] Maturity status: Test |
![]() |
|||
Date Modified | Username | Field | Change |
2017-09-25 11:15 | alostale | New Issue | |
2017-09-25 11:15 | alostale | Assigned To | => platform |
2017-09-25 11:15 | alostale | OBNetwork customer | => No |
2017-09-25 11:15 | alostale | Modules | => Core |
2017-09-25 11:15 | alostale | Triggers an Emergency Pack | => No |
2017-09-25 11:16 | alostale | Relationship added | related to 0035653 |
2017-09-25 11:23 | hgbot | Checkin | |
2017-09-25 11:23 | hgbot | Note Added: 0099530 | |
2017-09-28 12:32 | alostale | Status | new => scheduled |
2017-09-28 12:32 | alostale | Assigned To | platform => alostale |
2017-10-03 08:14 | hgbot | Checkin | |
2017-10-03 08:14 | hgbot | Note Added: 0099708 | |
2017-10-03 14:00 | alostale | Relationship added | duplicate of 0036996 |
2017-10-03 14:01 | alostale | Review Assigned To | => caristu |
2017-10-03 14:07 | hgbot | Checkin | |
2017-10-03 14:07 | hgbot | Note Added: 0099719 | |
2017-10-03 14:07 | hgbot | Status | scheduled => resolved |
2017-10-03 14:07 | hgbot | Resolution | open => fixed |
2017-10-03 14:07 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/900d0380bca96341fd0063d860b3c2ef541d877d [^] |
2017-10-03 14:07 | hgbot | Checkin | |
2017-10-03 14:07 | hgbot | Note Added: 0099721 | |
2017-10-03 14:07 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/900d0380bca96341fd0063d860b3c2ef541d877d [^] => http://code.openbravo.com/erp/devel/dbsm-main/rev/15b45f710a6e86faf4c09f74ce560f0ba76394ee [^] |
2017-10-03 14:08 | hgbot | Checkin | |
2017-10-03 14:08 | hgbot | Note Added: 0099722 | |
2017-10-03 14:08 | hgbot | Checkin | |
2017-10-03 14:08 | hgbot | Note Added: 0099723 | |
2017-10-03 14:09 | hgbot | Checkin | |
2017-10-03 14:09 | hgbot | Note Added: 0099725 | |
2017-10-06 08:42 | caristu | Note Added: 0099782 | |
2017-10-06 08:42 | caristu | Status | resolved => closed |
2017-10-06 08:42 | caristu | Fixed in Version | => 3.0PR18Q1 |
2018-01-03 12:38 | hudsonbot | Checkin | |
2018-01-03 12:38 | hudsonbot | Note Added: 0101309 |
Copyright © 2000 - 2009 MantisBT Group |