Allon Mureinik has uploaded a new change for review.

Change subject: core: FixedGetAllDisksByVmIdQueryTest
......................................................................

core: FixedGetAllDisksByVmIdQueryTest

The test silently assumed that a disk could have a snapshot which is the
disk itself, which is an illegal state.

Since change I6bfc9c6a this state became not only (silently) illegal,
but actually caused a StackOverflowError.

This behavior is inconsequential to the test, which dones't really care
what snapshots are assossiated to the disk, as long as there are some.
This patch creates new random snapshots for the disk, instead of just
reusing the same object.

Change-Id: Ib069780c99a04317a02d5017764c15528f931846
---
M 
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetAllDisksByVmIdQueryTest.java
1 file changed, 34 insertions(+), 14 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/17/3817/1
--
To view, visit http://gerrit.ovirt.org/3817
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib069780c99a04317a02d5017764c15528f931846
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to