test: ajoute tests unitaires pour 4 hooks RGAA (100% coverage)

Ajoute 39 tests pour atteindre 100% de couverture sur les 4 hooks RGAA :
- test_hooks_card_heading_level.py (8 tests) - h5 vers h3
- test_hooks_search_input_label.py (9 tests) - title recherche
- test_hooks_pagination_title.py (11 tests) - title pagination
- test_hooks_fix_sidemenu_ids.py (11 tests) - IDs uniques sidemenu

Résultats : 58 tests passés, 100% coverage (90 statements, 0 missed)
Fix GitHub Actions coverage failure (40% vers 100%)

Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>

Author: Alexmacapple

tests/unit/hooks/test_hooks_card_heading_level.py

@@ -0,0 +1,80 @@
+"""Tests unitaires pour hooks/card_heading_level.py"""
+
+from hooks.card_heading_level import on_page_content
+
+
+def test_transform_simple_card():
+    """Vérifie la transformation basique h5 → h3 pour une carte DSFR."""
+    html = '<h5 class="fr-card__title" id="sircom">SIRCOM</h5>'
+    result = on_page_content(html, None, None, None)
+    assert '<h3 class="fr-card__title" id="sircom">SIRCOM</h3>' in result
+    assert '<h5' not in result
+
+
+def test_transform_multiple_cards():
+    """Vérifie que plusieurs cartes sont transformées."""
+    html = """
+    <h5 class="fr-card__title" id="card1">Carte 1</h5>
+    <p>Contenu</p>
+    <h5 class="fr-card__title" id="card2">Carte 2</h5>
+    """
+    result = on_page_content(html, None, None, None)
+    assert result.count('<h3 class="fr-card__title"') == 2
+    assert '<h5' not in result
+
+
+def test_preserve_card_title_attributes():
+    """Vérifie que tous les attributs des titres de cartes sont préservés."""
+    html = '<h5 class="fr-card__title custom-class" id="test" data-attr="value">Titre</h5>'
+    result = on_page_content(html, None, None, None)
+    assert '<h3 class="fr-card__title custom-class" id="test" data-attr="value">Titre</h3>' in result
+
+
+def test_only_transform_card_titles():
+    """Vérifie que seuls les h5 avec fr-card__title sont transformés."""
+    html = """
+    <h5>Titre normal H5</h5>
+    <h5 class="other-class">Autre H5</h5>
+    <h5 class="fr-card__title">Carte</h5>
+    """
+    result = on_page_content(html, None, None, None)
+    assert '<h3 class="fr-card__title">Carte</h3>' in result
+
+
+def test_no_card_titles():
+    """Vérifie que le HTML sans titres de cartes n'est pas modifié."""
+    html = "<p>Pas de cartes</p><div>Contenu</div>"
+    result = on_page_content(html, None, None, None)
+    assert result == html
+
+
+def test_card_title_with_rich_content():
+    """Vérifie que le contenu riche du titre est préservé."""
+    html = '<h5 class="fr-card__title"><strong>SIRCOM</strong> - <em>Service</em></h5>'
+    result = on_page_content(html, None, None, None)
+    assert '<h3 class="fr-card__title"><strong>SIRCOM</strong> - <em>Service</em></h3>' in result
+
+
+def test_card_title_multiline():
+    """Vérifie le traitement des titres sur plusieurs lignes."""
+    html = """
+    <h5 class="fr-card__title" id="card">
+        Titre multiligne
+    </h5>
+    """
+    result = on_page_content(html, None, None, None)
+    assert '<h3 class="fr-card__title" id="card">' in result
+    assert '</h3>' in result
+    assert '<h5' not in result
+
+
+def test_card_title_with_various_class_orders():
+    """Vérifie que la classe fr-card__title est détectée quelle que soit sa position."""
+    html = """
+    <h5 class="fr-card__title">Ordre 1</h5>
+    <h5 class="custom fr-card__title">Ordre 2</h5>
+    <h5 class="fr-card__title custom other">Ordre 3</h5>
+    """
+    result = on_page_content(html, None, None, None)
+    assert result.count('<h3') == 3
+    assert '<h5' not in result

tests/unit/hooks/test_hooks_fix_sidemenu_ids.py

