Add tests.

Author: mattstein

.github/workflows/release.yml

@@ -12,6 +12,8 @@ jobs:
       - uses: actions/checkout@v3
       - name: Install Composer dependencies
         uses: php-actions/composer@v6
+        with:
+          php_version: "8.2"
       # Store version number without `v`
       - name: Write release version
         run: |

.github/workflows/tests.yml

@@ -0,0 +1,28 @@
+name: Tests
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        php: [8.2, 8.3, 8.4, 8.5]
+    name: PHP ${{ matrix.php }}
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          extensions: json, curl
+      - name: Install dependencies
+        run: composer update --prefer-dist --no-interaction --no-progress ${{ matrix.php == '8.5' && '--ignore-platform-req=php' || '' }}
+      - name: Run tests
+        run: composer test

.gitignore

@@ -1,3 +1,4 @@
 .idea
 .DS_Store
-vendor/
+vendor/
+composer.lock

composer.json

@@ -7,16 +7,27 @@
   ],
   "license": "MIT",
   "require": {
-    "php": "^8.0",
+    "php": "^8.2",
     "joetannenbaum/alfred-workflow": "dev-master",
     "loilo/fuse": "^7.0"
   },
+  "require-dev": {
+    "pestphp/pest": "^3.0"
+  },
   "minimum-stability": "stable",
   "prefer-stable": true,
   "repositories": [
     {
       "type": "vcs",
       "url": "https://github.com/mattstein/alfred-workflow"
     }
-  ]
+  ],
+  "config": {
+    "allow-plugins": {
+      "pestphp/pest-plugin": true
+    }
+  },
+  "scripts": {
+    "test": "pest"
+  }
 }

composer.lock

@@ -0,0 +1,31 @@
+<?php
+
+/**
+ * Takes the comma-separated setting value of paths, cleans any leading
+ * or trailing space, expands relative home references (`~/`) into
+ * absolute paths, and joins them back together into a glob search string
+ * using braces (`{/path/one,/path/two}/*`).
+ *
+ * @param string $settingValue
+ * @param string $homePath
+ * @return string
+ */
+function buildSearchPathString(string $settingValue, string $homePath): string
+{
+    $searchPaths = array_map(static function($path) use ($homePath) {
+        $path = trim($path);
+        if (str_starts_with($path, '~/')) {
+            // Expand `~/` to full path
+            $fullHomePath = $homePath.'/';
+            return substr_replace(
+                $path,
+                $fullHomePath,
+                0,
+                strlen('~/')
+            );
+        }
+        return $path;
+    }, explode(',', $settingValue));
+
+    return '{'.implode(',', $searchPaths).'}/*';
+}

helpers.php

@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+         colors="true"
+>
+    <testsuites>
+        <testsuite name="Tests">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+</phpunit>

phpunit.xml

@@ -1,6 +1,7 @@
 <?php
 
 include 'vendor/autoload.php';
+include 'helpers.php';
 
 use Alfred\Workflows\Workflow;
 use Alfred\Workflows\ParamBuilder\Mod;
@@ -76,33 +77,3 @@
 }
 
 $workflow->output();
-
-/**
- * Takes the comma-separated setting value of paths, cleans any leading
- * or trailing space, expands relative home references (`~/`) into
- * absolute paths, and joins them back together into a glob search string
- * using braces (`{/path/one,/path/two}/*`).
- *
- * @param string $settingValue
- * @param string $homePath
- * @return string
- */
-function buildSearchPathString(string $settingValue, string $homePath): string
-{
-    $searchPaths = array_map(static function($path) use ($homePath) {
-        $path = trim($path);
-        if (str_starts_with($path, '~/')) {
-            // Expand `~/` to full path
-            $fullHomePath = $homePath.'/';
-            return substr_replace(
-                $path,
-                $fullHomePath,
-                0,
-                strlen('~/')
-            );
-        }
-        return $path;
-    }, explode(',', $settingValue));
-
-    return '{'.implode(',', $searchPaths).'}/*';
-}

search.php

@@ -0,0 +1,47 @@
+<?php
+
+require_once __DIR__.'/../helpers.php';
+
+describe('buildSearchPathString', function () {
+    it('wraps a single path in braces with glob wildcard', function () {
+        $result = buildSearchPathString('/Projects', '/Users/test');
+
+        expect($result)->toBe('{/Projects}/*');
+    });
+
+    it('joins multiple comma-separated paths', function () {
+        $result = buildSearchPathString('/Projects,/Work', '/Users/test');
+
+        expect($result)->toBe('{/Projects,/Work}/*');
+    });
+
+    it('expands tilde to home directory', function () {
+        $result = buildSearchPathString('~/Projects', '/Users/test');
+
+        expect($result)->toBe('{/Users/test/Projects}/*');
+    });
+
+    it('expands tilde in multiple paths', function () {
+        $result = buildSearchPathString('~/Projects,~/Work', '/Users/test');
+
+        expect($result)->toBe('{/Users/test/Projects,/Users/test/Work}/*');
+    });
+
+    it('handles mixed absolute and tilde paths', function () {
+        $result = buildSearchPathString('~/Projects,/var/www', '/Users/test');
+
+        expect($result)->toBe('{/Users/test/Projects,/var/www}/*');
+    });
+
+    it('trims whitespace from paths', function () {
+        $result = buildSearchPathString('  /Projects  ,  /Work  ', '/Users/test');
+
+        expect($result)->toBe('{/Projects,/Work}/*');
+    });
+
+    it('trims whitespace and expands tilde together', function () {
+        $result = buildSearchPathString('  ~/Projects  ', '/Users/test');
+
+        expect($result)->toBe('{/Users/test/Projects}/*');
+    });
+});

tests/BuildSearchPathStringTest.php

@@ -0,0 +1,14 @@
+<?php
+
+/*
+|--------------------------------------------------------------------------
+| Test Case
+|--------------------------------------------------------------------------
+|
+| The closure you provide to your test functions is always bound to a specific PHPUnit test
+| case class. By default, that class is "PHPUnit\Framework\TestCase". Of course, you may
+| need to change it using the "pest()" function to bind a different classes or traits.
+|
+*/
+
+// pest()->extend(Tests\TestCase::class)->in('Feature');