added Finder::files() & directories()

Author: dg

src/Utils/Finder.php

@@ -72,21 +72,38 @@ public static function findDirectories(string|array $masks): static
 
 
 	/**
-	 * Creates filtering group by mask & type selector.
+	 * Finds files matching the specified masks.
 	 */
-	private function select(array $masks, string $type): static
+	public function files(string|array $masks): static
+	{
+		return $this->select((array) $masks, 'file');
+	}
+
+
+	/**
+	 * Finds directories matching the specified masks.
+	 */
+	public function directories(string|array $masks): static
+	{
+		return $this->select((array) $masks, 'dir');
+	}
+
+
+	private function select(array $masks, string $mode): static
 	{
 		foreach ($masks as $mask) {
 			$mask = FileSystem::unixSlashes($mask);
-			if ($mask === '') {
+			if ($mode === 'dir') {
+				$mask = rtrim($mask, '/');
+			}
+			if ($mask === '' || ($mode === 'file' && str_ends_with($mask, '/'))) {
 				throw new Nette\InvalidArgumentException("Invalid mask '$mask'");
 			}
 			if (str_starts_with($mask, '**/')) {
 				$mask = substr($mask, 3);
 			}
-			$this->find[] = [$mask, $type];
+			$this->find[] = [$mask, $mode];
 		}
-
 		return $this;
 	}
 

tests/Utils/Finder.basic.phpt

@@ -54,6 +54,12 @@ test('non-recursive file search', function () {
 });
 
 
+test('non-recursive file search alt', function () {
+	$finder = (new Finder)->files('file.txt')->in('fixtures.finder');
+	Assert::same(['fixtures.finder/file.txt'], export($finder));
+});
+
+
 test('recursive file search', function () {
 	$finder = Finder::findFiles('file.txt')->from('fixtures.finder');
 	Assert::same([
@@ -117,6 +123,14 @@ test('non-recursive directory search', function () {
 });
 
 
+test('non-recursive directory search alt', function () {
+	$finder = (new Finder)->directories('subdir*')->in('fixtures.finder');
+	Assert::same([
+		'fixtures.finder/subdir',
+	], export($finder));
+});
+
+
 test('recursive directory search', function () {
 	$finder = Finder::findDirectories('subdir*')->from('fixtures.finder');
 	Assert::same([