Added tests and workflow. Also, fixed some little bugs.

Author: izniburak

.github/workflows/run-tests.yml

@@ -0,0 +1,37 @@
+name: Tests
+
+on: [push, pull_request]
+
+jobs:
+  test:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: true
+      matrix:
+        os: [ubuntu-latest, windows-latest]
+        php: [7.2.5, 8.0]
+        stability: [prefer-stable]
+
+    name: P${{ matrix.php }} - ${{ matrix.stability }} - ${{ matrix.os }}
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php }}
+          extensions: dom, curl, libxml, mbstring, zip, pcntl, pdo, sqlite, pdo_sqlite, bcmath, soap, intl, gd, exif, iconv, imagick, fileinfo
+          coverage: none
+
+      - name: Setup problem matchers
+        run: |
+          echo "::add-matcher::${{ runner.tool_cache }}/php.json"
+          echo "::add-matcher::${{ runner.tool_cache }}/phpunit.json"
+
+      - name: Install dependencies
+        run: composer update --${{ matrix.stability }} --prefer-dist --no-interaction
+
+      - name: Execute tests
+        run: composer test

.gitignore

@@ -5,4 +5,5 @@
 /index.php
 /.htaccess
 .idea
+.phpunit.result.cache
 

composer.json

@@ -35,6 +35,11 @@
       "Buki\\Tests\\": "tests/"
     }
   },
+  "scripts": {
+    "test": "vendor/bin/phpunit",
+    "coverage": "vendor/bin/phpunit --coverage-html coverage",
+    "dev": "cd tests/Example && php -S 127.0.0.1:8000 -t ./"
+  },
   "minimum-stability": "dev",
   "prefer-stable": true
 }

src/RouterCommand.php

@@ -360,7 +360,7 @@ protected function resolveMiddleware(string $middleware)
      */
     public function sendResponse($response)
     {
-        if (is_array($response) || strpos($this->request->headers->get('Accept'), 'application/json') !== false) {
+        if (is_array($response) || strpos($this->request->headers->get('Accept') ?? '', 'application/json') !== false) {
             $this->response->headers->set('Content-Type', 'application/json');
             return $this->response
                 ->setContent($response instanceof Response ? $response->getContent() : json_encode($response))

tests/example/.htaccess tests/Example/.htaccesstests/example/.htaccess renamed to tests/Example/.htaccess

@@ -1,5 +1,5 @@
 RewriteEngine On
-RewriteBase /tests
+RewriteBase /tests/Example
 RewriteCond %{REQUEST_FILENAME} !-f
 RewriteCond %{REQUEST_FILENAME} !-d
 RewriteRule ^ index.php [QSA,L]

tests/Example/Controllers/TestController.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Buki\Tests\Example\Controllers;
+
+use Buki\Router\Http\Controller;
+use Symfony\Component\HttpFoundation\Response;
+
+class TestController extends Controller
+{
+    public function main(): string
+    {
+        return 'controller route';
+    }
+
+    public function foo(Response $response): Response
+    {
+        $response->setContent('Foo in TestController!');
+
+        return $response;
+    }
+}

tests/Example/Middlewares/TestMiddleware.php

@@ -0,0 +1,13 @@
+<?php
+
+namespace Buki\Tests\Example\Middlewares;
+
+use Buki\Router\Http\Middleware;
+
+class TestMiddleware extends Middleware
+{
+    public function handle(): bool
+    {
+        return true;
+    }
+}

tests/Example/index.php

@@ -0,0 +1,28 @@
+<?php
+
+require __DIR__ . '/../../vendor/autoload.php';
+
+$params = [
+    'paths' => [
+        'controllers' => __DIR__ . '/Controllers',
+        'middlewares' => __DIR__ . '/Middlewares',
+    ],
+    'namespaces' => [
+        'controllers' => 'Buki\\Tests\\Example\\Controllers',
+        'middlewares' => 'Buki\\Tests\\Example\\Middlewares',
+    ],
+    'base_folder' => __DIR__,
+    'main_method' => 'main',
+];
+
+$router = new \Buki\Router\Router($params);
+
+$router->get('/', function() {
+    return 'Hello World!';
+});
+
+$router->get('/test', 'TestController@main');
+
+$router->controller('/controller', 'TestController');
+
+$router->run();