Now throw exception when can not get permission value

Author: nguereza-tony

src/Adapter/Local/AbstractLocal.php

@@ -48,9 +48,11 @@
 namespace Platine\Filesystem\Adapter\Local;
 
 use Platine\Filesystem\Adapter\AdapterInterface;
+use Platine\Filesystem\Adapter\Local\Exception\FilesystemException;
 use Platine\Filesystem\FilesystemInterface;
 use Platine\Stdlib\Helper\Path;
 
+
 /**
  * Class AbstractLocal
  * @package Platine\Filesystem\Adapter\Local
@@ -191,7 +193,10 @@ public function getPermission(): string
     {
         $permission = fileperms($this->path);
         if ($permission === false) {
-            return '';
+            throw new FilesystemException(sprintf(
+                'Can not get permission for file [%s]',
+                $this->path
+            ));
         }
         return substr(base_convert((string) $permission, 10, 8), -4);
     }

src/Adapter/Local/Exception/FilesystemException.php

@@ -0,0 +1,44 @@
+<?php
+
+/**
+ * Platine Filesystem
+ *
+ * Platine Filesystem is file system abstraction layer extendable by adapters
+ *
+ * This content is released under the MIT License (MIT)
+ *
+ * Copyright (c) 2020 Platine Filesystem
+ * Copyright (c) 2019 Alex Sivka
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy
+ * of this software and associated documentation files (the "Software"), to deal
+ * in the Software without restriction, including without limitation the rights
+ * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+ * copies of the Software, and to permit persons to whom the Software is
+ * furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in all
+ * copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
+
+declare(strict_types=1);
+
+namespace Platine\Filesystem\Adapter\Local\Exception;
+
+use Exception;
+
+/**
+ * Class FilesystemException
+ * @package Platine\Filesystem\Adapter\Local\Exception
+ */
+class FilesystemException extends Exception
+{
+}

src/Adapter/Local/Exception/NotFoundException.php

@@ -33,12 +33,10 @@
 
 namespace Platine\Filesystem\Adapter\Local\Exception;
 
-use Exception;
-
 /**
  * Class NotFoundException
  * @package Platine\Filesystem\Adapter\Local\Exception
  */
-class NotFoundException extends Exception
+class NotFoundException extends FilesystemException
 {
 }

tests/Adapter/Local/FileTest.php

@@ -5,11 +5,12 @@
 namespace Platine\Test\Filesystem\Adapter\Local;
 
 use org\bovigo\vfs\vfsStream;
+use Platine\Dev\PlatineTestCase;
 use Platine\Filesystem\Adapter\AdapterInterface;
+use Platine\Filesystem\Adapter\Local\Exception\FilesystemException;
 use Platine\Filesystem\Adapter\Local\File;
 use Platine\Filesystem\Adapter\Local\LocalAdapter;
 use Platine\Filesystem\FileInterface;
-use Platine\Dev\PlatineTestCase;
 
 /**
  * File class tests
@@ -171,7 +172,8 @@ public function testGetPermissionFunctionfilePermsReturnFalse(): void
         );
 
         $t = new File($file->url(), $adapter);
-        $this->assertEmpty($t->getPermission());
+        $this->expectException(FilesystemException::class);
+        $t->getPermission();
     }
 
     public function testGetMimeUsingHelperToGetMimetype(): void