Laszlo Hornyak has uploaded a new change for review.

Change subject: engine: fix for failing VdsStaticDAOTest
......................................................................

engine: fix for failing VdsStaticDAOTest

null password fields are swapped as empty strings by the encryption
utils, after reading it it will be empty string rahter than null.

Change-Id: Ie340243c3eb3f9b909675b33017dcf501d4072fe
Signed-off-by: Laszlo Hornyak <[email protected]>
---
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDAOTest.java
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/79/16179/1

diff --git 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDAOTest.java
 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDAOTest.java
index 8727903..5a210d3 100644
--- 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDAOTest.java
+++ 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VdsStaticDAOTest.java
@@ -35,6 +35,9 @@
         newStaticVds.setVdsGroupId(existingVds.getVdsGroupId());
         
newStaticVds.setSSHKeyFingerprint("b5:ad:16:19:06:9f:b3:41:69:eb:1c:42:1d:12:b5:31");
         newStaticVds.setPmSecondaryOptionsMap(new ValueObjectMap());
+        //null values are not correctly saved to DB
+        newStaticVds.setPmPassword("");
+        newStaticVds.setPmSecondaryPassword("");
     }
 
     /**


-- 
To view, visit http://gerrit.ovirt.org/16179
To unsubscribe, visit http://gerrit.ovirt.org/settings

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

Reply via email to