feat: utiliser les headings de déclaration pour extraire les sections

[yann120]

Contexte

Sur certaines déclarations, l’extraction échoue car le découpage actuel s’appuie sur le heading HTML suivant, ce qui peut tronquer la section analysée.
exemple :

Changements

• ajout d’un découpage métier basé sur les headings attendus d’une déclaration
• extraction du contenu entre un heading de déclaration et le prochain heading de déclaration reconnu dans la page
• factorisation des headings attendus dans AccessibilityDeclarationHeadings

Effet

Corrige le cas de ce site et tout autre page ayant des sous-header dans la déclaration sans modifier le comportement générique de Page.

[Josyann]

Tu peux faire du clean là dedans et laisser un titre de plus au dessus et en dessous si vraiment besoin mais là c'est un peu beaucoup !
Et aussi je sais que notre exemple viens de Carrouf mais utilisons quelque chose de différent :D

[yann120]

J'ai fais du ménage et j'ai modifié les noms de la société, emails

[Josyann]

Suggested change

	module AccessibilityDeclarationHeadings
	EXPECTED_HEADINGS = [
	[2, "État de conformité"],
	[3, "Résultats des tests"],
	[2, "Contenus non accessibles"],
	[3, "Non-conformités"],
	[3, "Dérogations pour charge disproportionnée"],
	[3, "Contenus non soumis à l'obligation d'accessibilité"],
	[2, "Établissement de cette déclaration d'accessibilité"],
	[3, "Technologies utilisées pour la réalisation du site"],
	[3, "Environnement de test"],
	[3, "Outils pour évaluer l'accessibilité"],
	[3, "Pages du site ayant fait l'objet de la vérification de conformité"],
	[2, "Retour d'information et contact"],
	[2, "Voies de recours"],
	].freeze
	def self.expected_heading_titles
	EXPECTED_HEADINGS.map { |_, heading_title| heading_title }
	end
	end
	module AccessibilityDeclarationHeadings
	extend ActiveSupport::Concern
	EXPECTED_HEADINGS = [
	[2, "État de conformité"],
	[3, "Résultats des tests"],
	[2, "Contenus non accessibles"],
	[3, "Non-conformités"],
	[3, "Dérogations pour charge disproportionnée"],
	[3, "Contenus non soumis à l'obligation d'accessibilité"],
	[2, "Établissement de cette déclaration d'accessibilité"],
	[3, "Technologies utilisées pour la réalisation du site"],
	[3, "Environnement de test"],
	[3, "Outils pour évaluer l'accessibilité"],
	[3, "Pages du site ayant fait l'objet de la vérification de conformité"],
	[2, "Retour d'information et contact"],
	[2, "Voies de recours"],
	].freeze
	def expected_headings
	EXPECTED_HEADINGS
	end
	def expected_heading_titles
	expected_headings.map { |_, heading_title| heading_title }
	end
	end

Dans les autres fichiers :

    include AccessibilityDeclarationHeadings

[...]

    def total
      expected_heading_titles.count
    end

[...]

Pas de AccessibilityDeclarationHeadings et c'est construit comme un concern.
Gaffe au class << self dans FindAccessibilityPageService, met le include concern après.

Ça serait peut-être bien qu'on fasse des noms de methods plus explicit ici, d'inclure la notion de la déclaration... à voir 🤷‍♀️

[yann120]

Merci pour ces commentaires constructifs! 😄

Pas de AccessibilityDeclarationHeadings et c'est construit comme un concern.

J'ai converti en un concern

Gaffe au class << self dans FindAccessibilityPageService, met le include concern après.

J'ai mis FindAccessibilityPageService dans le class << self, c'est bien ça ?

Ça serait peut-être bien qu'on fasse des noms de methods plus explicit ici, d'inclure la notion de la déclaration... à voir

J'ai renommé en expected_declaration_headings