[BUG]bug-npe-is-thrown-when-creating-an-application-without-a-group

Author: fjr3o6

sugoi-api-file-store-provider/src/main/java/fr/insee/sugoi/store/file/FileWriterStore.java

@@ -27,6 +27,7 @@
 import fr.insee.sugoi.model.User;
 import fr.insee.sugoi.model.exceptions.ApplicationNotFoundException;
 import fr.insee.sugoi.model.exceptions.GroupAlreadyExistException;
+import fr.insee.sugoi.model.exceptions.NoGroupException;
 import fr.insee.sugoi.model.exceptions.UserNotFoundException;
 import java.io.File;
 import java.io.FileWriter;
@@ -119,6 +120,9 @@ public ProviderResponse deleteGroup(
   @Override
   public ProviderResponse createGroup(
       String appName, Group appGroup, ProviderRequest providerRequest) {
+    if (appGroup == null) {
+      throw new NoGroupException("no group provided");
+    }
     fileReaderStore.setResourceLoader(resourceLoader);
     Application application =
         fileReaderStore

sugoi-api-file-store-provider/src/test/java/fr/insee/sugoi/store/file/FileWriterStoreTest.java

@@ -15,9 +15,11 @@
 
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.*;
+import static org.junit.jupiter.api.Assertions.assertThrows;
 
 import com.fasterxml.jackson.databind.ObjectMapper;
 import fr.insee.sugoi.model.*;
+import fr.insee.sugoi.model.exceptions.NoGroupException;
 import java.io.FileWriter;
 import java.io.IOException;
 import java.util.ArrayList;
@@ -358,6 +360,17 @@ public void testCreateGroup() {
         is("Groupy_Applitest"));
   }
 
+  @Test
+  public void testCreateGroupWithNoGroup() {
+    Group group = null;
+    assertThrows(
+        NoGroupException.class,
+        () -> {
+          fileWriterStore.createGroup("Applitest", group, null);
+        },
+        "Should throw NoGroupException");
+  }
+
   @Test
   public void testDeleteGroup() {
     Group group = new Group("Asupprimer_WebServicesLdap", "WebServicesLdap");

sugoi-api-ldap-store-provider/src/main/java/fr/insee/sugoi/store/ldap/LdapWriterStore.java

@@ -51,6 +51,7 @@
 import fr.insee.sugoi.model.exceptions.GroupAlreadyExistException;
 import fr.insee.sugoi.model.exceptions.GroupNotFoundException;
 import fr.insee.sugoi.model.exceptions.InvalidPasswordException;
+import fr.insee.sugoi.model.exceptions.NoGroupException;
 import fr.insee.sugoi.model.exceptions.OrganizationAlreadyExistException;
 import fr.insee.sugoi.model.exceptions.OrganizationNotFoundException;
 import fr.insee.sugoi.model.exceptions.StoragePolicyNotMetException;
@@ -213,6 +214,9 @@ public ProviderResponse createGroup(
       String appName, Group group, ProviderRequest providerRequest) {
 
     try {
+      if (group == null) {
+        throw new NoGroupException("no group provided");
+      }
       if (ldapReaderStore.getApplication(appName).isEmpty()) {
         throw new ApplicationNotFoundException(appName);
       }

sugoi-api-ldap-store-provider/src/test/java/fr/insee/sugoi/ldap/LdapWriterStoreTest.java

@@ -14,17 +14,27 @@
 package fr.insee.sugoi.ldap;
 
 import static org.hamcrest.MatcherAssert.assertThat;
-import static org.hamcrest.Matchers.*;
+import static org.hamcrest.Matchers.is;
+import static org.hamcrest.Matchers.not;
+import static org.hamcrest.Matchers.nullValue;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 
 import fr.insee.sugoi.core.configuration.GlobalKeysConfig;
 import fr.insee.sugoi.core.model.ProviderRequest;
 import fr.insee.sugoi.core.model.ProviderResponse;
 import fr.insee.sugoi.core.model.ProviderResponse.ProviderResponseStatus;
 import fr.insee.sugoi.ldap.utils.config.LdapConfigKeys;
-import fr.insee.sugoi.model.*;
+import fr.insee.sugoi.model.Application;
+import fr.insee.sugoi.model.Group;
+import fr.insee.sugoi.model.Habilitation;
+import fr.insee.sugoi.model.Organization;
+import fr.insee.sugoi.model.PostalAddress;
+import fr.insee.sugoi.model.Realm;
+import fr.insee.sugoi.model.User;
+import fr.insee.sugoi.model.UserStorage;
 import fr.insee.sugoi.model.exceptions.ApplicationNotFoundException;
 import fr.insee.sugoi.model.exceptions.InvalidPasswordException;
+import fr.insee.sugoi.model.exceptions.NoGroupException;
 import fr.insee.sugoi.model.exceptions.StoragePolicyNotMetException;
 import fr.insee.sugoi.model.fixtures.StoreMappingFixture;
 import fr.insee.sugoi.store.ldap.LdapReaderStore;
@@ -383,6 +393,17 @@ public void testCreateGroup() {
         is("Groupy_Applitest"));
   }
 
+  @Test
+  public void testCreateGroupWithNoGroup() {
+    Group group = null;
+    assertThrows(
+        NoGroupException.class,
+        () -> {
+          ldapWriterStore.createGroup("Applitest", group, null);
+        },
+        "Should throw NoGroupException");
+  }
+
   @Test
   void testCreateGroupApplicationNotFound() {
     Group group = new Group();

sugoi-api-model/src/main/java/fr/insee/sugoi/model/exceptions/NoGroupException.java

@@ -0,0 +1,27 @@
+/*
+* 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 fr.insee.sugoi.model.exceptions;
+
+public class NoGroupException extends BadRequestException {
+
+  public NoGroupException(String message, Throwable cause) {
+    super(message, cause);
+    // TODO Auto-generated constructor stub
+  }
+
+  public NoGroupException(String message) {
+    super(message);
+    // TODO Auto-generated constructor stub
+  }
+}