fix test due to ACL update in commons

Author: Leopold-Cramer

organization/src/integrationTest/kotlin/com/cosmotech/organization/service/OrganizationServiceIntegrationTest.kt

@@ -361,6 +361,7 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
                   component = "organization",
                   roles =
                       mutableMapOf(
+                          ROLE_NONE to mutableListOf<String>(),
                           ROLE_VIEWER to mutableListOf(PERMISSION_READ),
                           ROLE_USER to
                               mutableListOf(
@@ -386,6 +387,7 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
                   component = "workspace",
                   roles =
                       mutableMapOf(
+                          ROLE_NONE to mutableListOf<String>(),
                           ROLE_VIEWER to mutableListOf(PERMISSION_READ),
                           ROLE_USER to
                               mutableListOf(
@@ -411,6 +413,7 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
                   component = "runner",
                   roles =
                       mutableMapOf(
+                          ROLE_NONE to mutableListOf<String>(),
                           ROLE_VIEWER to mutableListOf(PERMISSION_READ),
                           ROLE_EDITOR to
                               mutableListOf(
@@ -744,19 +747,6 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
       }
     }
 
-    @Test
-    fun `createOrganizationAccessControl as resource admin (ROLE_NONE)`() {
-      assertThrows<CsmClientException> {
-        val name = "o-connector-test-1"
-        val organizationRegistered =
-            organizationApiService.createOrganization(makeSimpleOrganizationCreateRequest(name))
-
-        val otherUserACL = OrganizationAccessControl(id = OTHER_TEST_USER_ID, role = ROLE_NONE)
-        organizationApiService.createOrganizationAccessControl(
-            organizationRegistered.id, otherUserACL)
-      }
-    }
-
     @Test
     fun `createOrganizationAccessControl as not resource admin and PERMISSION_WRITE_SECURITY`() {
       assertDoesNotThrow {
@@ -1329,6 +1319,7 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
                 component = "organization",
                 roles =
                     mutableMapOf(
+                        ROLE_NONE to mutableListOf<String>(),
                         ROLE_VIEWER to mutableListOf(PERMISSION_READ),
                         ROLE_USER to
                             mutableListOf(
@@ -1354,6 +1345,7 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
                 component = "workspace",
                 roles =
                     mutableMapOf(
+                        ROLE_NONE to mutableListOf<String>(),
                         ROLE_VIEWER to mutableListOf(PERMISSION_READ),
                         ROLE_USER to
                             mutableListOf(
@@ -1379,6 +1371,7 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
                 component = "runner",
                 roles =
                     mutableMapOf(
+                        ROLE_NONE to mutableListOf<String>(),
                         ROLE_VIEWER to mutableListOf(PERMISSION_READ),
                         ROLE_EDITOR to
                             mutableListOf(
@@ -1771,19 +1764,6 @@ class OrganizationServiceIntegrationTest : CsmTestBase() {
     }
   }
 
-  @Test
-  fun `createOrganizationAccessControl as resource admin (ROLE_NONE)`() {
-    assertThrows<CsmClientException> {
-      val name = "o-connector-test-1"
-      val organizationRegistered =
-          organizationApiService.createOrganization(makeSimpleOrganizationCreateRequest(name))
-
-      val otherUserACL = OrganizationAccessControl(id = OTHER_TEST_USER_ID, role = ROLE_NONE)
-      organizationApiService.createOrganizationAccessControl(
-          organizationRegistered.id, otherUserACL)
-    }
-  }
-
   @Test
   fun `createOrganizationAccessControl as not resource admin and PERMISSION_WRITE_SECURITY`() {
     assertDoesNotThrow {

organization/src/integrationTest/kotlin/com/cosmotech/organization/service/OrganizationServiceRBACTest.kt

@@ -425,11 +425,11 @@ class OrganizationServiceRBACTest : CsmTestBase() {
   @TestFactory
   fun `test RBAC listOrganizationSecurityUsers`() =
       mapOf(
+              ROLE_NONE to true,
               ROLE_VIEWER to true,
               ROLE_EDITOR to false,
               ROLE_VALIDATOR to true,
               ROLE_USER to false,
-              ROLE_NONE to true,
               ROLE_ADMIN to false,
           )
           .map { (role, shouldThrow) ->