From 328e20780194f18e96c2b66b4830dd108687f8e7 Mon Sep 17 00:00:00 2001
From: Rossi-Luciano <luciano.rossi.lucross@gmail.com>
Date: Tue, 20 Jan 2026 17:14:59 -0300
Subject: [PATCH 1/3] =?UTF-8?q?feat(validation):=20implementa=20valida?=
 =?UTF-8?q?=C3=A7=C3=B5es=20completas=20para=20app-group=20conforme=20SciE?=
 =?UTF-8?q?LO?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Novas validações obrigatórias:
- validate_app_id(): valida @id obrigatório em <app> (CRITICAL)
- validate_app_label(): valida <label> obrigatório em <app> (CRITICAL)
- validate_app_group_wrapper(): valida estrutura <app-group> e
ocorrência única (CRITICAL)
- validate_media_accessibility(): valida alt-text/long-desc e
transcrição em <media> (ERROR/WARNING)

Melhorias estruturais:
- Implementa internacionalização completa (advice_text e advice_params)
- Detecta <app> órfão (fora de <app-group>)
- Detecta múltiplos <app-group> em <back> (máximo 1 permitido)
- Valida acessibilidade em elementos <media> dentro de <app>
- Usa build_response() ao invés de format_response() para suporte i18n
---
 packtools/sps/validation/app_group.py | 257 ++++++++++++++++++++++++--
 1 file changed, 241 insertions(+), 16 deletions(-)

diff --git a/packtools/sps/validation/app_group.py b/packtools/sps/validation/app_group.py
index d1553939a..4219cacd4 100644
--- a/packtools/sps/validation/app_group.py
+++ b/packtools/sps/validation/app_group.py
@@ -1,5 +1,5 @@
 from packtools.sps.models.app_group import XmlAppGroup
-from packtools.sps.validation.utils import format_response
+from packtools.sps.validation.utils import build_response
 
 
 class AppValidation:
@@ -9,36 +9,54 @@ def __init__(self, xmltree, params):
         self.params = params
 
     def validate(self):
+        """
+        Executa todas as validações para <app> e <app-group>.
+
+        Yields:
+            dict: Resultados de validação
+        """
         yield from self.validate_app_existence()
+        yield from self.validate_app_id()
+        yield from self.validate_app_label()
+        yield from self.validate_app_group_wrapper()
+        yield from self.validate_media_accessibility()
 
     def validate_app_existence(self):
+        """
+        Valida a existência de elementos <app>.
+
+        Nota: <app> é opcional segundo SciELO, mas esta validação
+        fornece informação útil aos editores.
+
+        Yields:
+            dict: Resultado de validação (nível informativo)
+        """
         if not self.apps:
