fix: corrige TypeError em validate_secs e validate_lists para elementos sem atributos obrigatórios (#1227) (#1230)

* bump(version): incrementa versão de 4.16.5 para 4.16.6

- Atualiza __version__ em packtools/version.py de '4.16.5' para '4.16.6'
- Versão incrementada em patch para refletir a correção de bug nas funções
  validate_secs e validate_lists (issue #1227)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(xml_validations): corrige validate_secs, validate_lists e validate_response que não retornavam resultados

Problema (issue #1227):
- As funções validate_secs, validate_lists e validate_response em
  xml_validations.py instanciavam o validador correspondente mas nunca
  chamavam yield from validator.validate(), tornando-as funções regulares
  que retornavam None em vez de generators
- Ao tentar iterar sobre o valor retornado (None), o orquestrador de
  validação lançava TypeError: 'NoneType' object is not iterable
- Adicionalmente, validate_secs e validate_lists acessavam params["sec_rules"]
  e params["list_rules"] diretamente, o que lançava KeyError quando as
  chaves estavam ausentes no dicionário de parâmetros

Correções aplicadas:
- Adicionado `yield from validator.validate()` em validate_lists,
  validate_response e validate_secs, tornando-as generators válidos
- Substituído params["list_rules"] por params.get("list_rules", {})
  para evitar KeyError quando a chave não está presente
- Substituído params["sec_rules"] por params.get("sec_rules", {})
  pelo mesmo motivo

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* test(test_list): adiciona testes de regressão para validate_lists em xml_validations

Contexto (issue #1227):
- Não havia testes para a função wrapper validate_lists de xml_validations.py,
  apenas para a classe ArticleListValidation diretamente
- A ausência de cobertura permitiu que o bug (função retornando None)
  passasse despercebido

Testes adicionados na classe TestValidateListsWrapper:
- test_list_without_list_type_returns_structured_result: garante que
  <list> sem @list-type retorna resultado estruturado com response != "OK"
  em vez de lançar TypeError (cenário exato reportado na issue)
- test_list_with_valid_list_type_returns_ok: garante que <list list-type="bullet">
  retorna response "OK" para a validação de presença do atributo
- test_empty_params_does_not_raise: garante que params vazio ({}) não
  lança KeyError ao chamar validate_lists
- test_no_lists_returns_empty: garante que XML sem elementos <list>
  retorna um iterável vazio

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* test(test_sec): adiciona testes de regressão para validate_secs em xml_validations

Contexto (issue #1227):
- Não havia testes para a função wrapper validate_secs de xml_validations.py,
  apenas para as classes SecValidation e XMLSecValidation diretamente
- A ausência de cobertura permitiu que o bug (função retornando None)
  passasse despercebido

Testes adicionados na classe TestValidateSecsWrapper:
- test_sec_without_title_returns_structured_result: garante que <sec>
  sem <title> retorna resultado estruturado com response != "OK" em vez
  de lançar TypeError (cenário exato reportado na issue)
- test_sec_with_title_returns_ok: garante que <sec> com <title> retorna
  response "OK" para a regra de validação de título
- test_empty_params_does_not_raise: garante que params vazio ({}) não
  lança KeyError ao chamar validate_secs
- test_no_secs_returns_empty: garante que XML do tipo "editorial"
  (sem obrigatoriedade de seções) sem elementos <sec> retorna vazio;
  usa article-type="editorial" para evitar falso positivo de
  data-availability que afetaria article-type="research-article"

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix(xml_validations): reverte para acesso direto params["list_rules"] e params["sec_rules"]

Conforme revisão da robertatakenaka: params.get("list_rules", {}) e
params.get("sec_rules", {}) causavam execução dos validadores sem as
regras configuradas (usando apenas os defaults internos das classes)
quando as chaves estavam ausentes. O acesso direto torna explícito que
list_rules e sec_rules são obrigatórios no dict de parâmetros — o que
sempre ocorre na rota normal via get_default_rules().

Os testes de regressão do wrapper foram atualizados para usar
get_default_rules() em vez de {} como params, garantindo que a
validação seja exercitada com as regras reais de produção (list_rules.json
e sec_rules.json). O teste test_empty_params_does_not_raise foi
substituído por test_validate_*_does_not_raise_typeerror, que testa
a regressão correta do issue #1227 (TypeError por generator ausente).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Rossi-Luciano

packtools/sps/validation/xml_validations.py

@@ -377,7 +377,8 @@ def validate_ext_links(xmltree, params):
 def validate_lists(xmltree, params):
     rules = params["list_rules"]
     validator = ArticleListValidation(xmltree, rules)
-    
+    yield from validator.validate()
+
 
 def validate_graphics(xmltree, params):
     """
@@ -410,7 +411,8 @@ def validate_response(xmltree, params):
     """
     response_rules = params.get("response_rules", {})
     validator = ResponseValidation(xmltree, response_rules)
-    
+    yield from validator.validate()
+
 
 def validate_secs(xmltree, params):
     """
@@ -427,7 +429,8 @@ def validate_secs(xmltree, params):
     """
     sec_rules = params["sec_rules"]
     validator = XMLSecValidation(xmltree, sec_rules)
-    
+    yield from validator.validate()
+
 
 def validate_products(xmltree, params):
     """

packtools/version.py

@@ -1,4 +1,4 @@
 """Single source to the version across setup.py and the whole project.
 """
 from __future__ import unicode_literals
-__version__ = '4.16.5'
+__version__ = '4.16.6'

tests/sps/validation/test_list.py

@@ -2,6 +2,8 @@
 from lxml import etree
 
 from packtools.sps.validation.list import ArticleListValidation
+from packtools.sps.validation.xml_validations import validate_lists
+from packtools.sps.validation.xml_validator_rules import get_default_rules
 
 
 class ListValidationTest(unittest.TestCase):
@@ -416,5 +418,64 @@ def test_list_validation_with_title_present(self):
         self.assertIn("<title> present", title_validation["got_value"])
 
 
+class TestValidateListsWrapper(unittest.TestCase):
+    """Tests for validate_lists() in xml_validations — regression para issue #1227."""
+
+    def setUp(self):
+        self.params = get_default_rules()
+
+    def _make_tree(self, body):
+        xml = (
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink"'
+            ' dtd-version="1.0" article-type="research-article" xml:lang="pt">'
+            f"{body}</article>"
+        )
+        return etree.fromstring(xml.encode())
+
+    def test_list_without_list_type_returns_structured_result(self):
+        """<list> sem @list-type deve retornar resultado estruturado, não TypeError."""
+        tree = self._make_tree(
+            "<body><list>"
+            "<list-item><p>Item 1.</p></list-item>"
+            "<list-item><p>Item 2.</p></list-item>"
+            "</list></body>"
+        )
+        results = list(validate_lists(tree, self.params))
+        self.assertTrue(len(results) > 0)
+        presence_results = [r for r in results if r["title"] == "@list-type presence"]
+        self.assertEqual(len(presence_results), 1)
+        self.assertNotEqual(presence_results[0]["response"], "OK")
+
+    def test_list_with_valid_list_type_returns_ok(self):
+        """<list list-type='bullet'> deve retornar OK para presença e valor."""
+        tree = self._make_tree(
+            '<body><list list-type="bullet">'
+            "<list-item><p>Item 1.</p></list-item>"
+            "<list-item><p>Item 2.</p></list-item>"
+            "</list></body>"
+        )
+        results = list(validate_lists(tree, self.params))
+        presence_results = [r for r in results if r["title"] == "@list-type presence"]
+        self.assertEqual(presence_results[0]["response"], "OK")
+
+    def test_validate_lists_does_not_raise_typeerror(self):
+        """validate_lists deve ser um generator válido, não lançar TypeError (regressão #1227)."""
+        tree = self._make_tree(
+            "<body><list>"
+            "<list-item><p>Item.</p></list-item>"
+            "</list></body>"
+        )
+        try:
+            list(validate_lists(tree, self.params))
+        except TypeError as exc:
+            self.fail(f"validate_lists lançou TypeError: {exc}")
+
+    def test_no_lists_returns_empty(self):
+        """XML sem <list> deve retornar iterável vazio."""
+        tree = self._make_tree("<body><p>Sem listas.</p></body>")
+        results = list(validate_lists(tree, self.params))
+        self.assertEqual(results, [])
+
+
 if __name__ == "__main__":
     unittest.main()

tests/sps/validation/test_sec.py

@@ -16,6 +16,8 @@
 
 from packtools.sps.models.sec import ArticleSecs
 from packtools.sps.validation.sec import SecValidation, XMLSecValidation
+from packtools.sps.validation.xml_validations import validate_secs
+from packtools.sps.validation.xml_validator_rules import get_default_rules
 
 
 class TestSecValidationTitle(unittest.TestCase):
@@ -785,5 +787,58 @@ def test_body_sec_types(self):
         self.assertEqual(sec_types, ["intro", "methods"])
 
 
+class TestValidateSecsWrapper(unittest.TestCase):
+    """Tests for validate_secs() in xml_validations — regression for issue #1227."""
+
+    def setUp(self):
+        self.params = get_default_rules()
+
+    def _make_tree(self, body):
+        xml = (
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink"'
+            ' article-type="research-article" xml:lang="pt">'
+            f"{body}</article>"
+        )
+        return etree.fromstring(xml.encode())
+
+    def test_sec_without_title_returns_structured_result(self):
+        """<sec> sem <title> deve retornar resultado estruturado, não TypeError."""
+        tree = self._make_tree("<body><sec><p>Texto sem titulo.</p></sec></body>")
+        results = list(validate_secs(tree, self.params))
+        self.assertTrue(len(results) > 0)
+        title_results = [r for r in results if r["title"] == "sec title"]
+        self.assertEqual(len(title_results), 1)
+        self.assertNotEqual(title_results[0]["response"], "OK")
+
+    def test_sec_with_title_returns_ok(self):
+        """<sec> com <title> deve retornar OK para a regra de título."""
+        tree = self._make_tree(
+            "<body><sec><title>Intro</title><p>Conteudo.</p></sec></body>"
+        )
+        results = list(validate_secs(tree, self.params))
+        title_results = [r for r in results if r["title"] == "sec title"]
+        self.assertEqual(len(title_results), 1)
+        self.assertEqual(title_results[0]["response"], "OK")
+
+    def test_validate_secs_does_not_raise_typeerror(self):
+        """validate_secs deve ser um generator válido, não lançar TypeError (regressão #1227)."""
+        tree = self._make_tree("<body><sec><p>Texto.</p></sec></body>")
+        try:
+            list(validate_secs(tree, self.params))
+        except TypeError as exc:
+            self.fail(f"validate_secs lançou TypeError: {exc}")
+
+    def test_no_secs_returns_empty(self):
+        """XML sem <sec> e sem article-type que exija seções deve retornar vazio."""
+        xml = (
+            '<article xmlns:xlink="http://www.w3.org/1999/xlink"'
+            ' article-type="editorial" xml:lang="pt">'
+            "<body><p>Sem secoes.</p></body></article>"
+        )
+        tree = etree.fromstring(xml.encode())
+        results = list(validate_secs(tree, self.params))
+        self.assertEqual(results, [])
+
+
 if __name__ == "__main__":
     unittest.main()