From 130aa901dd94519cd6bc99140a9d935365220cc5 Mon Sep 17 00:00:00 2001 From: ayalaben Date: Wed, 10 Oct 2018 09:40:57 +0300 Subject: Unit test coverage Change-Id: I3634ccaa60c72ee91b89ae42b8da8926b5823436 Issue-ID: SDC-1822 Signed-off-by: ayalaben --- .../dao/impl/PermissionsServicesImplTest.java | 102 ++++++++++++--------- .../servlet/PermissionsFilterTest.java | 7 +- 2 files changed, 61 insertions(+), 48 deletions(-) (limited to 'openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src') 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 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 permissions = - permissionsServices.listItemPermissions(ITEM1_ID); - Assert.assertEquals(permissions.size(), 0); + Collection 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 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 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 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 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 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); } -- cgit 1.2.3-korg