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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Itamar Heim <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Shahar Havivi <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to