@@ -0,0 +1,179 @@
+"""Tests unitaires pour hooks/fix_sidemenu_ids.py"""
+
+from hooks.fix_sidemenu_ids import on_post_page
+
+
+def test_fix_single_sidemenu_id():
+    """Vérifie la correction d'un seul ID dupliqué dans le sidemenu."""
+    html = """
+    <button aria-controls="fr-sidemenu-wrapper">Section 1</button>
+    <div id="fr-sidemenu-wrapper">Contenu</div>
+    """
+    result = on_post_page(html, None, None)
+    assert 'aria-controls="fr-sidemenu-wrapper-1"' in result
+    assert 'id="fr-sidemenu-wrapper-1"' in result
+    assert 'fr-sidemenu-wrapper"' not in result
+
+
+def test_fix_multiple_sidemenu_ids():
+    """Vérifie la correction de plusieurs IDs dupliqués."""
+    html = """
+    <button aria-controls="fr-sidemenu-wrapper">Section 1</button>
+    <div id="fr-sidemenu-wrapper">Contenu 1</div>
+    <button aria-controls="fr-sidemenu-wrapper">Section 2</button>
+    <div id="fr-sidemenu-wrapper">Contenu 2</div>
+    """
+    result = on_post_page(html, None, None)
+    assert 'aria-controls="fr-sidemenu-wrapper-1"' in result
+    assert 'id="fr-sidemenu-wrapper-1"' in result
+    assert 'aria-controls="fr-sidemenu-wrapper-2"' in result
+    assert 'id="fr-sidemenu-wrapper-2"' in result
+    assert 'fr-sidemenu-wrapper"' not in result
+
+
+def test_fix_aria_controls_and_id_pairs():
+    """Vérifie que aria-controls et id sont tous deux numérotés."""
+    html = """
+    <button aria-controls="fr-sidemenu-wrapper">Titre</button>
+    <div class="fr-collapse" id="fr-sidemenu-wrapper">Liste</div>
+    """
+    result = on_post_page(html, None, None)
+    assert result.count('-1"') == 2  # aria-controls-1 et id-1
+
+
+def test_preserve_other_attributes():
+    """Vérifie que les autres attributs sont préservés."""
+    html = """
+    <button class="fr-sidemenu__btn" aria-controls="fr-sidemenu-wrapper" aria-expanded="false">Section</button>
+    <div class="fr-collapse" id="fr-sidemenu-wrapper" data-section="intro">Contenu</div>
+    """
+    result = on_post_page(html, None, None)
+    assert 'class="fr-sidemenu__btn"' in result
+    assert 'aria-expanded="false"' in result
+    assert 'class="fr-collapse"' in result
+    assert 'data-section="intro"' in result
+    assert 'aria-controls="fr-sidemenu-wrapper-1"' in result
+    assert 'id="fr-sidemenu-wrapper-1"' in result
+
+
+def test_no_sidemenu():
+    """Vérifie que le HTML sans sidemenu n'est pas modifié."""
+    html = "<p>Pas de menu</p><div id='autre-id'>Contenu</div>"
+    result = on_post_page(html, None, None)
+    assert result == html
+
+
+def test_sidemenu_with_different_id():
+    """Vérifie que seuls les IDs fr-sidemenu-wrapper sont modifiés."""
+    html = """
+    <button aria-controls="fr-sidemenu-wrapper">Section</button>
+    <div id="fr-sidemenu-wrapper">Menu</div>
+    <button aria-controls="autre-id">Autre</button>
+    <div id="autre-id">Autre contenu</div>
+    """
+    result = on_post_page(html, None, None)
+    assert 'aria-controls="fr-sidemenu-wrapper-1"' in result
+    assert 'id="fr-sidemenu-wrapper-1"' in result
+    assert 'aria-controls="autre-id"' in result  # Inchangé
+    assert 'id="autre-id"' in result  # Inchangé
+
+
+def test_three_or_more_sidemenus():
+    """Vérifie le traitement de 3+ sections de menu."""
+    html = """
+    <button aria-controls="fr-sidemenu-wrapper">S1</button>
+    <div id="fr-sidemenu-wrapper">C1</div>
+    <button aria-controls="fr-sidemenu-wrapper">S2</button>
+    <div id="fr-sidemenu-wrapper">C2</div>
+    <button aria-controls="fr-sidemenu-wrapper">S3</button>
+    <div id="fr-sidemenu-wrapper">C3</div>
+    """
+    result = on_post_page(html, None, None)
+    assert 'wrapper-1"' in result
+    assert 'wrapper-2"' in result
+    assert 'wrapper-3"' in result
+    assert result.count('aria-controls="fr-sidemenu-wrapper-') == 3
+    assert result.count('id="fr-sidemenu-wrapper-') == 3
+
+
+def test_sidemenu_structure_complete():
+    """Vérifie le traitement d'une structure sidemenu DSFR complète."""
+    html = """
+    <nav class="fr-sidemenu" role="navigation">
+        <div class="fr-sidemenu__inner">
+            <button class="fr-sidemenu__btn" aria-controls="fr-sidemenu-wrapper" aria-expanded="false">
+                Introduction
+            </button>
+            <div class="fr-collapse" id="fr-sidemenu-wrapper">
+                <ul class="fr-sidemenu__list">
+                    <li><a href="#section1">Section 1</a></li>
+                </ul>
+            </div>
+        </div>
+    </nav>
+    """
+    result = on_post_page(html, None, None)
+    assert 'aria-controls="fr-sidemenu-wrapper-1"' in result
+    assert 'id="fr-sidemenu-wrapper-1"' in result
+    assert 'class="fr-sidemenu"' in result
+    assert 'class="fr-sidemenu__btn"' in result
+    assert 'class="fr-collapse"' in result
+
+
+def test_sidemenu_multiline():
+    """Vérifie le traitement des éléments sur plusieurs lignes."""
+    html = """
+    <button
+        class="fr-sidemenu__btn"
+        aria-controls="fr-sidemenu-wrapper"
+        aria-expanded="false">
+        Titre
+    </button>
+    <div
+        class="fr-collapse"
+        id="fr-sidemenu-wrapper">
+        Contenu
+    </div>
+    """
+    result = on_post_page(html, None, None)
+    assert 'aria-controls="fr-sidemenu-wrapper-1"' in result
+    assert 'id="fr-sidemenu-wrapper-1"' in result
+
+
+def test_mismatched_aria_id_counts():
+    """Vérifie le traitement avec nombres différents d'aria-controls et id (edge case)."""
+    # Cas anormal mais le hook doit continuer sans crash
+    html = """
+    <button aria-controls="fr-sidemenu-wrapper">S1</button>
+    <button aria-controls="fr-sidemenu-wrapper">S2</button>
+    <div id="fr-sidemenu-wrapper">C1</div>
+    """
+    result = on_post_page(html, None, None)
+    # Le hook traite quand même, même si la structure HTML est cassée
+    assert 'aria-controls="fr-sidemenu-wrapper-1"' in result
+    assert 'aria-controls="fr-sidemenu-wrapper-2"' in result
+    assert 'id="fr-sidemenu-wrapper-1"' in result
+
+
+def test_sidemenu_ids_sequential():
+    """Vérifie que la numérotation est séquentielle et cohérente."""
+    html = """
+    <button aria-controls="fr-sidemenu-wrapper">S1</button>
+    <div id="fr-sidemenu-wrapper">C1</div>
+    <button aria-controls="fr-sidemenu-wrapper">S2</button>
+    <div id="fr-sidemenu-wrapper">C2</div>
+    <button aria-controls="fr-sidemenu-wrapper">S3</button>
+    <div id="fr-sidemenu-wrapper">C3</div>
+    """
+    result = on_post_page(html, None, None)
+
+    # Vérifier ordre séquentiel
+    pos_aria_1 = result.find('aria-controls="fr-sidemenu-wrapper-1"')
+    pos_id_1 = result.find('id="fr-sidemenu-wrapper-1"')
+    pos_aria_2 = result.find('aria-controls="fr-sidemenu-wrapper-2"')
+    pos_id_2 = result.find('id="fr-sidemenu-wrapper-2"')
+    pos_aria_3 = result.find('aria-controls="fr-sidemenu-wrapper-3"')
+    pos_id_3 = result.find('id="fr-sidemenu-wrapper-3"')
+
+    # Ordre: aria-1 < id-1 < aria-2 < id-2 < aria-3 < id-3
+    assert pos_aria_1 < pos_id_1 < pos_aria_2 < pos_id_2 < pos_aria_3 < pos_id_3

