[FEAT] Améliorer la recherche du taux

[Josyann]

Description

Sur https://www.carrefour.fr le taux n'est pas trouvé. La REGEX n'est pas complète.

Critères d’acceptation
• Le taux est trouvé : 71.21%

Notes / Contexte

Rajouter

"Le taux global de conformité était de 51,43% en Juin 2023, mis à jour à 71,21% sur l’ensemble critères du référentiel générale d’amélioration de l’accessibilité  (RGAA)." => 71.21

dans spec/models/checks/analyze_accessibility_page_spec.rb:87

[Josyann]

Le test sur carrefour.fr n'est pas concluant, à retester

[yann120]

@Josyann Donc j'ai bien investigué, et en effet j'aurais pas du me fier qu'a mon test sur la Regex et tester en vrai sur l'interface...

La regex est correct maintenant selon moi, mais un problème peut en cacher un autre...

Voici la page de Carrefour a la partie d'extraction du taux :

<div class="editorial-content__container">
   <h2> Résultats des tests </h2>
   <div class="editorial-content__description c-text c-text--size-m c-text--style-p c-text--spacing-default">
      <div>
         <h3><span>Résultats des tests&nbsp;</span></h3>
         <p dir="ltr"><span>L’audit de conformité réalisé par la société&nbsp;</span><strong><u>IPEDIS</u>&nbsp;</strong><span>révèle que :&nbsp;</span></p>
         <ul>
            <li>
               <p dir="ltr"><span>47 critères sont conformes&nbsp;</span></p>
            </li>
            <li>
               <p dir="ltr"><span>19 critères sont non-conformes&nbsp;</span></p>
            </li>
            <li>
               <p dir="ltr"><span>40 critères sont non-applicables&nbsp;</span></p>
            </li>
         </ul>
         <p><span>Le taux global de conformité était de 51,43% en Juin 2023, mis à jour à&nbsp;</span><strong>71,21% sur&nbsp; l’ensemble critères du référentiel générale d’amélioration de l’accessibilité&nbsp; (RGAA)</strong><span>.&nbsp;</span></p>
      </div>
   </div>
</div>

Aujourd'hui heading_relative_to (app/models/page.rb:152) avec :next (principalement utilisé dans source_for) va prendre le contenu jusqu'au prochain heading, peut importe le niveau du prochain heading...
Du coup dans ce cas précis, on va rien prendre comme contenu...

La question que je me pose est : Est-ce que c'est vraiment ce qu'on veut "Jusq'au prochain heading" ?

La solution que je suis entrain de tester sur une branche, serait d'aller jusqu'au prochain heading de même niveau ou de niveau au dessus.
Ce qui fait qu'on accepterait un H3, et meme d'autres niveaux en dessous si ils veulent sous-sous structurer leur page html.
Après-tout c'est comme ca que fonctionne le html et que quand on lit une section avec un titre h2, tout ce qui est en dessous jusqu'au prochain h2 est compris dedans, non ?

Le soucis est que mon changement casse des tests, et a juste raison, car on change notre logique d'extraction

J'appelle notamment dans la discussion notre expert declaration d'accessibilité @Erwan-le-Gall pour nous dire ce qu'on devrait réellement faire : s'arreter au prochain heading peut importe son niveau ou s'arreter au prochain heading de meme niveau ?

Edit : j'ai une PR qui corrige cela si on décide d'aller dans cette direction : #519

[Josyann]

La question que je me pose est : Est-ce que c'est vraiment ce qu'on veut "Jusq'au prochain heading" ?

Oui, on veut séctionner la déclaration

La solution que je suis entrain de tester sur une branche, serait d'aller jusqu'au prochain heading de même niveau ou de niveau au dessus.

Pas la meilleure solution selon moi. Ici on s'en fiche du niveau, en revanche pas de l'intitulé. On pourrait aller au suivant qui correspond à EXPECTED_HEADINGS - current. Ce qui veut dire qu'on se base sur les headings de la décla, on prend le prochain existant et conforme et différent de celui qu'on recherche déjà.

[Josyann]

Après discussion : aujourd'hui la recherche se base sur tous les headings.
Nous voulons, dans le cas des checks liés à la déclaration, nous baser sur les headings attendus d'une déclaration.
Solution :

• Ne pas se soucier du niveau
• Ne pas se soucier de l'exhaustivité de la liste de headings
• Se baser sur l'existant + la liste des headings de la déclaration
• Extraire le contenu présent entre un titre de la déclaration et un autre présent dans la liste (pas le même).

---

Voir : app/services/find_accessibility_page_service.rb:49
Qui pourrait être utile pour la comparaison à faire (probablement par là) dans app/models/page.rb:152