Allon Mureinik has uploaded a new change for review.

Change subject: core: AsyncTasksDAOTest: clean up comments
......................................................................

core: AsyncTasksDAOTest: clean up comments

Fixed a typo in an inline comment and removed another that was just
blatantly wrong.

Change-Id: Iced7c408393524096b5fea6e1c3be9ea2ddc28b7
Signed-off-by: Allon Mureinik <[email protected]>
---
M 
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AsyncTaskDAOTest.java
1 file changed, 1 insertion(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/20/7620/1

diff --git 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AsyncTaskDAOTest.java
 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AsyncTaskDAOTest.java
index 2104820..f615293 100644
--- 
a/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AsyncTaskDAOTest.java
+++ 
b/backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/AsyncTaskDAOTest.java
@@ -80,7 +80,7 @@
 
     @Test
     public void testGetAsyncTaskIdsByStoragePoolId() {
-        // According ot fixtures.xml , STORAGE_POOL_RHEL6_ISCSI_OTHER is the 
storage pool
+        // According to fixtures.xml , STORAGE_POOL_RHEL6_ISCSI_OTHER is the 
storage pool
         // that has storage domain associated to which has one task on it
         List<Guid> guids = 
dao.getAsyncTaskIdsByStoragePoolId(FixturesTool.STORAGE_POOL_RHEL6_ISCSI_OTHER);
         assertNotNull(guids);
@@ -101,9 +101,7 @@
     public void testGet() {
         async_tasks result = dao.get(existingAsyncTask.gettask_id());
 
-        // NOTE: There is no equals() method in async_tasks class.
         assertNotNull(result);
-
         assertEquals(existingAsyncTask, result);
     }
 


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

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