Unit test coverage

Change-Id: I3634ccaa60c72ee91b89ae42b8da8926b5823436
Issue-ID: SDC-1822
Signed-off-by: ayalaben <ayala.benzvi@amdocs.com>

2 files changed, 61 insertions, 48 deletions

diff --git a/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/dao/impl/PermissionsServicesImplTest.java b/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/dao/impl/PermissionsServicesImplTest.java
index 9346b0a5a5..a6baf9da19 100644
--- a/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/dao/impl/PermissionsServicesImplTest.java
+++ b/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/dao/impl/PermissionsServicesImplTest.java
@@ -1,3 +1,19 @@
+/*
+ * Copyright © 2016-2018 European Support Limited
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 package org.openecomp.sdc.itempermissions.dao.impl;
 
 import static org.mockito.Matchers.anyObject;
@@ -23,9 +39,10 @@ import org.openecomp.sdc.itempermissions.PermissionsRules;
 import org.openecomp.sdc.itempermissions.dao.ItemPermissionsDao;
 import org.openecomp.sdc.itempermissions.dao.UserPermissionsDao;
 import org.openecomp.sdc.itempermissions.type.ItemPermissionsEntity;
-import org.testng.Assert;
-import org.testng.annotations.BeforeMethod;
-import org.testng.annotations.Test;
+import org.junit.Test;
+import org.junit.Before;
+import org.junit.Assert;
+
 
 /**
  * Created by ayalaben on 7/9/2017
@@ -54,7 +71,7 @@ public class PermissionsServicesImplTest {
     private PermissionsServicesImpl permissionsServices;
 
 
-    @BeforeMethod
+    @Before
     public void setUp() {
         MockitoAnnotations.initMocks(this);
     }
@@ -91,32 +108,29 @@ public class PermissionsServicesImplTest {
     @Test
     public void testGetUserItemPermission() {
         Mockito.when(permissionsDaoMock.getUserItemPermission(anyString(), anyString()))
-                .thenReturn(Optional.of(PERMISSION));
+               .thenReturn(Optional.of(PERMISSION));
 
         Optional<String> permission = permissionsServices.getUserItemPermission(ITEM1_ID, USER1_ID);
 
         Assert.assertTrue(permission.isPresent());
-        Assert.assertEquals(permission.get(), PERMISSION);
+        Assert.assertEquals(PERMISSION, permission.get());
     }
 
     @Test
     public void testListItemPermissionsWhenNone() {
-        Collection<ItemPermissionsEntity> permissions =
-                permissionsServices.listItemPermissions(ITEM1_ID);
-        Assert.assertEquals(permissions.size(), 0);
+        Collection<ItemPermissionsEntity> permissions = permissionsServices.listItemPermissions(ITEM1_ID);
+        Assert.assertEquals( 0, permissions.size());
     }
 
 
     @Test
     public void testListItemPermissions() {
-        doReturn(Arrays.asList(
-                createPermissionEntity(ITEM1_ID, USER1_ID, PERMISSION),
-                createPermissionEntity(ITEM1_ID, USER2_ID, PERMISSION)))
-                .when(permissionsDaoMock).listItemPermissions(anyObject());
-
-        Collection<ItemPermissionsEntity> actual =
-                permissionsServices.listItemPermissions(ITEM1_ID);
-        Assert.assertEquals(actual.size(), 2);
+        doReturn(Arrays.asList(createPermissionEntity(ITEM1_ID, USER1_ID, PERMISSION),
+                createPermissionEntity(ITEM1_ID, USER2_ID, PERMISSION))).when(permissionsDaoMock)
+                                                                        .listItemPermissions(anyObject());
+
+        Collection<ItemPermissionsEntity> actual = permissionsServices.listItemPermissions(ITEM1_ID);
+        Assert.assertEquals(2, actual.size());
     }
 
 
@@ -129,10 +143,10 @@ public class PermissionsServicesImplTest {
 
         Assert.assertTrue(result);
 
-  }
+    }
 
-  @Test
-  public void shouldUpdatePermissions(){
+    @Test
+    public void shouldUpdatePermissions() {
 
         Set<String> addedUsers = new HashSet<>();
         addedUsers.add(USER2_ID);
@@ -141,33 +155,31 @@ public class PermissionsServicesImplTest {
 
         verify(permissionsRulesMock).executeAction(ITEM1_ID, USER1_ID, CHANGE_PERMISSIONS);
         verify(permissionsRulesMock).updatePermission(ITEM1_ID, USER1_ID, PERMISSION, addedUsers, new HashSet<>());
-        verify(permissionsDaoMock).updateItemPermissions(ITEM1_ID, PERMISSION, addedUsers, new
-                HashSet<>());
+        verify(permissionsDaoMock).updateItemPermissions(ITEM1_ID, PERMISSION, addedUsers, new HashSet<>());
+    }
+
+    @Test
+    public void shouldExecutePermissionRules() {
+        permissionsServices.execute(ITEM1_ID, USER1_ID, ACTION_SUBMIT);
+        verify(permissionsRulesMock).executeAction(ITEM1_ID, USER1_ID, ACTION_SUBMIT);
     }
 
-  @Test
-  public void shouldExecutePermissionRules(){
-    permissionsServices.execute(ITEM1_ID,USER1_ID,ACTION_SUBMIT);
-    verify(permissionsRulesMock).executeAction(ITEM1_ID,USER1_ID,ACTION_SUBMIT);
-  }
-
-  @Test
-  public void shouldReturnUserItemPermission(){
-    doReturn(Optional.of(PERMISSION)).when(permissionsDaoMock).getUserItemPermission(ITEM1_ID, USER1_ID);
-    Optional<String> actual = permissionsServices.getUserItemPermission(ITEM1_ID, USER1_ID);
-    Assert.assertTrue(actual.isPresent());
-    Assert.assertEquals(actual.get(), PERMISSION);
-  }
-
-  @Test
-  public void shouldDeleteItemPermissions(){
-   permissionsServices.deleteItemPermissions(ITEM1_ID);
-    verify(permissionsDaoMock).deleteItemPermissions(ITEM1_ID);
-  }
-
-
-    private static ItemPermissionsEntity createPermissionEntity(String itemId, String
-            userId, String permission) {
+    @Test
+    public void shouldReturnUserItemPermission() {
+        doReturn(Optional.of(PERMISSION)).when(permissionsDaoMock).getUserItemPermission(ITEM1_ID, USER1_ID);
+        Optional<String> actual = permissionsServices.getUserItemPermission(ITEM1_ID, USER1_ID);
+        Assert.assertTrue(actual.isPresent());
+        Assert.assertEquals(PERMISSION, actual.get());
+    }
+
+    @Test
+    public void shouldDeleteItemPermissions() {
+        permissionsServices.deleteItemPermissions(ITEM1_ID);
+        verify(permissionsDaoMock).deleteItemPermissions(ITEM1_ID);
+    }
+
+
+    private static ItemPermissionsEntity createPermissionEntity(String itemId, String userId, String permission) {
         ItemPermissionsEntity permissionsEntity = new ItemPermissionsEntity();
         permissionsEntity.setItemId(itemId);
         permissionsEntity.setUserId(userId);
diff --git a/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/servlet/PermissionsFilterTest.java b/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/servlet/PermissionsFilterTest.java
index 6bb51c30dd..fcf899245c 100644
--- a/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/servlet/PermissionsFilterTest.java
+++ b/openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org/openecomp/sdc/itempermissions/servlet/PermissionsFilterTest.java
@@ -33,8 +33,9 @@ import org.mockito.Mockito;
 import org.mockito.MockitoAnnotations;
 import org.mockito.Spy;
 import org.openecomp.sdc.itempermissions.PermissionsServices;
-import org.testng.annotations.BeforeMethod;
-import org.testng.annotations.Test;
+import org.junit.Test;
+import org.junit.Before;
+
 
 public class PermissionsFilterTest {
 
@@ -45,7 +46,7 @@ public class PermissionsFilterTest {
     @Spy
     private PermissionsFilter permissionsFilter;
 
-    @BeforeMethod
+    @Before
     public void setUp() {
         MockitoAnnotations.initMocks(this);
     }