Merge pull request #413 from JLG-WOCFR-DEV/codex/add-phpunit-configuration-and-tests-setup

Add PHPUnit configuration and expand notification failure tests

Author: JLG-WOCFR-DEV

.github/workflows/frontend-tests.yml

@@ -10,6 +10,15 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.1'
+          coverage: none
+      - name: Install Composer dependencies
+        run: composer install --no-ansi --no-interaction --no-progress
+      - name: Run PHP unit tests
+        run: composer test:php
       - uses: actions/setup-node@v4
         with:
           node-version: '20'

.gitignore

@@ -4,3 +4,6 @@
 .playwright/
 playwright-report/
 test-results/
+build/
+phpunit.junit.xml
+phpunit.result.cache

README.md

@@ -56,6 +56,11 @@ Liens Morts Detector est une extension WordPress qui détecte les liens et image
 
 ## Tests automatisés
 
+### PHP (PHPUnit)
+- Installer les dépendances PHP : `composer install`.
+- Lancer la suite unitaire : `composer test:php` (génère également un rapport JUnit `phpunit.junit.xml`).
+- ✅ Cette suite est exécutée dans la CI et doit réussir avant toute fusion.
+
 ### JavaScript (Jest)
 - `npm test` exécute l’intégralité de la suite Jest existante.
 
@@ -81,7 +86,8 @@ Liens Morts Detector est une extension WordPress qui détecte les liens et image
 La charte d’interface détaille les styles, couleurs et attentes d’interaction à respecter : voir [`docs/charte-ux.md`](docs/charte-ux.md).
 
 ### Combinaison des suites
-- `npm run test:all` exécute successivement Jest puis Playwright.
+- `composer test:all` enchaîne les tests PHP, Jest puis Playwright.
+- `npm run test:all` reste disponible si seuls les tests JavaScript sont nécessaires.
 
 ## Détection des soft 404
 

composer.json

@@ -9,7 +9,17 @@
         }
     },
     "scripts": {
-        "test:js": "npm test"
+        "test:php": [
+            "@php -r \"if (!is_dir('build')) { mkdir('build'); }\"",
+            "@php -r \"if (!is_dir('build/logs')) { mkdir('build/logs'); }\"",
+            "vendor/bin/phpunit"
+        ],
+        "test:js": "npm test",
+        "test:all": [
+            "@test:php",
+            "@test:js",
+            "npm run test:e2e"
+        ]
     },
     "autoload": {
         "psr-4": {

phpunit.xml.dist

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.6/phpunit.xsd"
+    bootstrap="vendor/autoload.php"
+    colors="true"
+    cacheResult="false"
+>
+    <testsuites>
+        <testsuite name="Liens Morts Detector Test Suite">
+            <directory suffix="Test.php">tests</directory>
+        </testsuite>
+    </testsuites>
+    <logging>
+        <junit outputFile="phpunit.junit.xml"/>
+    </logging>
+</phpunit>

tests/NotificationManagerTest.php

@@ -202,6 +202,52 @@ public function test_send_summary_notifications_records_history_and_throttles():
         $this->assertSame('throttled', $history[3]['status']);
     }
 
+    public function test_send_summary_notifications_records_failed_email_when_wp_mail_returns_false(): void
+    {
+        $testCase = $this;
+
+        Functions\when('wp_mail')->alias(static function ($recipients, $subject, $message) use ($testCase) {
+            $testCase->sentEmails[] = array(
+                'recipients' => $recipients,
+                'subject'    => $subject,
+                'message'    => $message,
+            );
+
+            return false;
+        });
+
+        $manager = new NotificationManager(static function () {
+            return 1500;
+        });
+
+        $summary = array(
+            'subject'       => 'Résumé',
+            'message'       => 'Contenu',
+            'dataset_label' => 'Analyse des liens',
+        );
+
+        $recipients = array('admin@example.test');
+        $settings   = array('url' => 'https://hooks.example.test', 'channel' => 'slack');
+
+        $result = $manager->sendSummaryNotifications('link', $summary, $recipients, array(
+            'context'          => 'scan',
+            'webhook_settings' => $settings,
+        ));
+
+        $failureMessage = 'Échec de l’envoi de l’e-mail pour l’analyse Analyse des liens.';
+
+        $this->assertSame('failed', $result['email']['status']);
+        $this->assertSame($failureMessage, $result['email']['error']);
+        $this->assertCount(1, $this->errorLogs);
+        $this->assertStringContainsString('Failed to send link summary email', $this->errorLogs[0]);
+
+        $history = $manager->getHistoryEntries();
+        $this->assertCount(2, $history);
+        $this->assertSame('failed', $history[0]['status']);
+        $this->assertSame('email', $history[0]['channel']);
+        $this->assertSame($failureMessage, $history[0]['error']);
+    }
+
     public function test_send_webhook_only_returns_wp_error_on_failure_and_logs_history(): void
     {
         $this->throttleWindow = 0;