Change in ovirt-engine[master]: core: Fix broken DAO tests broken
yzaslavs at redhat.com
yzaslavs at redhat.com
Sun Jan 26 08:06:53 UTC 2014
Yair Zaslavsky has submitted this change and it was merged.
Change subject: core: Fix broken DAO tests broken
......................................................................
core: Fix broken DAO tests broken
Patch b5dbfd09c72e8f06e72a9c951ec9ad75d6ffa548 (Vm Init - new Feature)
removed the domain column from vm_static, but neglected to remove it
from all the row descriptions in fixtures.xml.
The result were a few bogus row definitions where the old description
value was interpreted as the creation_date (which is the next column)
value. Since these strings could no be cast to dates, the fixtures data
could not be written to the database, causing almost all the DAO tests
to fail.
Change-Id: If107a7c8ab009d82317239bb1c55fca82ecfadfa
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/dal/src/test/resources/fixtures.xml
1 file changed, 0 insertions(+), 3 deletions(-)
Approvals:
Allon Mureinik: Verified
Yair Zaslavsky: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/23704
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If107a7c8ab009d82317239bb1c55fca82ecfadfa
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
More information about the Engine-commits
mailing list