fix: sonar issue and coverage

Author: fugerit79

fj-doc-maven-plugin/src/main/java/org/fugerit/java/doc/project/facade/FeatureFacade.java

@@ -24,11 +24,11 @@ private FeatureFacade() {}
      * Checks that the given file is inside the baseFolder after normalization.
      * Throws IOException if not.
      */
-    private static void checkIfInBaseFolder(File baseFolder, File file) throws IOException {
+    public static void checkIfInBaseFolder(File baseFolder, File file) throws IOException {
         Path base = baseFolder.getCanonicalFile().toPath().normalize();
         Path target = file.getCanonicalFile().toPath().normalize();
         if (!target.startsWith(base)) {
-            throw new IOException("File path " + file.getCanonicalPath() + " is not within permitted base folder " + baseFolder.getCanonicalPath());
+            throw new IOException( String.format( "File path %s is not within permitted base folder %s", file.getCanonicalPath(), baseFolder.getCanonicalPath() ) );
         }
     }
 
@@ -58,9 +58,9 @@ protected static void insureParent( File file ) throws IOException {
             if (baseFolder != null) {
                 checkIfInBaseFolder(baseFolder, parentFile);
             }
-        }
-        if ( !parentFile.exists() ) {
-            log.info( "creates parent directory {}, mkdirs:? {}", parentFile.getCanonicalPath(), parentFile.mkdirs() );
+            if ( !parentFile.exists() ) {
+                log.info( "creates parent directory {}, mkdirs:? {}", parentFile.getCanonicalPath(), parentFile.mkdirs() );
+            }
         }
     }
 

fj-doc-maven-plugin/src/test/java/test/org/fugerit/java/doc/project/facade/TestFeatureFacade.java

@@ -0,0 +1,27 @@
+package test.org.fugerit.java.doc.project.facade;
+
+import org.fugerit.java.doc.project.facade.FeatureFacade;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
+
+import java.io.File;
+import java.io.IOException;
+
+class TestFeatureFacade {
+
+    @Test
+    void checkIfInBaseFolderTestOk() throws IOException {
+        File baseFolder = new File( "." );
+        File file = new File( "pom.xml" );
+        FeatureFacade.checkIfInBaseFolder( baseFolder, file );
+        Assertions.assertTrue( file.exists() );
+    }
+
+    @Test
+    void checkIfInBaseFolderTestKo() {
+        File baseFolder = new File( "fj-doc-base" );
+        File file = new File( "pom.xml" );
+        Assertions.assertThrows( IOException.class, () -> FeatureFacade.checkIfInBaseFolder( baseFolder, file ) );
+    }
+
+}