Skip to content
Snippets Groups Projects
Verified Commit e3d20785 authored by Martin Weise's avatar Martin Weise
Browse files
parent 45ab2551
No related branches found
No related tags found
1 merge request!371Hotfix the wrong user id, ref #480
...@@ -922,12 +922,12 @@ public class TableEndpointUnitTest extends AbstractUnitTest { ...@@ -922,12 +922,12 @@ public class TableEndpointUnitTest extends AbstractUnitTest {
@Test @Test
@WithAnonymousUser @WithAnonymousUser
public void findById_privateDatabasePrivateDataPublicSchemaAnonymous_fails() { public void findById_privateDatabasePrivateDataPublicSchemaAnonymous_fails() throws UserNotFoundException,
TableNotFoundException, NotAllowedException, DataServiceException, DatabaseNotFoundException,
AccessNotFoundException, QueueNotFoundException, DataServiceConnectionException {
/* test */ /* test */
assertThrows(NotAllowedException.class, () -> {
generic_findById(DATABASE_1_ID, DATABASE_1, TABLE_2_ID, TABLE_2, null, null, null); generic_findById(DATABASE_1_ID, DATABASE_1, TABLE_2_ID, TABLE_2, null, null, null);
});
} }
@Test @Test
...@@ -965,12 +965,12 @@ public class TableEndpointUnitTest extends AbstractUnitTest { ...@@ -965,12 +965,12 @@ public class TableEndpointUnitTest extends AbstractUnitTest {
@Test @Test
@WithMockUser(username = USER_4_USERNAME) @WithMockUser(username = USER_4_USERNAME)
public void findById_privateNoRole_fails() { public void findById_privateDatabasePrivateDataPrivateSchemaNoRole_succeeds() throws UserNotFoundException,
TableNotFoundException, NotAllowedException, DataServiceException, DatabaseNotFoundException,
AccessNotFoundException, QueueNotFoundException, DataServiceConnectionException {
/* test */ /* test */
assertThrows(NotAllowedException.class, () -> {
generic_findById(DATABASE_1_ID, DATABASE_1, TABLE_1_ID, TABLE_1, USER_4_PRINCIPAL, USER_4, null); generic_findById(DATABASE_1_ID, DATABASE_1, TABLE_1_ID, TABLE_1, USER_4_PRINCIPAL, USER_4, null);
});
} }
@Test @Test
......
...@@ -88,10 +88,7 @@ public class DatabaseServicePersistenceTest extends AbstractUnitTest { ...@@ -88,10 +88,7 @@ public class DatabaseServicePersistenceTest extends AbstractUnitTest {
/* test */ /* test */
final List<Database> response = databaseService.findAllAtLestReadAccess(USER_4_ID); final List<Database> response = databaseService.findAllAtLestReadAccess(USER_4_ID);
assertEquals(3, response.size()); assertEquals(0, response.size());
assertEquals(DATABASE_4, response.get(0));
assertEquals(DATABASE_3, response.get(1));
assertEquals(DATABASE_2, response.get(2));
} }
@Test @Test
...@@ -100,11 +97,10 @@ public class DatabaseServicePersistenceTest extends AbstractUnitTest { ...@@ -100,11 +97,10 @@ public class DatabaseServicePersistenceTest extends AbstractUnitTest {
/* test */ /* test */
final List<Database> response = databaseService.findAllAtLestReadAccess(USER_2_ID); final List<Database> response = databaseService.findAllAtLestReadAccess(USER_2_ID);
assertEquals(4, response.size()); assertEquals(3, response.size());
assertEquals(DATABASE_4, response.get(0)); assertEquals(DATABASE_4, response.get(0));
assertEquals(DATABASE_3, response.get(1)); assertEquals(DATABASE_2, response.get(1));
assertEquals(DATABASE_2, response.get(2)); assertEquals(DATABASE_1, response.get(2));
assertEquals(DATABASE_1, response.get(3));
} }
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment