Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrigir e melhorar validações de afiliação #715

Open
7 tasks
robertatakenaka opened this issue Oct 7, 2024 · 0 comments
Open
7 tasks

Corrigir e melhorar validações de afiliação #715

robertatakenaka opened this issue Oct 7, 2024 · 0 comments
Assignees

Comments

@robertatakenaka
Copy link
Member

robertatakenaka commented Oct 7, 2024

Descrição da tarefa

  • Adicionar teste para AffiliationsListValidation.validate_affiliation_count_article_vs_sub_article #716
  • (bug) Revisar os valores para "title" da resposta, deve ser similar ao nome do método, remover o termo "validation", deve conter o sub-item ou o "sub-item do sub-item", por exemplo: se está sendo validado o DOI. Há o aspecto formato, há o aspecto registrado no CrossRef, há o aspecto unicidade no XML, etc... O title quando necessário é para dar este tipo de contexto, um contexto mais detalhado
  • (melhoria) Usar return no lugar de yield no retorno dos métodos de AffiliationValidation e yield no lugar de yield from
  • (melhoria) Criar AffiliationsListValidation.validate_translated_affiliations e adicione todos os parâmetros de error_level
  • (melhoria) Criar AffiliationsListValidation.validate_affiliations (não traduções) e adicione todos os parâmetros de error_level
  • (melhoria) Revisar validade_affiliations_list e validate, sugiro juntar os dois
  • (melhoria) Crie e use uma nova função para formatar a resposta:
# se data sempre tiver os dados de parent, pode remover o parent e usar data no seu lugar
def format_response(
    title,
   parent,
    item,
    sub_item,
    validation_type,
    is_valid,
    expected,
    obtained,
    advice,
    data,
    error_level,
):
    return {
        "title": title,
             "parent": parent.get("parent"),
            "parent_id": parent.get("parent_id"),
            "parent_article_type": parent.get("parent_article_type"),
            "parent_lang": parent.get("parent_lang"),
        "item": item,
        "sub_item": sub_item,
        "validation_type": validation_type,
        "response": "OK" if is_valid else error_level,
        "expected_value": expected,
        "got_value": obtained,
        "message": f"Got {obtained}, expected {expected}",
        "advice": None if is_valid else advice,
        "data": data,
    }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants