Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0036938
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Y. DBSourceManagerminorhave not tried2017-09-25 11:152018-01-03 12:38
ReporteralostaleView Statuspublic 
Assigned Toalostale 
PrioritynormalResolutionfixedFixed in Version3.0PR18Q1
StatusclosedFix in branchFixed in SCM revision15b45f710a6e
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tocaristu
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0036938: dbsm test: src changes not applied

DescriptionWhen dbsm test cases are executed, modifications in src data are not correctly applied.
Steps To Reproduce1. 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 SolutionTest infrastructure should be refactored to reuse actual production code rather than maintaining an ad-hoc copy of it.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
duplicate of feature request 0036996 closedalostale remove unused platforms from dbsm 
related to defect 0035653 closedalostale update.database can fail updating src tables with a check constraint where more than one column participates 

-  Notes
(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 (developer)
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 (developer)
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

- Issue History
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 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
Powered by Mantis Bugtracker