added File::realpath()

Author: codemasher

src/Directory.php

@@ -19,7 +19,6 @@
 use function is_readable;
 use function is_writable;
 use function mkdir;
-use function realpath;
 use function rmdir;
 use function sprintf;
 use function trim;
@@ -78,16 +77,9 @@ public static function create(string $dir, int $permissions = 0o777, bool $recur
 			throw new RuntimeException(sprintf('could not create directory: %s', $dir)); // @codeCoverageIgnore
 		}
 
-		$dir = realpath($dir);
-
-		// reaplpath error
-		if(!$dir){
-			throw new RuntimeException('invalid directory (realpath)'); // @codeCoverageIgnore
-		}
-
 		clearstatcache();
 
-		return $dir;
+		return File::realpath($dir);
 	}
 
 	/**

src/File.php

@@ -11,6 +11,7 @@
 
 namespace chillerlan\Utilities;
 
+use InvalidArgumentException;
 use RuntimeException;
 use function clearstatcache;
 use function dirname;
@@ -55,11 +56,29 @@ public static function isWritable(string $file):bool{
 		return self::exists($file) && is_writable($file);
 	}
 
+	/**
+	 * Returns the absolute real path to the given file or directory
+	 *
+	 * @codeCoverageIgnore
+	 */
+	public static function realpath(string $path):string{
+		$realpath = realpath($path);
+
+		if($realpath === false){
+			throw new InvalidArgumentException('invalid file path');
+		}
+
+		return $realpath;
+	}
+
+	/**
+	 * Deletes a file
+	 */
 	public static function delete(string $file):bool{
-		$file = realpath($file);
+		$file = self::realpath($file);
 
-		if($file === false || !self::isWritable($file)){
-			throw new RuntimeException('cannot read the given file');
+		if(!self::isWritable($file)){
+			throw new RuntimeException('cannot read the given file'); // @codeCoverageIgnore
 		}
 
 		if(!unlink($file)){
@@ -79,10 +98,10 @@ public static function delete(string $file):bool{
 	 * @throws \RuntimeException
 	 */
 	public static function load(string $file, int $offset = 0, int|null $length = null):string{
-		$file = realpath($file);
+		$file = self::realpath($file);
 
-		if($file === false || !self::isReadable($file)){
-			throw new RuntimeException('cannot read the given file');
+		if(!self::isReadable($file)){
+			throw new RuntimeException('cannot read the given file'); // @codeCoverageIgnore
 		}
 
 		$content = file_get_contents(filename: $file, offset: $offset, length: $length);

tests/DirAndFileTest.php

@@ -54,8 +54,8 @@ public function loadFile():void{
 
 	#[Test]
 	public function loadInvalidFileException():void{
-		$this->expectException(RuntimeException::class);
-		$this->expectExceptionMessage('cannot read the given file');
+		$this->expectException(InvalidArgumentException::class);
+		$this->expectExceptionMessage('invalid file path');
 
 		File::load(self::invalidFile);
 	}
@@ -68,8 +68,8 @@ public function deleteFile():void{
 
 	#[Test]
 	public function deleteInvalidFileException():void{
-		$this->expectException(RuntimeException::class);
-		$this->expectExceptionMessage('cannot read the given file');
+		$this->expectException(InvalidArgumentException::class);
+		$this->expectExceptionMessage('invalid file path');
 
 		File::delete(self::invalidFile);
 	}