fix(diasuteis): corrige feriados em intervalos entre anos by lucianopf · Pull Request #922 · BrasilAPI/BrasilAPI · GitHub
Skip to content

fix(diasuteis): corrige feriados em intervalos entre anos - #922

Merged
lucianopf merged 1 commit into
mainfrom
fix/diasuteis-multiyear-holidays
Oct 15, 2026
Merged

lucianopf merged 1 commit into
mainfrom
fix/diasuteis-multiyear-holidays

Conversation

@lucianopf

Copy link
Copy Markdown
Member

Descrição

Corrige o erro 500 em intervalos que atravessam anos com feriados nacionais ativados. Array.flatMap passa o índice como segundo argumento; ao receber getHolidays diretamente, esse índice era interpretado como UF. A chamada agora repassa somente o ano.

Acrescenta dois testes HTTP de regressão: intervalo entre anos com feriados ativados e com feriados desativados.

Tipo de mudança

  • Correção de bug
  • Testes

Compatibilidade

  • Não remove ou renomeia campos, tipos ou URLs.
  • Sem novas dependências, alterações de documentação pública ou chamadas externas adicionais.

Validação local

Validação conjunta das duas correções preparadas, com TZ=UTC:

  • 18 testes aprovados (10 de dias úteis e 8 de contrato NCM).
  • 7 E2E de NCM pulados pela verificação de disponibilidade preexistente da fonte. Esses testes e a suíte completa não são declarados como aprovados.
  • ESLint dos arquivos alterados e de pages/, services/, lib/, util/, middlewares/ e errors/: aprovado.
  • Build de produção: aprovado.
  • git diff --check: aprovado.

PR aberto como draft para revisão.

Como testar

Com a aplicação iniciada, executar:

TZ=UTC npm test -- tests/diasuteis-v1.test.js --coverage.enabled=false

Antes da correção: 1 falha (HTTP 500) e 9 testes aprovados. Depois: 10 aprovados.

Relação com outros PRs

Complementar ao #907: este PR corrige a passagem indevida do índice como UF; não inclui a mudança de fuso horário proposta lá. A validação HTTP deste PR foi feita com TZ=UTC. Uma verificação isolada do handler combinando as duas correções passou em America/Sao_Paulo; isso não representa uma execução HTTP completa da combinação.

@vercel

vercel Bot commented Oct 15, 2026

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@lucianopf
lucianopf marked this pull request as ready for review October 15, 2026 11:50
@lucianopf
lucianopf merged commit 4b552e7 into main Oct 15, 2026
3 of 4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 46c3e938 Deployed Oct 15, 2026 by vercel[bot]
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

Successfully merging this pull request may close these issues.

1 participant