diff options
author | 2018-10-10 09:40:57 +0300 | |
---|---|---|
committer | 2018-10-14 10:06:00 +0300 | |
commit | 130aa901dd94519cd6bc99140a9d935365220cc5 (patch) | |
tree | b061655b21da6c133059d11e7e772d1bb9fda759 /openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org | |
parent | 7d05e16f0ab24ba8918bd4466d50a9fb77571552 (diff) |
Unit test coverage
Change-Id: I3634ccaa60c72ee91b89ae42b8da8926b5823436
Issue-ID: SDC-1822
Signed-off-by: ayalaben <ayala.benzvi@amdocs.com>
Diffstat (limited to 'openecomp-be/lib/openecomp-item-permissions-lib/openecomp-item-permissions-core/src/test/java/org')
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); } |