Test plugin extension logic

Author: bezin

classes/PhpBlueprintLoader.php

@@ -13,7 +13,7 @@ public static function loadFromFolder(string $dir): array
             $name             = basename($path, '.php');
             $parentDirectory  = basename(dirname($path));
             $key              = $parentDirectory === 'blueprints' ? $name : $parentDirectory . '/' . $name;
-            $blueprints[$key] = require $path;
+            $blueprints[$key] = $path;
         }
 
         return $blueprints;

composer.json

@@ -19,7 +19,8 @@
   },
   "autoload": {
     "psr-4": {
-      "PresProg\\MyPlugin\\": "classes/"
+      "PresProg\\PhpBlueprintLoader\\": "classes/",
+      "Tests\\PresProg\\PhpBlueprintLoader\\": "tests/"
     }
   },
   "config": {
@@ -29,7 +30,8 @@
     "optimize-autoloader": true
   },
   "extra": {
-    "installer-name": "php-blueprint-loader"
+    "installer-name": "php-blueprint-loader",
+    "kirby-cms-path": false
   },
   "scripts": {
     "analyze": [

index.php

@@ -3,6 +3,8 @@
 use Kirby\Cms\App;
 use PresProg\PhpBlueprintLoader\PhpBlueprintLoader;
 
+@include_once __DIR__ . '/vendor/autoload.php';
+
 App::plugin('presprog/php-blueprint-loader', [
     'hooks' => [
         'system.loadPlugins:after' => function () {

tests/.gitkeep

@@ -2,7 +2,6 @@
 
 namespace Tests\PresProg\PhpBlueprintLoader;
 
-use PHPUnit\Framework\TestCase;
 use PresProg\PhpBlueprintLoader\PhpBlueprintLoader;
 
 class PhpBlueprintLoaderTest extends TestCase
@@ -11,14 +10,11 @@ public function testLoadsPhpBlueprintsFromFolder(): void
     {
         $blueprintDir = __DIR__ . '/fixtures/site/blueprints';
 
-        $pageBlueprint = require  $blueprintDir . '/pages/php-blueprint.php';
-        $fileBlueprint = require  $blueprintDir . '/files/php-blueprint.php';
-
         $blueprints = PhpBlueprintLoader::loadFromFolder($blueprintDir);
 
         $this->assertEquals([
-            'pages/php-blueprint' => $pageBlueprint,
-            'files/php-blueprint' => $fileBlueprint,
+            'pages/blueprint' => $blueprintDir . '/pages/blueprint.php',
+            'files/blueprint' => $blueprintDir . '/files/blueprint.php',
         ], $blueprints);
     }
 }

tests/PhpBlueprintLoaderTest.php

@@ -0,0 +1,27 @@
+<?php declare(strict_types=1);
+
+namespace Tests\PresProg\PhpBlueprintLoader;
+
+use Kirby\Cms\App;
+
+class PluginTest extends TestCase
+{
+    public function testExtendsBlueprintsOnLoad(): void
+    {
+        $blueprintDir  = __DIR__ . '/temp/site/blueprints';
+
+        // Load plugin
+        require dirname(__DIR__) . '/index.php';
+
+        $kirby = new App([
+            'roots' => [
+                'index' => self::$tmpDir,
+            ],
+        ]);
+
+        $blueprints = $kirby->extensions('blueprints');
+
+        $this->assertEquals($blueprints['pages/blueprint'], $blueprintDir . '/pages/blueprint.php');
+        $this->assertEquals($blueprints['files/blueprint'], $blueprintDir . '/files/blueprint.php');
+    }
+}

tests/PluginTest.php

@@ -0,0 +1,29 @@
+<?php declare(strict_types=1);
+
+namespace Tests\PresProg\PhpBlueprintLoader;
+
+use Kirby\Cms\App;
+use Kirby\Filesystem\Dir;
+
+class TestCase extends \PHPUnit\Framework\TestCase
+{
+    protected static string $tmpDir;
+
+    public static function setUpBeforeClass(): void
+    {
+        self::$tmpDir = __DIR__ . '/temp';
+    }
+
+    protected function setUp(): void
+    {
+        // Do not register any error handler
+        App::$enableWhoops = false;
+
+        Dir::copy(__DIR__ . '/fixtures', self::$tmpDir);
+    }
+
+    protected function tearDown(): void
+    {
+        Dir::remove(self::$tmpDir);
+    }
+}

tests/TestCase.php

@@ -1,3 +1,3 @@
 <?php declare(strict_types=1);
 
-require __DIR__ . '/../classes/PhpBlueprintLoader.php';
+require dirname(__DIR__) . '/vendor/autoload.php';