tests/unit/hooks/test_hooks_pagination_title.py

@@ -0,0 +1,121 @@
+"""Tests unitaires pour hooks/pagination_title.py"""
+
+from hooks.pagination_title import on_post_page
+
+
+def test_fix_next_pagination_title():
+    """Vérifie la correction du title pour lien 'Page suivante'."""
+    html = '<a class="fr-pagination__link fr-pagination__link--next" href="/page2/" title="SPAN (SG)">Page suivante</a>'
+    result = on_post_page(html, None, None)
+    assert 'title="Page suivante : SPAN (SG)"' in result
+
+
+def test_fix_prev_pagination_title():
+    """Vérifie la correction du title pour lien 'Page précédente'."""
+    html = '<a class="fr-pagination__link fr-pagination__link--prev" href="/page1/" title="Accueil">Page précédente</a>'
+    result = on_post_page(html, None, None)
+    assert 'title="Page précédente : Accueil"' in result
+
+
+def test_preserve_next_and_prev_both():
+    """Vérifie que les deux liens de pagination sont corrigés."""
+    html = """
+    <a class="fr-pagination__link fr-pagination__link--prev" href="/page1/" title="Intro">Page précédente</a>
+    <a class="fr-pagination__link fr-pagination__link--next" href="/page3/" title="Conclusion">Page suivante</a>
+    """
+    result = on_post_page(html, None, None)
+    assert 'title="Page précédente : Intro"' in result
+    assert 'title="Page suivante : Conclusion"' in result
+
+
+def test_preserve_other_attributes():
+    """Vérifie que tous les attributs du lien sont préservés."""
+    html = '<a class="fr-pagination__link fr-pagination__link--next custom-class" href="/page2/" title="SPAN" id="next-link" data-page="2">Page suivante</a>'
+    result = on_post_page(html, None, None)
+    assert 'href="/page2/"' in result
+    assert 'id="next-link"' in result
+    assert 'data-page="2"' in result
+    assert 'custom-class' in result
+    assert 'title="Page suivante : SPAN"' in result
+
+
+def test_no_pagination_links():
+    """Vérifie que le HTML sans pagination n'est pas modifié."""
+    html = "<p>Pas de pagination</p><a href='/autre'>Lien normal</a>"
+    result = on_post_page(html, None, None)
+    assert result == html
+
+
+def test_pagination_title_already_compliant():
+    """Vérifie le traitement d'un title déjà conforme RGAA."""
+    html = '<a class="fr-pagination__link fr-pagination__link--next" href="/page2/" title="Titre original">Page suivante</a>'
+    result = on_post_page(html, None, None)
+    # Le hook ajoute toujours le préfixe, même si déjà présent
+    assert 'title="Page suivante : Titre original"' in result
+
+
+def test_pagination_with_additional_classes():
+    """Vérifie le traitement avec classes CSS supplémentaires."""
+    html1 = '<a class="fr-pagination__link--next other-class" href="/p2/" title="Page 2">Page suivante</a>'
+    html2 = '<a class="other-class fr-pagination__link--prev" href="/p1/" title="Page 1">Page précédente</a>'
+
+    result1 = on_post_page(html1, None, None)
+    result2 = on_post_page(html2, None, None)
+
+    assert 'title="Page suivante : Page 2"' in result1
+    assert 'title="Page précédente : Page 1"' in result2
+
+
+def test_pagination_multiline():
+    """Vérifie le traitement des liens sur plusieurs lignes."""
+    html = """
+    <a class="fr-pagination__link fr-pagination__link--next"
+       href="/page2/"
+       title="SPAN (SG)">
+        Page suivante
+    </a>
+    """
+    result = on_post_page(html, None, None)
+    assert 'title="Page suivante : SPAN (SG)"' in result
+
+
+def test_various_attribute_orders():
+    """Vérifie que la classe fr-pagination__link peut être à différentes positions."""
+    # Note: Le pattern regex nécessite class="..." avant title="..." pour matcher
+    html1 = '<a class="fr-pagination__link--next" title="Page 2" href="/p2/">Page suivante</a>'
+    html2 = '<a class="fr-pagination__link--next" href="/p2/" title="Page 2">Page suivante</a>'
+    html3 = '<a href="/p2/" class="fr-pagination__link--next" title="Page 2">Page suivante</a>'
+
+    result1 = on_post_page(html1, None, None)
+    result2 = on_post_page(html2, None, None)
+    result3 = on_post_page(html3, None, None)
+
+    assert 'title="Page suivante : Page 2"' in result1
+    assert 'title="Page suivante : Page 2"' in result2
+    assert 'title="Page suivante : Page 2"' in result3
+
+
+def test_pagination_with_special_chars_in_title():
+    """Vérifie le traitement des caractères spéciaux dans les titres."""
+    html1 = '<a class="fr-pagination__link--next" href="/p2/" title="L\'accessibilité">Page suivante</a>'
+    html2 = '<a class="fr-pagination__link--next" href="/p2/" title="État & Services">Page suivante</a>'
+    html3 = '<a class="fr-pagination__link--next" href="/p2/" title="Données (2025)">Page suivante</a>'
+
+    result1 = on_post_page(html1, None, None)
+    result2 = on_post_page(html2, None, None)
+    result3 = on_post_page(html3, None, None)
+
+    assert "title=\"Page suivante : L'accessibilité\"" in result1
+    assert 'title="Page suivante : État & Services"' in result2
+    assert 'title="Page suivante : Données (2025)"' in result3
+
+
+def test_pagination_with_different_page_titles():
+    """Vérifie le traitement de divers titres de pages."""
+    html = """
+    <a class="fr-pagination__link--prev" href="/intro/" title="Introduction générale">Page précédente</a>
+    <a class="fr-pagination__link--next" href="/module-sircom/" title="SIRCOM - Services d'information">Page suivante</a>
+    """
+    result = on_post_page(html, None, None)
+    assert 'title="Page précédente : Introduction générale"' in result
+    assert 'title="Page suivante : SIRCOM - Services d\'information"' in result