Allon Mureinik has uploaded a new change for review.

Change subject: core: OvfVmWriterTest.assertVm cleanup
......................................................................

core: OvfVmWriterTest.assertVm cleanup

Fixed the method's modifiers and removed a commented out line.

Change-Id: I65c7d797a725626adb7bd67f1f73f8084b803daf
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfVmWriterTest.java
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/58/12358/1

diff --git 
a/backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfVmWriterTest.java
 
b/backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfVmWriterTest.java
index 36e5711..bca8316 100644
--- 
a/backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfVmWriterTest.java
+++ 
b/backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ovf/OvfVmWriterTest.java
@@ -33,11 +33,10 @@
         manager = new OvfManager();
     }
 
-    private void assertVm(VM vm, VM newVm, long expectedDbGeneration) {
+    private static void assertVm(VM vm, VM newVm, long expectedDbGeneration) {
         assertEquals("imported vm is different than expected", vm, newVm);
         assertEquals("imported db generation is different than expected", 
expectedDbGeneration, newVm.getDbGeneration());
         assertEquals(vm.getStaticData(), newVm.getStaticData());
-        //assertEquals(vm.getInterfaces(), newVm.getInterfaces());
     }
 
     @Test


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I65c7d797a725626adb7bd67f1f73f8084b803daf
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