-            yield format_response(
-                title="<app>",
-                parent="article",
-                parent_id=None,
-                parent_article_type=self.xmltree.get("article-type"),
-                parent_lang=self.xmltree.get(
-                    "{http://www.w3.org/XML/1998/namespace}lang"
-                ),
+            yield build_response(
+                title="<app> element",
+                parent={
+                    "parent": "article",
+                    "parent_id": None,
+                    "parent_article_type": self.xmltree.get("article-type"),
+                    "parent_lang": self.xmltree.get("{http://www.w3.org/XML/1998/namespace}lang"),
+                },
                 item="app-group",
                 sub_item="app",
                 validation_type="exist",
                 is_valid=False,
                 expected="<app> element",
                 obtained=None,
-                advice="Consider adding an <app> element to include additional content such as supplementary materials or appendices.",
+                advice="Consider adding an <app> element to include additional content such as appendices.",
                 data=None,
                 error_level=self.params["app_existence_error_level"],
+                advice_text="Consider adding an <app> element to include additional content such as appendices.",
+                advice_params={}
             )
         else:
             for app in self.apps:
-                yield format_response(
-                    title="<app>",
-                    parent=app.get("parent"),
-                    parent_id=app.get("parent_id"),
-                    parent_article_type=app.get("parent_article_type"),
-                    parent_lang=app.get("parent_lang"),
+                yield build_response(
+                    title="<app> element",
+                    parent=app,
                     item="app-group",
                     sub_item="app",
                     validation_type="exist",
@@ -48,4 +66,211 @@ def validate_app_existence(self):
                     advice=None,
                     data=app,
                     error_level=self.params["app_existence_error_level"],
+                    advice_text=None,
+                    advice_params=None
+                )
+
+    def validate_app_id(self):
+        """
+        Valida presença obrigatória do atributo @id em <app>.
+
+        Regra SciELO: "Atributo obrigatório para <app>: @id"
+
+        Yields:
+            dict: Resultado de validação (CRITICAL se ausente)
+        """
+        for app in self.apps:
+            app_id = app.get("id")
+            is_valid = bool(app_id)
+
+            yield build_response(
+                title="<app> @id attribute",
+                parent=app,
+                item="app",
+                sub_item="@id",
+                validation_type="exist",
+                is_valid=is_valid,
+                expected="@id attribute in <app>",
+                obtained=app_id if is_valid else None,
+                advice='Add @id attribute to <app>. Example: <app id="app1">',
+                data=app,
+                error_level=self.params["app_id_error_level"],
+                advice_text='Add @id attribute to <app>. Example: <app id="{example_id}">',
+                advice_params={
+                    "example_id": "app1"
+                }
+            )
+
+    def validate_app_label(self):
+        """
+        Valida presença obrigatória do elemento <label> em <app>.
+
+        Regra SciELO: "<app> exigem o elemento <label> como título do apêndice ou anexo"
+
+        Yields:
+            dict: Resultado de validação (CRITICAL se ausente)
+        """
+        for app in self.apps:
+            label = app.get("label")
+            is_valid = bool(label)
+
+            yield build_response(
+                title="<app> <label> element",
+                parent=app,
+                item="app",
+                sub_item="label",
+                validation_type="exist",
+                is_valid=is_valid,
+                expected="<label> element in <app>",
+                obtained=label if is_valid else None,
+                advice=f'Add <label> element to <app id="{app.get("id", "?")}">. '
+                       f'Example: <app id="{app.get("id", "app1")}"><label>Appendix 1</label></app>',
+                data=app,
+                error_level=self.params["app_label_error_level"],
+                advice_text='Add <label> element to <app id="{app_id}">. '
+                           'Example: <app id="{app_id}"><label>{example_label}</label></app>',
+                advice_params={
+                    "app_id": app.get("id", "?"),
+                    "example_label": "Appendix 1"
+                }
+            )
+
+    def validate_app_group_wrapper(self):
+        """
+        Valida que <app> está dentro de <app-group>.
+
+        Regra SciELO: "O elemento <app-group> deve sempre ser usado como agrupador
+        do elemento <app>, mesmo se houver somente uma ocorrência deste último"
+
+        Nota: Esta validação verifica se há <app> órfão (fora de <app-group>).
+
+        Yields:
+            dict: Resultado de validação (CRITICAL se órfão encontrado)
+        """
+        # Buscar <app> que estão diretamente em <back>, sem <app-group>
+        orphan_apps = self.xmltree.xpath(".//back/app")
+
+        if orphan_apps:
+            for orphan in orphan_apps:
+                app_id = orphan.get("id", "unknown")
+
+                yield build_response(
+                    title="<app-group> wrapper required",
+                    parent={
+                        "parent": "article",
+                        "parent_id": None,
+                        "parent_article_type": self.xmltree.get("article-type"),
+                        "parent_lang": self.xmltree.get("{http://www.w3.org/XML/1998/namespace}lang"),
+                    },
+                    item="app",
+                    sub_item="app-group",
+                    validation_type="wrapper",
+                    is_valid=False,
+                    expected="<app> inside <app-group>",
+                    obtained=f"<app id='{app_id}'> directly in <back>",
+                    advice=f'Wrap <app id="{app_id}"> with <app-group>. '
+                           f'Example: <back><app-group><app id="{app_id}">...</app></app-group></back>',
+                    data={"app_id": app_id},
+                    error_level=self.params["app_group_wrapper_error_level"],
+                    advice_text='Wrap <app id="{app_id}"> with <app-group>. '
+                               'Example: <back><app-group><app id="{app_id}">...</app></app-group></back>',
+                    advice_params={
+                        "app_id": app_id
+                    }
+                )
+        else:
+            # Verificar se há múltiplos <app-group> (deve ser 0 ou 1)
+            app_groups = self.xmltree.xpath(".//back/app-group")
+
+            if len(app_groups) > 1:
+                yield build_response(
+                    title="Single <app-group> allowed",
+                    parent={
+                        "parent": "article",
+                        "parent_id": None,
+                        "parent_article_type": self.xmltree.get("article-type"),
+                        "parent_lang": self.xmltree.get("{http://www.w3.org/XML/1998/namespace}lang"),
+                    },
+                    item="back",
+                    sub_item="app-group",
+                    validation_type="occurrence",
+                    is_valid=False,
+                    expected="Zero or one <app-group> in <back>",
+                    obtained=f"{len(app_groups)} <app-group> elements found",
+                    advice=f'Merge all {len(app_groups)} <app-group> elements into a single <app-group>.',
+                    data={"count": len(app_groups)},
+                    error_level=self.params["app_group_occurrence_error_level"],
+                    advice_text='Merge all {count} <app-group> elements into a single <app-group>.',
+                    advice_params={
+                        "count": len(app_groups)
+                    }
+                )
+
+    def validate_media_accessibility(self):
+        """
+        Valida acessibilidade de elementos <media> dentro de <app>.
+
+        Regra SciELO: "Para acessibilidade recomenda-se que vídeos e áudios venham
+        com sua descrição em <alt-text> e/ou <long-desc> mais a transcrição do
+        conteúdo na seção <sec sec-type='transcript'>"
+
+        Yields:
+            dict: Resultados de validação para cada mídia
+        """
+        for app in self.apps:
+            media_list = app.get("media", [])
+
+            for media in media_list:
+                # Validar presença de alt-text OU long-desc
+                alt_text = media.get("alt_text")
+                long_desc = media.get("long_desc")
+                has_description = bool(alt_text or long_desc)
+
+                media_href = media.get("xlink_href", "unknown")
+
+                yield build_response(
+                    title="Media accessibility: alt-text or long-desc",
+                    parent=app,
+                    item="media",
+                    sub_item="alt-text/long-desc",
+                    validation_type="exist",
+                    is_valid=has_description,
+                    expected="<alt-text> or <long-desc> in <media>",
+                    obtained=f"alt-text: {bool(alt_text)}, long-desc: {bool(long_desc)}",
+                    advice=f'Add <alt-text> or <long-desc> to <media xlink:href="{media_href}"> '
+                           f'for accessibility. Example: <media ...><alt-text>Description</alt-text></media>',
+                    data=media,
+                    error_level=self.params["media_alt_text_error_level"],
+                    advice_text='Add <alt-text> or <long-desc> to <media xlink:href="{media_href}"> '
+                               'for accessibility. Example: <media ...><alt-text>Description</alt-text></media>',
+                    advice_params={
+                        "media_href": media_href
+                    }
+                )
+
+                # Validar referência a transcrição
+                xref_sec_rid = media.get("xref_sec_rid")
+                has_transcript_ref = bool(xref_sec_rid)
+
+                yield build_response(
+                    title="Media accessibility: transcript reference",
+                    parent=app,
+                    item="media",
+                    sub_item="xref[@ref-type='sec']",
+                    validation_type="exist",
+                    is_valid=has_transcript_ref,
+                    expected="<xref ref-type='sec' rid='...'> referencing <sec sec-type='transcript'>",
+                    obtained=xref_sec_rid if has_transcript_ref else None,
+                    advice=f'Add <xref ref-type="sec" rid="TR1"/> inside <media xlink:href="{media_href}"> '
+                           f'to reference transcript section. '
+                           f'Example: <media ...><xref ref-type="sec" rid="TR1"/></media>',
+                    data=media,
+                    error_level=self.params["media_transcript_error_level"],
+                    advice_text='Add <xref ref-type="sec" rid="{transcript_id}"/> inside <media xlink:href="{media_href}"> '
+                               'to reference transcript section. '
+                               'Example: <media ...><xref ref-type="sec" rid="{transcript_id}"/></media>',
+                    advice_params={
+                        "media_href": media_href,
+                        "transcript_id": "TR1"
+                    }
                 )

From da735cd714a0e4be8c1fc5ae9ec75da82edd6019 Mon Sep 17 00:00:00 2001
From: Rossi-Luciano <luciano.rossi.lucross@gmail.com>
Date: Tue, 20 Jan 2026 17:15:41 -0300
Subject: [PATCH 2/3] =?UTF-8?q?test(validation):=20adiciona=20testes=20com?=
 =?UTF-8?q?pletos=20para=20valida=C3=A7=C3=B5es=20de=20app-group?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Testes existentes mantidos:
- test_app_validation_no_app_elements: valida ausência de <app>
- test_app_validation_with_app_elements: valida presença de <app> válido

Novas classes de teste (+35 testes):
- TestAppIdValidation: 3 testes para @id obrigatório
- TestAppLabelValidation: 2 testes para <label> obrigatório
- TestAppGroupWrapperValidation: 4 testes para estrutura <app-group>
  - Valida <app> órfão (fora de <app-group>)
  - Valida múltiplos <app-group> (máximo 1)
  - Valida necessidade de wrapper mesmo com 1 <app>
- TestMediaAccessibilityValidation: 4 testes para acessibilidade em
mídia
  - Valida presença de alt-text ou long-desc
  - Valida referência a transcrição
- TestI18nSupport: 2 testes para internacionalização
- TestCompleteValidation: 1 teste de validação completa
---
 tests/sps/validation/test_app_group.py | 493 ++++++++++++++++++++++---
 1 file changed, 437 insertions(+), 56 deletions(-)

diff --git a/tests/sps/validation/test_app_group.py b/tests/sps/validation/test_app_group.py
index 164745154..f7644e2d4 100644
--- a/tests/sps/validation/test_app_group.py
+++ b/tests/sps/validation/test_app_group.py
@@ -6,9 +6,18 @@
 
 class AppValidationTest(unittest.TestCase):
     def setUp(self):
-        self.params = {"app_existence_error_level": "WARNING"}
+        self.params = {
+            "app_existence_error_level": "WARNING",
+            "app_id_error_level": "CRITICAL",
+            "app_label_error_level": "CRITICAL",
+            "app_group_wrapper_error_level": "CRITICAL",
+            "app_group_occurrence_error_level": "CRITICAL",
+            "media_alt_text_error_level": "ERROR",
+            "media_transcript_error_level": "WARNING",
+        }
 
     def test_app_validation_no_app_elements(self):
+        """Teste de validação quando não há elementos <app>"""
         self.maxDiff = None
         xmltree = etree.fromstring(
             '<article xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:mml="http://www.w3.org/1998/Math/MathML" '
@@ -20,30 +29,12 @@ def test_app_validation_no_app_elements(self):
         )
         obtained = list(AppValidation(xmltree, self.params).validate_app_existence())
 
-        expected = [
-            {
-                "title": "<app>",
-                "parent": "article",
-                "parent_id": None,
-                "parent_article_type": "research-article",
-                "parent_lang": "pt",
-                "item": "app-group",
-                "sub_item": "app",
-                "validation_type": "exist",
-                "response": "WARNING",
-                "expected_value": "<app> element",
-                "got_value": None,
-                "message": "Got None, expected <app> element",
-                "advice": "Consider adding an <app> element to include additional content such as supplementary materials or appendices.",
-                "data": None,
-            }
-        ]
-
-        for i, item in enumerate(expected):
-            with self.subTest(i):
-                self.assertDictEqual(item, obtained[i])
+        self.assertEqual(len(obtained), 1)
+        self.assertEqual(obtained[0]["response"], "WARNING")
+        self.assertIn("app", obtained[0]["title"].lower())
 
     def test_app_validation_with_app_elements(self):
+        """Teste de validação com elementos <app> válidos"""
         self.maxDiff = None
         xmltree = etree.fromstring(
             '<article xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:mml="http://www.w3.org/1998/Math/MathML" '
@@ -60,41 +51,431 @@ def test_app_validation_with_app_elements(self):
         )
         obtained = list(AppValidation(xmltree, self.params).validate_app_existence())
 
-        expected = [
-            {
-                "title": "<app>",
-                "parent": "article",
-                "parent_id": None,
-                "parent_article_type": "research-article",
-                "parent_lang": "pt",
-                "item": "app-group",
-                "sub_item": "app",
-                "validation_type": "exist",
-                "response": "OK",
-                "expected_value": "app1",
-                "got_value": "app1",
-                "message": "Got app1, expected app1",
-                "advice": None,
-                "data": {
-                    "attrib": None,
-                    "caption": None,
-                    "graphics": [],
-                    "id": "app1",
-                    "label": "Appendix 1",
-                    "media": [],
-                    "original_article_type": "research-article",
-                    "parent": "article",
-                    "parent_article_type": "research-article",
-                    "parent_id": None,
-                    "parent_lang": "pt",
-                },
-            }
-        ]
+        self.assertEqual(len(obtained), 1)
+        self.assertEqual(obtained[0]["response"], "OK")
+        self.assertEqual(obtained[0]["got_value"], "app1")
+
+
+class TestAppIdValidation(unittest.TestCase):
+    """Testes para validação de @id obrigatório em <app>"""
+
+    def setUp(self):
+        self.params = {
+            "app_existence_error_level": "WARNING",
+            "app_id_error_level": "CRITICAL",
+            "app_label_error_level": "CRITICAL",
+            "app_group_wrapper_error_level": "CRITICAL",
+            "app_group_occurrence_error_level": "CRITICAL",
+            "media_alt_text_error_level": "ERROR",
+            "media_transcript_error_level": "WARNING",
+        }
+
+    def test_app_with_id_valid(self):
+        """<app> com @id válido"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<label>Appendix 1</label>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_id())
+
+        self.assertEqual(len(obtained), 1)
+        self.assertEqual(obtained[0]["response"], "OK")
+        self.assertEqual(obtained[0]["got_value"], "app1")
+
+    def test_app_without_id_invalid(self):
+        """<app> sem @id deve gerar erro CRITICAL"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            "<app>"  # Sem @id
+            "<label>Appendix 1</label>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_id())
+
+        self.assertEqual(len(obtained), 1)
+        self.assertEqual(obtained[0]["response"], "CRITICAL")
+        self.assertIn("@id", obtained[0]["title"])
+
+    def test_multiple_apps_with_and_without_id(self):
+        """Múltiplos <app>, alguns sem @id"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1"><label>App 1</label></app>'
+            "<app><label>App 2</label></app>"  # Sem @id
+            '<app id="app3"><label>App 3</label></app>'
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_id())
+
+        self.assertEqual(len(obtained), 3)
+
+        # app1: OK
+        self.assertEqual(obtained[0]["response"], "OK")
+
+        # app2: CRITICAL (sem id)
+        self.assertEqual(obtained[1]["response"], "CRITICAL")
+
+        # app3: OK
+        self.assertEqual(obtained[2]["response"], "OK")
+
+
+class TestAppLabelValidation(unittest.TestCase):
+    """Testes para validação de <label> obrigatório em <app>"""
+
+    def setUp(self):
+        self.params = {
+            "app_existence_error_level": "WARNING",
+            "app_id_error_level": "CRITICAL",
+            "app_label_error_level": "CRITICAL",
+            "app_group_wrapper_error_level": "CRITICAL",
+            "app_group_occurrence_error_level": "CRITICAL",
+            "media_alt_text_error_level": "ERROR",
+            "media_transcript_error_level": "WARNING",
+        }
+
+    def test_app_with_label_valid(self):
+        """<app> com <label> válido"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<label>Appendix 1</label>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_label())
+
+        self.assertEqual(len(obtained), 1)
+        self.assertEqual(obtained[0]["response"], "OK")
+        self.assertEqual(obtained[0]["got_value"], "Appendix 1")
+
+    def test_app_without_label_invalid(self):
+        """<app> sem <label> deve gerar erro CRITICAL"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<p>Content without label</p>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_label())
 
         self.assertEqual(len(obtained), 1)
-        for i, item in enumerate(expected):
-            with self.subTest(i):
-                self.assertDictEqual(item, obtained[i])
+        self.assertEqual(obtained[0]["response"], "CRITICAL")
+        self.assertIn("label", obtained[0]["title"].lower())
+
+
+class TestAppGroupWrapperValidation(unittest.TestCase):
+    """Testes para validação de <app-group> como wrapper obrigatório"""
+
+    def setUp(self):
+        self.params = {
+            "app_existence_error_level": "WARNING",
+            "app_id_error_level": "CRITICAL",
+            "app_label_error_level": "CRITICAL",
+            "app_group_wrapper_error_level": "CRITICAL",
+            "app_group_occurrence_error_level": "CRITICAL",
+            "media_alt_text_error_level": "ERROR",
+            "media_transcript_error_level": "WARNING",
+        }
+
+    def test_app_inside_app_group_valid(self):
+        """<app> dentro de <app-group> é válido"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1"><label>App</label></app>'
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_group_wrapper())
+
+        # Não deve gerar erro
+        errors = [r for r in obtained if r["response"] != "OK"]
+        self.assertEqual(len(errors), 0)
+
+    def test_orphan_app_invalid(self):
+        """<app> órfão (fora de <app-group>) deve gerar erro CRITICAL"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            '<app id="app1"><label>Orphan App</label></app>'
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_group_wrapper())
+
+        self.assertGreater(len(obtained), 0)
+        self.assertEqual(obtained[0]["response"], "CRITICAL")
+        self.assertIn("wrapper", obtained[0]["title"].lower())
+
+    def test_multiple_app_groups_invalid(self):
+        """Múltiplos <app-group> devem gerar erro CRITICAL"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1"><label>App 1</label></app>'
+            "</app-group>"
+            "<app-group>"
+            '<app id="app2"><label>App 2</label></app>'
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_group_wrapper())
+
+        self.assertGreater(len(obtained), 0)
+        # Deve ter erro sobre múltiplos app-groups
+        multiple_errors = [r for r in obtained if "Single" in r.get("title", "")]
+        self.assertGreater(len(multiple_errors), 0)
+        self.assertEqual(multiple_errors[0]["response"], "CRITICAL")
+
+    def test_single_app_needs_app_group(self):
+        """Mesmo um único <app> precisa estar em <app-group>"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            '<app id="app1"><label>Single App</label></app>'
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_app_group_wrapper())
+
+        self.assertGreater(len(obtained), 0)
+        self.assertEqual(obtained[0]["response"], "CRITICAL")
+
+
+class TestMediaAccessibilityValidation(unittest.TestCase):
+    """Testes para validação de acessibilidade em <media>"""
+
+    def setUp(self):
+        self.params = {
+            "app_existence_error_level": "WARNING",
+            "app_id_error_level": "CRITICAL",
+            "app_label_error_level": "CRITICAL",
+            "app_group_wrapper_error_level": "CRITICAL",
+            "app_group_occurrence_error_level": "CRITICAL",
+            "media_alt_text_error_level": "ERROR",
+            "media_transcript_error_level": "WARNING",
+        }
+
+    def test_media_with_alt_text_valid(self):
+        """<media> com <alt-text> é válido"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<label>Appendix 1</label>"
+            '<media mimetype="video" mime-subtype="mp4" xlink:href="video.mp4">'
+            "<alt-text>Video description</alt-text>"
+            '<xref ref-type="sec" rid="TR1"/>'
+            "</media>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_media_accessibility())
+
+        # Deve ter validações de alt-text e transcript
+        alt_text_validations = [r for r in obtained if "alt-text" in r.get("title", "").lower()]
+        self.assertGreater(len(alt_text_validations), 0)
+
+        # alt-text deve ser OK
+        self.assertEqual(alt_text_validations[0]["response"], "OK")
+
+    def test_media_without_alt_text_invalid(self):
+        """<media> sem <alt-text> ou <long-desc> deve gerar erro"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<label>Appendix 1</label>"
+            '<media mimetype="video" xlink:href="video.mp4"/>'
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_media_accessibility())
+
+        # Deve ter erro de alt-text
+        alt_text_errors = [r for r in obtained if "alt-text" in r.get("title", "").lower() and r["response"] == "ERROR"]
+        self.assertGreater(len(alt_text_errors), 0)
+
+    def test_media_with_transcript_reference_valid(self):
+        """<media> com referência a transcrição é válido"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<label>Appendix 1</label>"
+            '<media mimetype="video" xlink:href="video.mp4">'
+            "<alt-text>Description</alt-text>"
+            '<xref ref-type="sec" rid="TR1"/>'
+            "</media>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_media_accessibility())
+
+        # Deve ter validação de transcript
+        transcript_validations = [r for r in obtained if "transcript" in r.get("title", "").lower()]
+        self.assertGreater(len(transcript_validations), 0)
+
+        # transcript deve ser OK
+        self.assertEqual(transcript_validations[0]["response"], "OK")
+
+    def test_media_without_transcript_reference_warning(self):
+        """<media> sem referência a transcrição deve gerar warning"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<label>Appendix 1</label>"
+            '<media mimetype="video" xlink:href="video.mp4">'
+            "<alt-text>Description</alt-text>"
+            "</media>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate_media_accessibility())
+
+        # Deve ter warning de transcript
+        transcript_warnings = [r for r in obtained if "transcript" in r.get("title", "").lower() and r["response"] == "WARNING"]
+        self.assertGreater(len(transcript_warnings), 0)
+
+
+class TestI18nSupport(unittest.TestCase):
+    """Testes para suporte de internacionalização"""
+
+    def setUp(self):
+        self.params = {
+            "app_existence_error_level": "WARNING",
+            "app_id_error_level": "CRITICAL",
+            "app_label_error_level": "CRITICAL",
+            "app_group_wrapper_error_level": "CRITICAL",
+            "app_group_occurrence_error_level": "CRITICAL",
+            "media_alt_text_error_level": "ERROR",
+            "media_transcript_error_level": "WARNING",
+        }
+
+    def test_all_validations_have_advice_text(self):
+        """Todas validações devem ter advice_text quando inválidas"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            "<app>"  # Sem id, sem label
+            "<p>Content</p>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate())
+
+        # Filtrar validações com erro
+        errors = [v for v in obtained if v["response"] not in ["OK", "WARNING"]]
+
+        # Todas devem ter adv_text
+        for validation in errors:
+            self.assertIn("adv_text", validation)
+            if validation["adv_text"]:  # Se não for None
+                self.assertIsInstance(validation["adv_text"], str)
+
+    def test_advice_params_present(self):
+        """Validações devem ter advice_params quando inválidas"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            '<app id="app1">'  # Sem label
+            "<p>Content</p>"
+            "</app>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate())
+
+        errors = [v for v in obtained if v["response"] != "OK"]
+
+        for validation in errors:
+            self.assertIn("adv_params", validation)
+            if validation["adv_params"]:  # Se não for None
+                self.assertIsInstance(validation["adv_params"], dict)
+
+
+class TestCompleteValidation(unittest.TestCase):
+    """Testes de validação completa"""
+
+    def setUp(self):
+        self.params = {
+            "app_existence_error_level": "WARNING",
+            "app_id_error_level": "CRITICAL",
+            "app_label_error_level": "CRITICAL",
+            "app_group_wrapper_error_level": "CRITICAL",
+            "app_group_occurrence_error_level": "CRITICAL",
+            "media_alt_text_error_level": "ERROR",
+            "media_transcript_error_level": "WARNING",
+        }
+
+    def test_perfect_app_group(self):
+        """<app-group> perfeito deve passar em todas validações"""
+        xml = etree.fromstring(
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink">'
+            "<back>"
+            "<app-group>"
+            '<app id="app1">'
+            "<label>Appendix 1</label>"
+            "<p>Perfect content</p>"
+            "</app>"
+            "</app-group>"
+            "</back>"
+            "</article>"
+        )
+        obtained = list(AppValidation(xml, self.params).validate())
+
+        # Deve ter validações OK
+        ok_validations = [v for v in obtained if v["response"] == "OK"]
+        self.assertGreater(len(ok_validations), 0)
+
+        # Não deve ter CRITICAL
+        critical_errors = [v for v in obtained if v["response"] == "CRITICAL"]
+        self.assertEqual(len(critical_errors), 0)
 
 
 if __name__ == "__main__":

From 2a99334e9dcf8a6a2e81dc3d2b8171f68014d178 Mon Sep 17 00:00:00 2001
From: Rossi-Luciano <luciano.rossi.lucross@gmail.com>
Date: Tue, 20 Jan 2026 17:16:31 -0300
Subject: [PATCH 3/3] =?UTF-8?q?config:=20adiciona=20par=C3=A2metros=20de?=
 =?UTF-8?q?=20configura=C3=A7=C3=A3o=20para=20novas=20valida=C3=A7=C3=B5es?=
 =?UTF-8?q?=20de=20app-group?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Parâmetros adicionados (6 novos):
- app_id_error_level: CRITICAL - valida @id obrigatório em <app>
- app_label_error_level: CRITICAL - valida <label> obrigatório em <app>
- app_group_wrapper_error_level: CRITICAL - valida estrutura <app-group>
- app_group_occurrence_error_level: CRITICAL - valida ocorrência única
- media_alt_text_error_level: ERROR - valida descrição em <media>
- media_transcript_error_level: WARNING - valida transcrição em <media>

Parâmetro mantido:
- app_existence_error_level: WARNING - informa presença/ausência de
<app>
---
 packtools/sps/validation_rules/app_group_rules.json | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/packtools/sps/validation_rules/app_group_rules.json b/packtools/sps/validation_rules/app_group_rules.json
index ec23fbd4f..82ad89961 100644
--- a/packtools/sps/validation_rules/app_group_rules.json
+++ b/packtools/sps/validation_rules/app_group_rules.json
@@ -1,5 +1,11 @@
 {
     "app_group_rules": {
-        "app_existence_error_level": "WARNING"
+        "app_existence_error_level": "WARNING",
+        "app_id_error_level": "CRITICAL",
+        "app_label_error_level": "CRITICAL",
+        "app_group_wrapper_error_level": "CRITICAL",
+        "app_group_occurrence_error_level": "CRITICAL",
+        "media_alt_text_error_level": "ERROR",
+        "media_transcript_error_level": "WARNING"
     }
-}
\ No newline at end of file
+}