OpenNIT

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109

<?php

declare(strict_types=1);

/**
 * Deckt ab, dass Controller nur eigene Methoden aufrufen, die es auch gibt.
 *
 * `php -l` prüft die Syntax, nicht die Auflösung von Methoden: Ein Aufruf auf
 * eine nicht vorhandene Methode fällt erst beim Seitenaufruf auf und endet dort
 * als interner Fehler. Diese Prüfung zieht den Befund auf die Testzeit vor.
 */
return static function (): void {

    /** @return list<string> Alle Controller-Dateien */
    $dateien = static function (): array {
        $liste = [];
        $lauf  = new RecursiveIteratorIterator(
            new RecursiveDirectoryIterator(
                dirname(__DIR__) . '/src/Controllers',
                FilesystemIterator::SKIP_DOTS
            )
        );
        foreach ($lauf as $datei) {
            if ($datei->isFile() && $datei->getExtension() === 'php') {
                $liste[] = $datei->getPathname();
            }
        }
        sort($liste);
        return $liste;
    };

    /** Voll qualifizierter Klassenname einer Datei. */
    $klasseVon = static function (string $datei): ?string {
        $inhalt = (string)file_get_contents($datei);
        if (!preg_match('/^namespace\s+([^;]+);/m', $inhalt, $ns)) {
            return null;
        }
        if (!preg_match('/^(?:final\s+|abstract\s+)?class\s+(\w+)/m', $inhalt, $cl)) {
            return null;
        }
        return trim($ns[1]) . '\\' . $cl[1];
    };

    /**
     * Alle auf $this aufgerufenen Methodennamen einer Datei.
     * Aufrufe auf andere Objekte bleiben außen vor – deren Klasse steht hier
     * nicht fest.
     *
     * @return list<string>
     */
    $aufrufe = static function (string $datei): array {
        $inhalt = (string)file_get_contents($datei);
        preg_match_all('/\$this->([a-zA-Z_]\w*)\s*\(/', $inhalt, $m);
        return array_values(array_unique($m[1]));
    };

    // ----------------------------------------------------------------
    // Die Prüfung
    // ----------------------------------------------------------------

    test('Jeder Controller-Aufruf auf $this trifft eine vorhandene Methode', function () use ($dateien, $klasseVon, $aufrufe) {
        $geprueft = 0;
        $fehlend  = [];

        foreach ($dateien() as $datei) {
            $klasse = $klasseVon($datei);
            if ($klasse === null || !class_exists($klasse)) {
                continue;
            }
            $reflex = new ReflectionClass($klasse);
            if ($reflex->isAbstract()) {
                continue;
            }
            // Klassen mit __call lösen Methoden zur Laufzeit auf.
            if ($reflex->hasMethod('__call')) {
                continue;
            }

            foreach ($aufrufe($datei) as $name) {
                $geprueft++;
                if (!$reflex->hasMethod($name)) {
                    $fehlend[] = $reflex->getShortName() . '::' . $name . '()';
                }
            }
        }

        assertTrue($geprueft > 200, 'Zu wenige Aufrufe geprüft – die Sammlung greift nicht');
        assertSame([], $fehlend, 'Nicht vorhandene Methoden: ' . implode(', ', $fehlend));
    });

    // ----------------------------------------------------------------
    // Der konkrete Fall
    // ----------------------------------------------------------------

    test('Die Budget-Einstellungen können ihren Wert lesen und schreiben', function () {
        // Die Seite liest den Standard-Steuersatz und schreibt ihn zurück;
        // beides läuft über system_settings.
        $klasse = new ReflectionClass(\OpenNIT\Controllers\Admin\BudgetAdminController::class);

        foreach (['getSetting', 'saveSetting'] as $name) {
            assertTrue($klasse->hasMethod($name), 'Fehlt: ' . $name);
        }

        $src = (string)file_get_contents(
            dirname(__DIR__) . '/src/Controllers/Admin/BudgetAdminController.php'
        );
        assertTrue(str_contains($src, 'SystemSetting($this->db())'), 'Die Ablage weicht vom üblichen Weg ab');
    });
};